From: David Gibson <david@gibson.dropbear.id.au>
Date: Tue, 26 Jan 2016 11:28:41 +0000 (+1100)
Subject: configurator: Clarify empty if
X-Git-Url: http://git.ozlabs.org/?p=ccan;a=commitdiff_plain;h=434fcf81cb19472a05d58a35333d4f9d68ea52b9

configurator: Clarify empty if

configurator.c contains an if with an empty statement on the same line as
the condition.  This is very easy to misread, and also causes a warning
from clang, so move the ; onto the next line.

Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
---

diff --git a/tools/configurator/configurator.c b/tools/configurator/configurator.c
index a590a8d4..021fc0f9 100644
--- a/tools/configurator/configurator.c
+++ b/tools/configurator/configurator.c
@@ -497,7 +497,8 @@ static bool run_test(const char *cmd, struct test *test)
 	fclose(outf);
 
 	if (verbose > 1)
-		if (system("cat " INPUT_FILE) == -1);
+		if (system("cat " INPUT_FILE) == -1)
+			;
 
 	newcmd = strdup(cmd);