r2689 - trunk/varnish-cache/bin/varnishtest

[phk at projects.linpro.no]

Author: phk
Date: 2008-06-15 21:53:36 +0200 (Sun, 15 Jun 2008)
New Revision: 2689

Modified:
   trunk/varnish-cache/bin/varnishtest/vtc.c
Log:
Flexelint



Modified: trunk/varnish-cache/bin/varnishtest/vtc.c
===================================================================
--- trunk/varnish-cache/bin/varnishtest/vtc.c	2008-06-15 19:52:34 UTC (rev 2688)
+++ trunk/varnish-cache/bin/varnishtest/vtc.c	2008-06-15 19:53:36 UTC (rev 2689)
@@ -56,28 +56,29 @@
 	vsb = vsb_new(NULL, NULL, 0, VSB_AUTOEXTEND);
 	if (str == NULL) 
 		vsb_printf(vsb, "#### %-4s %s(null)\n", ident, pfx);
-	for(; *str; str++) {
-		if (nl) {
-			vsb_printf(vsb, "#### %-4s %s| ", ident, pfx);
-			nl = 0;
+	else
+		for(; *str; str++) {
+			if (nl) {
+				vsb_printf(vsb, "#### %-4s %s| ", ident, pfx);
+				nl = 0;
+			}
+			if (*str == '\r')
+				vsb_printf(vsb, "\\r");
+			else if (*str == '\t')
+				vsb_printf(vsb, "\\t");
+			else if (*str == '\n') {
+				vsb_printf(vsb, "\\n\n");
+				nl = 1;
+			} else if (*str < 0x20 || *str > 0x7e)
+				vsb_printf(vsb, "\\x%02x", *str);
+			else
+				vsb_printf(vsb, "%c", *str);
 		}
-		if (*str == '\r')
-			vsb_printf(vsb, "\\r");
-		else if (*str == '\t')
-			vsb_printf(vsb, "\\t");
-		else if (*str == '\n') {
-			vsb_printf(vsb, "\\n\n");
-			nl = 1;
-		} else if (*str < 0x20 || *str > 0x7e)
-			vsb_printf(vsb, "\\x%02x", *str);
-		else
-			vsb_printf(vsb, "%c", *str);
-	}
 	if (!nl)
 		vsb_printf(vsb, "\n");
 	vsb_finish(vsb);
 	AZ(vsb_overflowed(vsb));
-	fputs(vsb_data(vsb), stdout);
+	(void)fputs(vsb_data(vsb), stdout);
 	vsb_delete(vsb);
 }