Fixing Oculus Deployment Issue, whitespace error (#10584)

* Fixing Oculus Deployment Issue, whitespace error

* fixing whitespace in the other direction

* Oculus documentation fixes

Author: RogPodge

Assets/MRTK/Providers/Oculus/XRSDK/MRTK-Quest/Editor/OculusXRSDKHandtrackingConfigurationChecker.cs

@@ -211,10 +211,10 @@ internal static void ReconcileOculusIntegrationDefine(bool oculusIntegrationPres
         }
 
         /// <summary>
-        /// Adds warnings to the nowarn line in the csc.rsp file located at the root of assets.  Warning 414, 618 and 649 are added to the nowarn line because if
+        /// Adds warnings to the nowarn line in the csc.rsp file located at the root of assets.  Warning 618 is added to the nowarn line because if
         /// the MRTK source is from the repo, warnings are converted to errors. Warnings are not converted to errors if the MRTK source is from the unity packages.
-        /// Warning 414, 618 and 649 are logged when Oculus Integration is imported into the project, 414 is an unsued variable warning,
-        /// 618 is the obsolete warning and 649 is a null on start warning.
+        /// Warning 618 is logged when building the project as of Oculus Integration 39.0 https://developer.oculus.com/downloads/package/unity-integration/39.0
+        /// 618 is the obsolete property/function warning
         /// </summary>
         static void UpdateCSC()
         {
@@ -228,9 +228,9 @@ static void UpdateCSC()
             List<string> warningNumbers = new List<string>();
 
             // List of new warning numbers to add to the csc file
-            // Empty as of Oculus Integration 39.0 https://developer.oculus.com/downloads/package/unity-integration/39.0
             List<string> warningNumbersToAdd = new List<string>()
             {
+                "618"
             };
 
             if (!File.Exists(cscFilePath))

Assets/MRTK/Providers/Oculus/XRSDK/OculusXRSDKDeviceManager.cs

@@ -51,7 +51,6 @@ public override void Initialize()
 #if !UNITY_2020_1_OR_NEWER
             UnityEngine.Debug.Log(@"Detected a potential deployment issue for the Oculus Quest. In order to use hand tracking with the Oculus Quest, download the Oculus Integration Package from the Unity Asset Store and run the Integration tool before deploying.
 The tool can be found under <i>Mixed Reality > Toolkit > Utilities > Oculus > Integrate Oculus Integration Unity Modules</i>");
-            // Add a note about abandoning Oculus XRSDK if on 2020+, use open xr instead please
 #endif
         }
 #endif

Assets/MRTK/SDK/Features/UX/Scripts/Pointers/BaseControllerPointer.cs

@@ -508,7 +508,7 @@ public virtual void Reset()
             IsFocusLocked = false;
         }
 
-#endregion IMixedRealityPointer Implementation
+        #endregion IMixedRealityPointer Implementation
 
         #region IEquality Implementation
 
@@ -556,7 +556,7 @@ public override int GetHashCode()
             }
         }
 
-#endregion IEquality Implementation
+        #endregion IEquality Implementation
 
         #region IMixedRealitySourcePoseHandler Implementation
 
@@ -592,7 +592,7 @@ public override void OnSourceLost(SourceStateEventData eventData)
             }
         }
 
-#endregion IMixedRealitySourcePoseHandler Implementation
+        #endregion IMixedRealitySourcePoseHandler Implementation
 
         #region IMixedRealityInputHandler Implementation
 
@@ -679,6 +679,6 @@ public override void OnInputDown(InputEventData eventData)
             }
         }
 
-#endregion  IMixedRealityInputHandler Implementation
+        #endregion  IMixedRealityInputHandler Implementation
     }
 }