fix warnings: using the result of an assignment as a condition without parentheses...
authorGiel van Schijndel <me@mortis.eu>
Fri, 24 Jun 2011 18:03:16 +0000 (20:03 +0200)
committerGiel van Schijndel <me@mortis.eu>
Wed, 13 Jul 2011 03:07:44 +0000 (05:07 +0200)
Don't unnecessarily assign to variables within the *boolean* expression
of a conditional.

Signed-off-by: Giel van Schijndel <me@mortis.eu>

src/util.cpp

index 479c601..7693aaa 100644 (file)
@@ -760,8 +760,8 @@ string GetPidFile()
 
 void CreatePidFile(string pidFile, pid_t pid)
 {
-    FILE* file;
-    if (file = fopen(pidFile.c_str(), "w"))
+    FILE* file = fopen(pidFile.c_str(), "w");
+    if (file)
     {
         fprintf(file, "%d\n", pid);
         fclose(file);
@@ -790,7 +790,9 @@ void ShrinkDebugFile()
         fseek(file, -sizeof(pch), SEEK_END);
         int nBytes = fread(pch, 1, sizeof(pch), file);
         fclose(file);
-        if (file = fopen(strFile.c_str(), "w"))
+
+        file = fopen(strFile.c_str(), "w");
+        if (file)
         {
             fwrite(pch, 1, nBytes, file);
             fclose(file);