Fixed logging conditional for asm issues.

fpapadopou · fpapadopou · commit 15f5f1b9c27e · 2014-07-11T13:32:37.000+03:00
diff --git a/Symfony/src/Codebender/CompilerBundle/Handler/CompilerHandler.php b/Symfony/src/Codebender/CompilerBundle/Handler/CompilerHandler.php
@@ -1167,7 +1167,7 @@ private function cleanUpClangOutput ($clang_output, $compiler_config, $option) {
                             $this->compiler_logger->addInfo($compiler_config["compiler_dir"] . " - Clang reports core issue.");
                             $coreFound = true;
                         }
-                        if ((strpos($header, "in asm") !== false || strpos($body, "in asm") !== false) && $asmFound === false) {
+                        if ((strpos($header, "in asm") !== false || strpos($body, "in asm") !== false) && $asmFound === false && $option == "asm") {
                             $this->compiler_logger->addInfo($compiler_config["compiler_dir"] . " - Clang reports assembly issue.");
                             $asmFound = true;
                         }
@@ -1219,7 +1219,7 @@ private function cleanUpClangOutput ($clang_output, $compiler_config, $option) {
                         && $coreFound === false && $option != "asm") {
                         $this->compiler_logger->addInfo($compiler_config["compiler_dir"] . " - Clang reports core issue.");
                     }
-                    if ((strpos($header, "in asm") !== false || strpos($body, "in asm") !== false) && $asmFound === false) {
+                    if ((strpos($header, "in asm") !== false || strpos($body, "in asm") !== false) && $asmFound === false && $option == "asm") {
                         $this->compiler_logger->addInfo($compiler_config["compiler_dir"] . " - Clang reports assembly issue.");
                     }
                 }

Original file line number	Diff line number	Diff line change
`@@ -1167,7 +1167,7 @@ private function cleanUpClangOutput ($clang_output, $compiler_config, $option) {`
`1167`	`1167`	`$this->compiler_logger->addInfo($compiler_config["compiler_dir"] . " - Clang reports core issue.");`
`1168`	`1168`	`$coreFound = true;`
`1169`	`1169`	`}`
`1170`		`- if ((strpos($header, "in asm") !== false \|\| strpos($body, "in asm") !== false) && $asmFound === false) {`
	`1170`	`+ if ((strpos($header, "in asm") !== false \|\| strpos($body, "in asm") !== false) && $asmFound === false && $option == "asm") {`
`1171`	`1171`	`$this->compiler_logger->addInfo($compiler_config["compiler_dir"] . " - Clang reports assembly issue.");`
`1172`	`1172`	`$asmFound = true;`
`1173`	`1173`	`}`
`@@ -1219,7 +1219,7 @@ private function cleanUpClangOutput ($clang_output, $compiler_config, $option) {`
`1219`	`1219`	`&& $coreFound === false && $option != "asm") {`
`1220`	`1220`	`$this->compiler_logger->addInfo($compiler_config["compiler_dir"] . " - Clang reports core issue.");`
`1221`	`1221`	`}`
`1222`		`- if ((strpos($header, "in asm") !== false \|\| strpos($body, "in asm") !== false) && $asmFound === false) {`
	`1222`	`+ if ((strpos($header, "in asm") !== false \|\| strpos($body, "in asm") !== false) && $asmFound === false && $option == "asm") {`
`1223`	`1223`	`$this->compiler_logger->addInfo($compiler_config["compiler_dir"] . " - Clang reports assembly issue.");`
`1224`	`1224`	`}`
`1225`	`1225`	`}`