Let's have only one FREE_AND_NULL variant.

This commit removes the old FREE_AND_NULL, and renames the old
FREE_AND_NULL_UNMATCHED so that it is now called FREE_AND_NULL.

This will break all the FREE_AND_NULL_* users; the next commit will
fix them.

diff --git a/src/common/util.h b/src/common/util.h
index e85be57c07f57c037afd8c8ba4764d32ad25881b..97ad15f8347bb356cbb4418207448b19e82481d2 100644 (file)
--- a/src/common/util.h
+++ b/src/common/util.h
@@ -121,11 +121,8 @@ void tor_log_mallinfo(int severity);
 
 /* Helper macro: free a variable of type 'typename' using freefn, and
  * set the variable to NULL.
- *
- * We use this for legacy cases when freefn and typename don't line up
- * perfectly.
  */
-#define FREE_AND_NULL_UNMATCHED(typename, freefn, var)                  \
+#define FREE_AND_NULL(typename, freefn, var)                            \
   do {                                                                  \
     /* only evaluate (var) once. */                                     \
     typename **tmp__free__ptr ## freefn = &(var);                       \
@@ -133,11 +130,6 @@ void tor_log_mallinfo(int severity);
     (*tmp__free__ptr ## freefn) = NULL;                                 \
   } while (0)
 
-/* Helper macro: free a variable of type 'type' using type_free_, and
- * set the variable to NULL. */
-#define FREE_AND_NULL(type, var)                                        \
-  FREE_AND_NULL_UNMATCHED(type ## _t, type ## _free_, (var))
-
 /** Macro: yield a pointer to the field at position <b>off</b> within the
  * structure <b>st</b>.  Example:
  * <pre>