addressing more code review feedback

Author: darax

Assets/HoloToolkit/SpatialMapping/Scripts/TapToPlace.cs

@@ -24,20 +24,20 @@ public partial class TapToPlace : MonoBehaviour
         /// <summary>
         /// Manages persisted anchors.
         /// </summary>
-        WorldAnchorManager anchorManager;
+        private WorldAnchorManager anchorManager;
 
         /// <summary>
         /// Controls spatial mapping.  In this script we access spatialMappingManager
         /// to control rendering and to access the physics layer mask.
         /// </summary>
-        SpatialMappingManager spatialMappingManager;
+        private SpatialMappingManager spatialMappingManager;
 
         /// <summary>
         /// Keeps track of if the user is moving the object or not.
         /// </summary>
-        bool placing = false;
+        private bool placing;
 
-        void Start()
+        private void Start()
         {
             // Make sure we have all the components in the scene we need.
             anchorManager = WorldAnchorManager.Instance;
@@ -64,7 +64,7 @@ void Start()
         }
 
         // Called by GazeGestureManager when the user performs a tap gesture.
-        void OnSelect()
+        public void OnSelect()
         {
             // On each tap gesture, toggle whether the user is in placing mode.
             placing = !placing;
@@ -87,7 +87,7 @@ void OnSelect()
             }
         }
 
-        void Update()
+        private void Update()
         {
             // If the user is in placing mode,
             // update the placement to match the user's gaze.

Assets/HoloToolkit/Utilities/Scripts/WorldAnchorManager.cs

@@ -42,15 +42,15 @@ private struct AnchorAttachmentInfo
         /// Callback function that contains the WorldAnchorStore object.
         /// </summary>
         /// <param name="Store">The WorldAnchorStore to cache.</param>
-        void AnchorStoreReady(WorldAnchorStore Store)
+        private void AnchorStoreReady(WorldAnchorStore Store)
         {
             AnchorStore = Store;
         }
 
         /// <summary>
         /// When the app starts grab the anchor store immediately.
         /// </summary>
-        void Awake()
+        private void Awake()
         {
             AnchorStore = null;
             WorldAnchorStore.GetAsync(AnchorStoreReady);
@@ -59,7 +59,7 @@ void Awake()
         /// <summary>
         /// Each frame see if there is work to do and if we can do a unit, do it.
         /// </summary>
-        void Update()
+        private void Update()
         {
             if (AnchorStore != null && anchorOperations.Count > 0)
             {
@@ -123,7 +123,7 @@ public void RemoveAnchor(GameObject gameObjectToUnanchor)
         /// Function that actually adds the anchor to the game object.
         /// </summary>
         /// <param name="anchorAttachmentInfo">Parameters for attaching the anchor.</param>
-        void DoAnchorOperation(AnchorAttachmentInfo anchorAttachmentInfo)
+        private void DoAnchorOperation(AnchorAttachmentInfo anchorAttachmentInfo)
         {
             string AnchorName = anchorAttachmentInfo.AnchorName;
             GameObject gameObjectToAnchor = anchorAttachmentInfo.GameObjectToAnchor;
@@ -156,7 +156,7 @@ void DoAnchorOperation(AnchorAttachmentInfo anchorAttachmentInfo)
         /// </summary>
         /// <param name="gameObjectToAnchor">The GameObject to attach the anchor to.</param>
         /// <param name="anchorName">The name to give to the anchor.</param>
-        void CreateAnchor(GameObject gameObjectToAnchor, string anchorName)
+        private void CreateAnchor(GameObject gameObjectToAnchor, string anchorName)
         {
             WorldAnchor anchor = gameObjectToAnchor.AddComponent<WorldAnchor>();
             anchor.name = anchorName;