r1795 - trunk/varnish-cache/bin/varnishd

phk at projects.linpro.no phk at projects.linpro.no
Sun Aug 5 21:37:44 CEST 2007


Author: phk
Date: 2007-08-05 21:37:44 +0200 (Sun, 05 Aug 2007)
New Revision: 1795

Modified:
   trunk/varnish-cache/bin/varnishd/mgt_param.c
Log:
Synchronize the paramters after we call their accessor functions rather
than when we don't find one.

This makes changing runtime paramters work again, without the need to
ask for a nonexistent parameter to trigger the update.

Ticket: 136



Modified: trunk/varnish-cache/bin/varnishd/mgt_param.c
===================================================================
--- trunk/varnish-cache/bin/varnishd/mgt_param.c	2007-08-03 18:50:05 UTC (rev 1794)
+++ trunk/varnish-cache/bin/varnishd/mgt_param.c	2007-08-05 19:37:44 UTC (rev 1795)
@@ -754,12 +754,12 @@
 	for (pp = parspec; pp->name != NULL; pp++) {
 		if (!strcmp(pp->name, param)) {
 			pp->func(cli, pp, val);
+			MCF_ParamSync();
 			return;
 		}
 	}
 	cli_result(cli, CLIS_PARAM);
 	cli_out(cli, "Unknown paramter \"%s\".", param);
-	MCF_ParamSync();
 }
 
 




More information about the varnish-commit mailing list