[master] 2c2ef29b6 mgt_param: Remove stale comment

Dridi Boukelmoune dridi.boukelmoune at gmail.com
Wed Oct 25 07:46:11 UTC 2023


commit 2c2ef29b6b2d34e12343aa7f9489d07989cfb9a3
Author: Dridi Boukelmoune <dridi.boukelmoune at gmail.com>
Date:   Wed Oct 25 09:14:23 2023 +0200

    mgt_param: Remove stale comment
    
    We've had contextual min/max/default values in the param.show and
    param.set output and the manual for a while now.

diff --git a/bin/varnishd/mgt/mgt_param.c b/bin/varnishd/mgt/mgt_param.c
index b34fe6772..1cfd789ba 100644
--- a/bin/varnishd/mgt/mgt_param.c
+++ b/bin/varnishd/mgt/mgt_param.c
@@ -857,11 +857,6 @@ MCF_DumpRstParam(void)
 		MCF_DYN_REASON(Minimum, min);
 		MCF_DYN_REASON(Maximum, max);
 #undef MCF_DYN_REASON
-		/*
-		 * XXX: we should mark the params with one/two flags
-		 * XXX: that say if ->min/->max are valid, so we
-		 * XXX: can emit those also in help texts.
-		 */
 		if (pp->flags & ~DOCS_FLAGS) {
 			printf("\t* Flags: ");
 			q = "";


More information about the varnish-commit mailing list