From ca09ea0a8b292a6a06a3c05c2a375817d02031e1 Mon Sep 17 00:00:00 2001 From: Nick Mathewson Date: Fri, 24 Aug 2012 12:48:23 -0400 Subject: [PATCH] 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 file changed, 2 insertions(+), 8 deletions(-) diff --git a/src/or/nodelist.h b/src/or/nodelist.h index c93b67a252..5558827961 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);