Removed some extra checks that compare set and setting the visibility in SetVisibility to make things easier to read.

killerantz · killerantz · commit 312036e9f206 · 2017-08-07T13:29:47.000-07:00
diff --git a/Assets/HoloToolkit/Input/Scripts/Cursor/InteractiveMeshCursor.cs b/Assets/HoloToolkit/Input/Scripts/Cursor/InteractiveMeshCursor.cs
@@ -152,20 +152,12 @@ public override void SetVisiblity(bool visible)
             base.SetVisiblity(visible);
 
             mIsVisible = visible;
+            ElementVisibility(visible);
 
             if (visible)
             {
-                if (Light != null)
-                {
-                    Light.SetActive(visible);
-                }
-
                 OnCursorStateChange(CursorState);
             }
-            else
-            {
-                ElementVisibility(visible);
-            }
         }
 
         /// <summary>

Original file line number	Diff line number	Diff line change
`@@ -152,20 +152,12 @@ public override void SetVisiblity(bool visible)`
`152`	`152`	`base.SetVisiblity(visible);`
`153`	`153`
`154`	`154`	`mIsVisible = visible;`
	`155`	`+ ElementVisibility(visible);`
`155`	`156`
`156`	`157`	`if (visible)`
`157`	`158`	`{`
`158`		`- if (Light != null)`
`159`		`- {`
`160`		`- Light.SetActive(visible);`
`161`		`- }`
`162`		`-`
`163`	`159`	`OnCursorStateChange(CursorState);`
`164`	`160`	`}`
`165`		`- else`
`166`		`- {`
`167`		`- ElementVisibility(visible);`
`168`		`- }`
`169`	`161`	`}`
`170`	`162`
`171`	`163`	`/// <summary>`