Minor example spelling/grammar tweaks & optimizations (#457)

* Removing superfluous Use() calls from SetUniform

Both as good practice and the approach used, these are superfluous and potentially confusing when debugging

* Small Spelling and Grammar adjustments

Author: Toby-Lawrance

.github/workflows/build.yml

@@ -5,13 +5,15 @@ on:
     paths:
       - 'src/**/*'
       - 'build/nuke/*'
+      - 'examples/**/*'
     branches:
       - 'main'
       - 'release/*'
   pull_request:
     paths:
       - 'src/**/*'
       - 'build/nuke/*'
+      - 'examples/**/*'
 
 jobs:
   Build:

examples/CSharp/Tutorial 1.3 - Abstractions/Shader.cs

@@ -52,7 +52,6 @@ public void SetUniform(string name, int value)
             {
                 throw new Exception($"{name} uniform not found on shader.");
             }
-            Use();
             _gl.Uniform1(location, value);
         }
 
@@ -63,7 +62,6 @@ public void SetUniform(string name, float value)
             {
                 throw new Exception($"{name} uniform not found on shader.");
             }
-            Use();
             _gl.Uniform1(location, value);
         }
 
@@ -75,7 +73,7 @@ public void Dispose()
 
         private uint LoadShader(ShaderType type, string path)
         {
-            //To load a single shader we need to load
+            //To load a single shader we need to:
             //1) Load the shader from a file.
             //2) Create the handle.
             //3) Upload the source to opengl.

examples/CSharp/Tutorial 1.3 - Abstractions/shader.frag

@@ -1,7 +1,7 @@
 //Specifying the version like in our vertex shader.
 #version 330 core
 //The input variables, again prefixed with an f as they are the input variables of our fragment shader.
-//These have to share name for now even tho there is a way around this later on.
+//These have to share name for now even though there is a way around this later on.
 in vec4 fColor;
 
 //The output of our fragment shader, this just has to be a vec3 or a vec4, containing the color information about

examples/CSharp/Tutorial 1.3 - Abstractions/shader.vert

@@ -2,20 +2,20 @@
 #version 330 core
 //These lines specify the location and type of our attributes,
 //the attributes here are prefixed with a "v" as they are our inputs to the vertex shader
-//this isnt strictly necessary tho, but a good habbit.
+//this isn't strictly necessary though, but a good habit.
 layout (location = 0) in vec3 vPos;
 layout (location = 1) in vec4 vColor;
 
 //This is how we declare a uniform, they can be used in all our shaders and share the same name.
-//This is prefixed with a u as its our uniform.
+//This is prefixed with a u as it's our uniform.
 uniform float uBlue;
 
-//This is our output variables, notice that this is prefixed with an f as its the input of our fragment shader.
+//This is our output variable, notice that this is prefixed with an f as it's the input of our fragment shader.
 out vec4 fColor;
 
 void main()
 {
-    //gl_Position, is a builting variable on all vertex shaders that will specify the position of our vertex.
+    //gl_Position, is a built-in variable on all vertex shaders that will specify the position of our vertex.
     gl_Position = vec4(vPos, 1.0);
     //The rest of this code looks like plain old c (almost c#)
     vec4 color = vec4(vColor.rb / 2, uBlue, vColor.a); //Swizzling and constructors in glsl.

examples/CSharp/Tutorial 1.4 - Textures/Shader.cs

@@ -42,7 +42,6 @@ public void SetUniform(string name, int value)
             {
                 throw new Exception($"{name} uniform not found on shader.");
             }
-            Use();
             _gl.Uniform1(location, value);
         }
 
@@ -53,7 +52,6 @@ public void SetUniform(string name, float value)
             {
                 throw new Exception($"{name} uniform not found on shader.");
             }
-            Use();
             _gl.Uniform1(location, value);
         }
 

examples/CSharp/Tutorial 1.4 - Textures/Texture.cs

@@ -17,7 +17,7 @@ public unsafe Texture(GL gl, string path)
             //Loading an image using imagesharp.
             Image<Rgba32> img = (Image<Rgba32>) Image.Load(path);
             //We need to flip our image as image sharps coordinates has origin (0, 0) in the top-left corner,
-            //where as openGL has origin in the bottom-left corner.
+            //whereas openGL has origin in the bottom-left corner.
             img.Mutate(x => x.Flip(FlipMode.Vertical));
 
             fixed (void* data = &MemoryMarshal.GetReference(img.GetPixelRowSpan(0)))

examples/CSharp/Tutorial 1.5 - Transformations/Program.cs

@@ -72,7 +72,7 @@ private static void OnLoad()
 
             Texture = new Texture(Gl, "silk.png");
 
-            //Unlike in the transformation, because of our abstraction order dosent matter here.
+            //Unlike in the transformation, because of our abstraction, order doesn't matter here.
             //Translation.
             Transforms[0] = new Transform();
             Transforms[0].Position = new Vector3(0.5f, 0.5f, 0f);

examples/CSharp/Tutorial 1.5 - Transformations/Shader.cs

@@ -43,7 +43,6 @@ public void SetUniform(string name, int value)
             {
                 throw new Exception($"{name} uniform not found on shader.");
             }
-            Use();
             _gl.Uniform1(location, value);
         }
 
@@ -55,7 +54,6 @@ public unsafe void SetUniform(string name, Matrix4x4 value)
             {
                 throw new Exception($"{name} uniform not found on shader.");
             }
-            Use();
             _gl.UniformMatrix4(location, 1, false, (float*) &value);
         }
 
@@ -66,7 +64,6 @@ public void SetUniform(string name, float value)
             {
                 throw new Exception($"{name} uniform not found on shader.");
             }
-            Use();
             _gl.Uniform1(location, value);
         }
 

examples/CSharp/Tutorial 1.5 - Transformations/Transform.cs

@@ -6,7 +6,7 @@ public class Transform
     {
         //A transform abstraction.
         //For a transform we need to have a position a scale and a rotation,
-        //depending on what application you are creating the type for these may vary.
+        //depending on what application you are creating, the type for these may vary.
 
         //Here we have chosen a vec3 for position, float for scale and quaternion for rotation,
         //as that is the most normal to go with.

examples/CSharp/Tutorial 2.1 - Co-ordinate Systems/Program.cs

@@ -21,7 +21,7 @@ class Program
         private static Texture Texture;
         private static Shader Shader;
 
-        //Setup the cameras location, and relative up and right directions
+        //Setup the camera's location, and relative up and right directions
         private static Vector3 CameraPosition = new Vector3(0.0f, 0.0f, 3.0f);
         private static Vector3 CameraTarget = Vector3.Zero;
         private static Vector3 CameraDirection = Vector3.Normalize(CameraPosition - CameraTarget);