Clean up a few rbt comments

Avoid HTML entities, and describe what a function does
instead of explaining why it used to be a macro.

diff --git a/lib/dns/rbt.c b/lib/dns/rbt.c
index affb764d09376214129fcedd338f5a7dc443dab3..194a4585eac88e1eebc4bb59e725a4bffd66a287 100644 (file)
--- a/lib/dns/rbt.c
+++ b/lib/dns/rbt.c
@@ -85,12 +85,11 @@ struct dns_rbt {
  * The variable length stuff stored after the node has the following
  * structure.
  *
- *     <name_data>{1..255}<oldoffsetlen>{1}<offsets>{1..128}
+ *     NAME_DATA{1..255} OLDOFFSETLEN{1} OFFSETS{1..128}
  *
- * <name_data> contains the name of the node when it was created.
- * <oldoffsetlen> contains the length of <offsets> when the node
- * was created.
- * <offsets> contains the offsets into name for each label when the node
+ * NAME_DATA contains the name of the node when it was created.
+ * OLDOFFSETLEN contains the length of OFFSETS when the node was created.
+ * OFFSETS contains the offsets into name for each label when the node
  * was created.
  */
 
@@ -122,12 +121,9 @@ struct dns_rbt {
                (chain)->levels[(chain)->level_count++] = (node);  \
        } while (0)
 
-/*%
- * The following macros directly access normally private name variables.
- * These macros are used to avoid a lot of function calls in the critical
- * path of the tree traversal code.
+/*
+ * Initialize a dns_name_t that refers to a node's name.
  */
-
 static void
 node_name(dns_rbtnode_t *node, dns_name_t *name) {
        name->length = node->namelen;