]> git.puffer.fish Git - matthieu/frr.git/commitdiff
pimd: replace CPP_NOTICE lines with TODO comments
authorMark Stapp <mjs@labn.net>
Thu, 4 May 2023 14:26:33 +0000 (10:26 -0400)
committerMark Stapp <mjs@labn.net>
Thu, 4 May 2023 14:26:33 +0000 (10:26 -0400)
Replace the noisy CPP_NOTICE lines with TODO comments.

Signed-off-by: Mark Stapp <mjs@labn.net>
pimd/pim6_mld.c

index 5eedfb7dcada663cecb86d7b6d9ca0ddb57049d5..aa39be63dbbb00e0ffb50b12695214849eef58c1 100644 (file)
@@ -344,7 +344,8 @@ static const char *const gm_states[] = {
 };
 /* clang-format on */
 
-CPP_NOTICE("TODO: S,G entries in EXCLUDE (i.e. prune) unsupported");
+/* TODO: S,G entries in EXCLUDE (i.e. prune) unsupported" */
+
 /* tib_sg_gm_prune() below is an "un-join", it doesn't prune S,G when *,G is
  * joined.  Whether we actually want/need to support this is a separate
  * question - it is almost never used.  In fact this is exactly what RFC5790
@@ -645,7 +646,7 @@ static void gm_handle_v2_pass1(struct gm_packet_state *pkt,
                         */
                        gm_packet_sg_drop(old_grp);
                        gm_sg_update(grp, false);
-                       CPP_NOTICE("need S,G PRUNE => NO_INFO transition here");
+/* TODO "need S,G PRUNE => NO_INFO transition here" */
                }
                break;
 
@@ -793,7 +794,8 @@ static void gm_handle_v2_pass2_excl(struct gm_packet_state *pkt, size_t offs)
        gm_sg_update(sg_grp, false);
 }
 
-CPP_NOTICE("TODO: QRV/QQIC are not copied from queries to local state");
+/* TODO: QRV/QQIC are not copied from queries to local state" */
+
 /* on receiving a query, we need to update our robustness/query interval to
  * match, so we correctly process group/source specific queries after last
  * member leaves
@@ -949,7 +951,8 @@ static void gm_handle_v1_report(struct gm_if *gm_ifp,
 
        item = gm_packet_sg_setup(pkt, grp, true, false);
        item->n_exclude = 0;
-       CPP_NOTICE("set v1-seen timer on grp here");
+
+/* TODO "set v1-seen timer on grp here" */
 
        /* } */
 
@@ -1012,7 +1015,9 @@ static void gm_handle_v1_leave(struct gm_if *gm_ifp,
                if (old_grp) {
                        gm_packet_sg_drop(old_grp);
                        gm_sg_update(grp, false);
-                       CPP_NOTICE("need S,G PRUNE => NO_INFO transition here");
+
+/* TODO "need S,G PRUNE => NO_INFO transition here" */
+
                }
        }