Bug: Prevent 'uniform' variables having their names GOLFed.

These aren't allowed to be modified, as they are initialized from the shader's host code.

Author: DeanEdisGG

ShaderShrinker/Shrinker.Parser/SyntaxNodes/SyntaxNodeExtensions.cs

@@ -103,7 +103,8 @@ public static bool HasEntryPointFunction(this SyntaxNode rootNode) =>
         /// </summary>
         public static IEnumerable<string> FindUserDefinedNames(this SyntaxNode root)
         {
-            var names = root.GlobalVariables().Select(o => o.Name).ToList();
+            var nonUniformGlobals = root.GlobalVariables().Where(o => (o.Parent as VariableDeclarationSyntaxNode)?.VariableType.IsUniform != true);
+            var names = nonUniformGlobals.Select(o => o.Name).ToList();
 
             var functionDefinitions = root.FunctionDefinitions().ToList();
             names.AddRange(functionDefinitions.Select(o => o.Name));

ShaderShrinker/UnitTests/GolfTests.cs

@@ -75,7 +75,8 @@ public void CheckGolfingCodeNames(
                        "void f(float n) { } void g() { vec2 vector = vec2(1); f(vector.x); }|void f(float n) { } void g() { vec2 v = vec2(1); f(v.x); }",
                        "void f(vec2 a) { } void g() { vec2 vector = vec2(1); f(vector); }|void f(vec2 a) { } void g() { vec2 v = vec2(1); f(v); }",
                        "struct S { int num; }; void f() { int num; S s; s.num = 1; }|struct S { int num; }; void f() { int n; S s; s.num = 1; }",
-                       "#define NUM 1.0\nfloat f() { return NUM; }|#define N 1.0 float f() { return N; }"
+                       "#define NUM 1.0\nfloat f() { return NUM; }|#define N 1.0 float f() { return N; }",
+                       "uniform float iTime; double main() { return iTime; }|uniform float iTime; double main(){ return iTime; }"
                    )]
             string codeAndGolfed)
         {

ShaderShrinker/UnitTests/TestFiles/GolfedReference/Bonzo.glsl

@@ -1,28 +1,28 @@
-// Processed by 'GLSL Shader Shrinker' (Shrunk by 303 characters)
+// Processed by 'GLSL Shader Shrinker' (Shrunk by 198 characters)
 // (https://github.com/deanthecoder/GLSLShaderShrinker)
 
 #version 410 core
 
 precision mediump float;
-uniform float a;
-uniform vec2 e;
-uniform sampler1D t;
+uniform float fGlobalTime;
+uniform vec2 v2Resolution;
+uniform sampler1D texFFT;
 layout(location = 0)out vec4 o;
-vec4 p(vec2 v, float b) {
-	float c = .5 + sin(v.x * 10.) + cos(sin(b + v.y) * 20.);
-	return vec4(sin(c * .2 + cos(b)), c * .15, cos(c * .1 + b / .4) * .25, 1);
+vec4 p(vec2 v, float t) {
+	float c = .5 + sin(v.x * 10.) + cos(sin(t + v.y) * 20.);
+	return vec4(sin(c * .2 + cos(t)), c * .15, cos(c * .1 + t / .4) * .25, 1);
 }
 
 void main() {
 	vec2 m,
-	     u = vec2(gl_FragCoord.x / e.x, gl_FragCoord.y / e.y);
+	     u = vec2(gl_FragCoord.x / v2Resolution.x, gl_FragCoord.y / v2Resolution.y);
 	u -= .5;
-	u /= vec2(e.y / e.x, 1);
+	u /= vec2(v2Resolution.y / v2Resolution.x, 1);
 	m.x = atan(u.x / u.y) / 3.14;
 	m.y = 1 / length(u) * .2;
 	float d = m.y,
-	      f = texture(t, d).r * 100;
-	m.x += sin(a) * .1;
-	m.y += a * .25;
-	o = f + clamp(p(m * 3.14, a) / d, 0., 1.);
+	      f = texture(texFFT, d).r * 100;
+	m.x += sin(fGlobalTime) * .1;
+	m.y += fGlobalTime * .25;
+	o = f + clamp(p(m * 3.14, fGlobalTime) / d, 0., 1.);
 }