Merge pull request gui-cs#3339 from tig/v2_3338-Toplevel-Must-Be-Disp…

…osed Fixes gui-cs#3338. `Application.Run/End` -> Callers must dispose Toplevel
tznind · Mar 26, 2024 · 6070e80 · 6070e80
2 parents d5f3129 + fda20df
commit 6070e80
Show file tree

Hide file tree

Showing 136 changed files with 3,191 additions and 2,182 deletions.
diff --git a/Example/Example.cs b/Example/Example.cs
@@ -6,9 +6,11 @@
 using System;
 using Terminal.Gui;
 
-Application.Run<ExampleWindow> ();
+var app = Application.Run<ExampleWindow> ();
 
-Console.WriteLine ($"Username: {((ExampleWindow)Application.Top).UserNameText.Text}");
+Console.WriteLine ($"Username: {app.UserNameText.Text}");
+
+app.Dispose ();
 
 // Before the application exits, reset Terminal.Gui for clean shutdown
 Application.Shutdown ();

diff --git a/Terminal.Gui/Application.MainLoopSyncContext.cs b/Terminal.Gui/Application.MainLoopSyncContext.cs
@@ -12,7 +12,7 @@ private sealed class MainLoopSyncContext : SynchronizationContext
 
         public override void Post (SendOrPostCallback d, object state)
         {
-            MainLoop.AddIdle (
+            MainLoop?.AddIdle (
                               () =>
                               {
                                   d (state);