Fix off-by-one error on bounds check in JSISwitchableVariableLabel

JonnyOThan · JonnyOThan · commit bae3dd36f110 · 2023-05-10T09:25:55.000-04:00
diff --git a/RasterPropMonitor/Auxiliary modules/JSISwitchableVariableLabel.cs b/RasterPropMonitor/Auxiliary modules/JSISwitchableVariableLabel.cs
@@ -172,7 +172,7 @@ private bool UpdateCheck()
             // Saw an out-of-range exception in the next if clause once as a
             // side effect of docking.  Not sure if OnUpdate was called after
             // onDestroy, or before Start.
-            if (activeLabel > labelsEx.Count)
+            if (activeLabel >= labelsEx.Count)
             {
                 activeLabel = labelsEx.Count - 1;
                 if(activeLabel < 0)

Original file line number	Diff line number	Diff line change
`@@ -172,7 +172,7 @@ private bool UpdateCheck()`
`172`	`172`	`// Saw an out-of-range exception in the next if clause once as a`
`173`	`173`	`// side effect of docking. Not sure if OnUpdate was called after`
`174`	`174`	`// onDestroy, or before Start.`
`175`		`- if (activeLabel > labelsEx.Count)`
	`175`	`+ if (activeLabel >= labelsEx.Count)`
`176`	`176`	`{`
`177`	`177`	`activeLabel = labelsEx.Count - 1;`
`178`	`178`	`if(activeLabel < 0)`