Merge pull request #3718 from keveleigh/cleanup-pass

Assorted formatting and comment updates

Author: keveleigh

Assets/MixedRealityToolkit.Services/InputSystem/FocusProvider.cs

@@ -42,7 +42,7 @@ float IMixedRealityFocusProvider.GlobalPointingExtent
             get
             {
                 MixedRealityInputSystemProfile profile = ConfigurationProfile as MixedRealityInputSystemProfile;
-                
+
                 if ((Service != null) &&
                     (profile != null) &&
                     profile.PointerProfile != null)
@@ -554,6 +554,7 @@ public bool UnregisterPointer(IMixedRealityPointer pointer)
 
                 if (!objectIsStillFocusedByOtherPointer)
                 {
+                    // Policy: only raise focus exit if no other pointers are still focusing the object
                     MixedRealityToolkit.InputSystem.RaiseFocusExit(pointer, unfocusedObject);
                 }
 
@@ -699,7 +700,7 @@ private static void RaycastPhysics(PointerData pointerData, LayerMask[] prioriti
                     case RaycastMode.Simple:
                         if (MixedRealityRaycaster.RaycastSimplePhysicsStep(pointerRays[i], prioritizedLayerMasks, out physicsHit))
                         {
-                            UpdatePointerRayOnHit(pointerData, pointerRays, in physicsHit, i);  
+                            UpdatePointerRayOnHit(pointerData, pointerRays, in physicsHit, i);
                             return;
                         }
                         break;
@@ -772,8 +773,10 @@ private void RaycastGraphics(PointerData pointerData, LayerMask[] prioritizedLay
             }
 
             // Check if we need to overwrite the physics raycast info
-            if ((pointerData.CurrentPointerTarget == null || overridePhysicsRaycast) && raycastResult.isValid &&
-                 raycastResult.module != null && raycastResult.module.eventCamera == UIRaycastCamera)
+            if ((pointerData.CurrentPointerTarget == null || overridePhysicsRaycast) &&
+                raycastResult.isValid &&
+                raycastResult.module != null &&
+                raycastResult.module.eventCamera == UIRaycastCamera)
             {
                 newUiRaycastPosition.x = raycastResult.screenPosition.x;
                 newUiRaycastPosition.y = raycastResult.screenPosition.y;
@@ -935,7 +938,7 @@ private void UpdateFocusedObjects()
             pendingPointerSpecificFocusChange.Clear();
         }
 
-        #endregion Accessors
+        #endregion Utilities
 
         #region ISourceState Implementation
 

Assets/MixedRealityToolkit/Attributes/MixedRealityExtensionServiceAttribute.cs

@@ -74,12 +74,17 @@ public MixedRealityExtensionServiceAttribute(
             PackageFolder = packageFolder;
         }
 
+#if UNITY_EDITOR
         /// <summary>
         /// Convenience function for retrieving the attribute given a certain class type.
         /// </summary>
+        /// <remarks>
+        /// This function is only available in a UnityEditor context.
+        /// </remarks>
         public static MixedRealityExtensionServiceAttribute Find(Type type)
         {
             return type.GetCustomAttributes(typeof(MixedRealityExtensionServiceAttribute), true).FirstOrDefault() as MixedRealityExtensionServiceAttribute;
         }
+#endif // UNITY_EDITOR
     }
 }

Assets/MixedRealityToolkit/Definitions/Devices/MixedRealityControllerMapping.cs

@@ -44,7 +44,7 @@ public string Description
                     }
                 }
 
-                    string handednessText = string.Empty;
+                string handednessText = string.Empty;
                 switch (handedness)
                 {
                     case Handedness.Left: