From: Alexis Laferrière <alexis.laf@xymus.net>
Date: Sun, 28 Aug 2016 15:38:29 +0000 (-0400)
Subject: frontend: always raise advice about missing glslangValidator
X-Git-Url: http://nitlanguage.org

frontend: always raise advice about missing glslangValidator

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>
---

diff --git a/src/frontend/glsl_validation.nit b/src/frontend/glsl_validation.nit
index 1d11589..f6f3760 100644
--- a/src/frontend/glsl_validation.nit
+++ b/src/frontend/glsl_validation.nit
@@ -58,9 +58,7 @@ private class GLSLValidationPhase
 
 		# Do not double check if tool is in path
 		var in_path = tool_is_in_path
-		if in_path != null then
-			if not in_path then return
-		else
+		if in_path == null then
 			# Is _glslangValidator_ installed?
 			var proc_which = new ProcessReader("which", "glslangValidator")
 			proc_which.wait
@@ -68,11 +66,12 @@ private class GLSLValidationPhase
 			var status = proc_which.status
 			in_path = status == 0
 			tool_is_in_path = in_path
-			if not in_path then
-				toolcontext.advice(nat.location, "glslvalidator",
-					"Warning: program `glslangValidator` not in PATH, cannot validate this shader.")
-				return
-			end
+		end
+
+		if not in_path then
+			toolcontext.advice(nat.location, "glslvalidator",
+				"Warning: program `glslangValidator` not in PATH, cannot validate this shader.")
+			return
 		end
 
 		# Get the shader source