commit ca09ea0a8b292a6a06a3c05c2a375817d02031e1 Author: Nick Mathewson nickm@torproject.org 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);