r2213 - trunk/varnish-cache/bin/varnishd

[des at projects.linpro.no]

Author: des
Date: 2007-10-30 14:42:52 +0100 (Tue, 30 Oct 2007)
New Revision: 2213

Modified:
   trunk/varnish-cache/bin/varnishd/cache_ws.c
Log:
Simplify WS_DEBUG(), and make it conditional on DIAGNOSTICS.


Modified: trunk/varnish-cache/bin/varnishd/cache_ws.c
===================================================================
--- trunk/varnish-cache/bin/varnishd/cache_ws.c	2007-10-30 13:41:56 UTC (rev 2212)
+++ trunk/varnish-cache/bin/varnishd/cache_ws.c	2007-10-30 13:42:52 UTC (rev 2213)
@@ -46,10 +46,10 @@
 #include "cache.h"
 
 /* Enable this to get detailed logging of WS usage */
-#if 0
-#  define WS_DEBUG(foo)	VSL foo
+#ifdef DIAGNOSTICS
+#  define WS_DEBUG(fmt, ...)	VSL(SLT_Debug, 0, fmt, __VA_ARGS__)
 #else
-#  define WS_DEBUG(foo)	/* nothing */
+#  define WS_DEBUG(fmt, ...)	/* nothing */
 #endif
 
 void
@@ -57,10 +57,10 @@
 {
 
 	CHECK_OBJ_NOTNULL(ws, WS_MAGIC);
-	WS_DEBUG((SLT_Debug, 0, "WS(%p = (%s, %p %u %u %u)",
+	WS_DEBUG("WS(%p = (%s, %p %u %u %u)",
 	    ws, ws->id, ws->s, pdiff(ws->s, ws->f),
 	    ws->r == NULL ? 0 : pdiff(ws->f, ws->r),
-	    pdiff(ws->s, ws->e)));
+	    pdiff(ws->s, ws->e));
 	assert(ws->s != NULL);
 	assert(ws->e != NULL);
 	assert(ws->s < ws->e);
@@ -76,7 +76,7 @@
 WS_Init(struct ws *ws, const char *id, void *space, unsigned len)
 {
 
-	WS_DEBUG((SLT_Debug, 0, "WS_Init(%p, \"%s\", %p, %u)", ws, id, space, len));
+	WS_DEBUG("WS_Init(%p, \"%s\", %p, %u)", ws, id, space, len);
 	assert(space != NULL);
 	memset(ws, 0, sizeof *ws);
 	ws->magic = WS_MAGIC;
@@ -92,7 +92,7 @@
 {
 
 	WS_Assert(ws);
-	WS_DEBUG((SLT_Debug, 0, "WS_Reset(%p, %p)", ws, p));
+	WS_DEBUG("WS_Reset(%p, %p)", ws, p);
 	assert(ws->r == NULL);
 	if (p == NULL)
 		ws->f = ws->s;
@@ -114,7 +114,7 @@
 		return(NULL);
 	r = ws->f;
 	ws->f += bytes;
-	WS_DEBUG((SLT_Debug, 0, "WS_Alloc(%p, %u) = %p", ws, bytes, r));
+	WS_DEBUG("WS_Alloc(%p, %u) = %p", ws, bytes, r);
 	return (r);
 }
 
@@ -128,7 +128,7 @@
 	p = WS_Alloc(ws, l);
 	if (p != NULL)
 		memcpy(p, s, l);
-	WS_DEBUG((SLT_Debug, 0, "WS_Dup(%p, \"%s\") = %p", ws, s, p));
+	WS_DEBUG("WS_Dup(%p, \"%s\") = %p", ws, s, p);
 	return (p);
 }
 
@@ -138,7 +138,7 @@
 
 	WS_Assert(ws);
 	assert(ws->r == NULL);
-	WS_DEBUG((SLT_Debug, 0, "WS_Snapshot(%p) = %p", ws, ws->f));
+	WS_DEBUG("WS_Snapshot(%p) = %p", ws, ws->f);
 	return (ws->f);
 }
 
@@ -153,8 +153,8 @@
 		b2 = ws->e - ws->f;
 	xxxassert(ws->f + b2 <= ws->e);
 	ws->r = ws->f + b2;
-	WS_DEBUG((SLT_Debug, 0, "WS_Reserve(%p, %u/%u) = %u",
-	    ws, b2, bytes, pdiff(ws->f, ws->r)));
+	WS_DEBUG("WS_Reserve(%p, %u/%u) = %u",
+	    ws, b2, bytes, pdiff(ws->f, ws->r));
 	return (pdiff(ws->f, ws->r));
 }
 
@@ -162,7 +162,7 @@
 WS_Release(struct ws *ws, unsigned bytes)
 {
 	WS_Assert(ws);
-	WS_DEBUG((SLT_Debug, 0, "WS_Release(%p, %u)", ws, bytes));
+	WS_DEBUG("WS_Release(%p, %u)", ws, bytes);
 	assert(ws->r != NULL);
 	assert(ws->f + bytes <= ws->r);
 	ws->f += bytes;
@@ -173,7 +173,7 @@
 WS_ReleaseP(struct ws *ws, char *ptr)
 {
 	WS_Assert(ws);
-	WS_DEBUG((SLT_Debug, 0, "WS_ReleaseP(%p, %p)", ws, ptr));
+	WS_DEBUG("WS_ReleaseP(%p, %p)", ws, ptr);
 	assert(ws->r != NULL);
 	assert(ptr >= ws->f);
 	assert(ptr <= ws->r);