[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

[tor-commits] [tor/master] Make node_assert_ok less duplicatey



commit ca09ea0a8b292a6a06a3c05c2a375817d02031e1
Author: Nick Mathewson <nickm@xxxxxxxxxxxxxx>
Date:   Fri Aug 24 12:48:23 2012 -0400

    Make node_assert_ok less duplicatey
    
    This comes at the cost of making its failure message a little less
    friendly, but since when do assertion failures count as
    user-friendly?
---
 src/or/nodelist.h |   10 ++--------
 1 files changed, 2 insertions(+), 8 deletions(-)

diff --git a/src/or/nodelist.h b/src/or/nodelist.h
index c93b67a..5558827 100644
--- a/src/or/nodelist.h
+++ b/src/or/nodelist.h
@@ -12,14 +12,8 @@
 #ifndef _TOR_NODELIST_H
 #define _TOR_NODELIST_H
 
-/* XXX duplicating code from tor_assert(). */
-#define node_assert_ok(n) STMT_BEGIN                            \
-  if (PREDICT_UNLIKELY((n)->ri == NULL && (n)->rs == NULL)) {   \
-    log_err(LD_BUG, "%s:%d: %s: Node is invalid; aborting.",    \
-            _SHORT_FILE_, __LINE__, __func__);                  \
-    fprintf(stderr, "%s:%d: %s: Node is invalid; aborting.\n",  \
-            _SHORT_FILE_, __LINE__, __func__);                  \
-    abort();                                                    \
+#define node_assert_ok(n) STMT_BEGIN {                          \
+    tor_assert((n)->ri || (n)->rs);                             \
   } STMT_END
 
 node_t *node_get_mutable_by_id(const char *identity_digest);

_______________________________________________
tor-commits mailing list
tor-commits@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits