From a343bc1c764c87b20edbdf960d870fd4b084ce8a Mon Sep 17 00:00:00 2001 From: Brad King Date: Thu, 28 Jul 2011 09:04:45 -0400 Subject: RunSingleCommand: Avoid assignment in condition Several compilers warn about this case even when an extra layer of parenthesis surrounds the assignment. Make the condition explicit. --- Source/cmSystemTools.cxx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Source/cmSystemTools.cxx b/Source/cmSystemTools.cxx index 1971610..03364bd 100644 --- a/Source/cmSystemTools.cxx +++ b/Source/cmSystemTools.cxx @@ -615,7 +615,7 @@ bool cmSystemTools::RunSingleCommand(std::vectorconst& command, int pipe; if ( output || outputflag != OUTPUT_NONE ) { - while((pipe = cmsysProcess_WaitForData(cp, &data, &length, 0))) + while((pipe = cmsysProcess_WaitForData(cp, &data, &length, 0)) > 0) { if(output || outputflag != OUTPUT_NONE) { -- cgit v0.12