update comments

Author: bdeweygit

UnityProject/Assets/Plugins/iOS/NativeState.h

@@ -12,8 +12,8 @@ struct NativeState {
 typedef void (*SetNativeStateCallback)(struct NativeState nextState);
 
 @protocol SetsNativeState
-/* This is the critical function pointer that will be used to send state from Swift to Unity.
-   Encapsulation within a protocol will let us take advantage of Swift's didSet property observer. */
+/* Function pointer that will be used to send state from Swift to Unity.
+   Encapsulation within a protocol lets us take advantage of Swift's didSet property observer. */
 @property (nullable) SetNativeStateCallback setNativeState;
 @end
 

UnityProject/Assets/Plugins/iOS/NativeState.m

@@ -2,12 +2,12 @@
 
 id<SetsNativeState> nativeStateSetter;
 
-// Called from Swift. Can we hide this from the C# dll import?
+// Called from Swift. Can we hide this from the C# DllImport()?
 void RegisterNativeStateSetter(id<SetsNativeState> setter) {
     nativeStateSetter = setter;
 }
 
-/* Called from Unity. Interop marshals the argument from a C# delegate to a C function pointer.
+/* Called from Unity. Interop marshals argument from C# delegate to C function pointer.
    See section on marshalling delegates:
    learn.microsoft.com/en-us/dotnet/framework/interop/default-marshalling-behavior */
 void OnSetNativeState(SetNativeStateCallback callback) {

UnityProject/Assets/Scripts/NativeStateManager.cs

@@ -1,14 +1,11 @@
-using System.ComponentModel;
-
 /* Support init only setters. See section on record
    support: docs.unity3d.com/Manual/CSharpCompiler.html */
 namespace System.Runtime.CompilerServices
 {
-    [EditorBrowsable(EditorBrowsableState.Never)]
     internal class IsExternalInit{}
 }
 
-// Should match the NativeState struct in Plugins/iOS/NativeState.h
+// Should match NativeState struct in Plugins/iOS/NativeState.h
 public readonly struct NativeState
 {
     public float scale { get; init; }
@@ -23,15 +20,15 @@ public static class NativeStateManager
 {
     public static NativeState State { get; private set; }
 
-    // Should match the SetNativeStateCallback typedef in Plugins/iOS/NativeState.h
+    // Should match SetNativeStateCallback typedef in Plugins/iOS/NativeState.h
     private delegate void SetNativeStateCallback(NativeState nextState);
 
-    /* Imported from Plugins/iOS/NativeState.m to pass an instance of
+    /* Imported from Plugins/iOS/NativeState.m to pass instance of
        SetNativeStateCallback to C. See section on using delegates: docs.unity3d.com/Manual/PluginsForIOS.html */
     [System.Runtime.InteropServices.DllImport("__Internal")]
     private static extern void OnSetNativeState(SetNativeStateCallback callback);
 
-    /* Reverse P/Invoke wrapped method to set the state value. iOS is an AOT platform hence the decorator.
+    /* Reverse P/Invoke wrapped method to set state value. iOS is an AOT platform hence the decorator.
        See section on calling managed methods from native code: docs.unity3d.com/Manual/ScriptingRestrictions.html */
     [AOT.MonoPInvokeCallback(typeof(SetNativeStateCallback))]
     private static void SetState(NativeState nextState) { State = nextState; }

UnityProject/Assets/Scripts/RootBehavior.cs

@@ -20,8 +20,8 @@ void Update()
         Texture2D nextMainTexture = null;
         if (state.texture != System.IntPtr.Zero) {
             /* In practice it looks like our values for width and height are ignored.
-               It probably determines correct values from the native MTLTexture's own properties.
-               Documentation still insists that we pass the correct width and height values, so we will.*/
+               It probably determines correct values from native MTLTexture's own properties.
+               Documentation still insists that we pass correct width and height values, so we will. */
             nextMainTexture = Texture2D.CreateExternalTexture(state.textureWidth, state.textureHeight, TextureFormat.BGRA32, false, false, state.texture);
         }
 
@@ -37,14 +37,14 @@ It probably determines correct values from the native MTLTexture's own propertie
 
             if (state.visible)
             {
-                // Rotate in the same direciton as the touch delta
+                // Rotate in same direciton as touch delta
                 Vector2 delta = touch.deltaPosition * 0.1f;
                 cube.transform.Rotate(delta.y, -delta.x, 0, Space.World);
             }
 
             if (touchIndicator is null)
             {
-                // Create a touch indicator
+                // Create touch indicator
                 touchIndicator = new GameObject("Touch Indicator");
                 touchIndicator.AddComponent<Image>().sprite = sprite;
                 touchIndicator.transform.SetParent(canvas.transform);