Final polish: proper null comparison and timeout documentation

Copilot · BeSmarterWithData · Copilot · commit 220edc445f5d · 2025-12-16T18:49:44.000Z
Co-authored-by: chris1642 &lt;24726206+chris1642@users.noreply.github.com&gt;
diff --git a/Final PS Script.txt b/Final PS Script.txt
@@ -180,7 +180,7 @@ function Show-EnvironmentPicker {
 
     # Timer (auto-close after 120 seconds)
     $timer = New-Object System.Windows.Forms.Timer
-    $timer.Interval = 120000
+    $timer.Interval = 120000  # 120 seconds in milliseconds
     $timer.Add_Tick({
         $timer.Stop()
         $form.Tag = 'Timeout'
@@ -214,8 +214,8 @@ function Show-EnvironmentPicker {
     }
     finally {
         # Ensure resources are always disposed
-        if ($timer) { $timer.Dispose() }
-        if ($form) { $form.Dispose() }
+        if ($null -ne $timer) { $timer.Dispose() }
+        if ($null -ne $form) { $form.Dispose() }
     }
 }
 

Original file line number	Diff line number	Diff line change
`@@ -180,7 +180,7 @@ function Show-EnvironmentPicker {`
`180`	`180`
`181`	`181`	`# Timer (auto-close after 120 seconds)`
`182`	`182`	`$timer = New-Object System.Windows.Forms.Timer`
`183`		`- $timer.Interval = 120000`
	`183`	`+ $timer.Interval = 120000 # 120 seconds in milliseconds`
`184`	`184`	`$timer.Add_Tick({`
`185`	`185`	`$timer.Stop()`
`186`	`186`	`$form.Tag = 'Timeout'`
`@@ -214,8 +214,8 @@ function Show-EnvironmentPicker {`
`214`	`214`	`}`
`215`	`215`	`finally {`
`216`	`216`	`# Ensure resources are always disposed`
`217`		`- if ($timer) { $timer.Dispose() }`
`218`		`- if ($form) { $form.Dispose() }`
	`217`	`+ if ($null -ne $timer) { $timer.Dispose() }`
	`218`	`+ if ($null -ne $form) { $form.Dispose() }`
`219`	`219`	`}`
`220`	`220`	`}`
`221`	`221`