Missed some .ini initializations and async saving to file

ivanpmartell · Jul 20, 2024 · bff2a8a · bff2a8a
1 parent 41bf155
commit bff2a8a
Show file tree

Hide file tree

Showing 5 changed files with 29 additions and 14 deletions.
diff --git a/LittleWarGameClient/Form1.Designer.cs b/LittleWarGameClient/Form1.Designer.cs
diff --git a/LittleWarGameClient/Form1.cs b/LittleWarGameClient/Form1.cs
@@ -74,7 +74,7 @@ private void webView_WebMessageReceived(object sender, Microsoft.Web.WebView2.Co
                         else
                             mouseLocked = false;
                         settings.SetMouseLock(mouseLocked);
-                        settings.Save();
+                        settings.SaveAsync();
                         CaptureCursor();
                         break;
                     default:
@@ -109,7 +109,7 @@ private void Form1_ResizeEnd(object sender, EventArgs e)
         {
             CaptureCursor();
             settings.SetWindowSize(this.Size);
-            settings.Save();
+            settings.SaveAsync();
         }
 
         private void Form1_Resize(object sender, EventArgs e)
@@ -134,5 +134,10 @@ private void ResizeGameWindows()
                 }
             }
         }
+
+        private void Form1_FormClosing(object sender, FormClosingEventArgs e)
+        {
+            settings.Save();
+        }
     }
 }
diff --git a/LittleWarGameClient/Fullscreen.cs b/LittleWarGameClient/Fullscreen.cs
@@ -33,7 +33,7 @@ internal void Toggle()
                 Enter();
             }
             settings.SetFullScreen(state);
-            settings.Save();
+            settings.SaveAsync();
         }
 
         private void Enter()

diff --git a/LittleWarGameClient/Settings.cs b/LittleWarGameClient/Settings.cs
@@ -43,6 +43,8 @@ private void Init()
             SetUpdateInterval(GetUpdateInterval());
             SetOptionsMenuHotkey(GetOptionsMenuHotkey());
             SetFriendsMenuHotkey(GetFriendsMenuHotkey());
+            SetChatHistoryMenuHotkey(GetChatHistoryMenuHotkey());
+            SetFullscreenHotkey(GetFullscreenHotkey());
             Save();
         }
 
@@ -78,18 +80,25 @@ private void CreateDefaultIniFile()
 
         internal void Save()
         {
-            for (int numTries = 0; numTries < 5; numTries++)
-            {
-                try
-                {
-                    settings.SaveTo(fileName);
-                    break;
-                }
-                catch
+            settings.SaveTo(fileName);
+        }
+
+        internal async void SaveAsync()
+        {
+            await Task.Run(() => {
+                for (int numTries = 0; numTries < 5; numTries++)
                 {
-                    Thread.Sleep(50);
+                    try
+                    {
+                        settings.SaveTo(fileName);
+                        break;
+                    }
+                    catch
+                    {
+                        Thread.Sleep(50);
+                    }
                 }
-            }
+            });
         }
 
         internal void SetMouseLock(bool value)

diff --git a/LittleWarGameClient/VersionHandler.cs b/LittleWarGameClient/VersionHandler.cs
@@ -54,7 +54,7 @@ internal virtual void CheckForUpdate(object? sender, EventArgs e)
                     System.Windows.Forms.Application.Exit();
                 }
             settings.SetLastUpdateChecked(DateTime.Now.Date);
-            settings.Save();
+            settings.SaveAsync();
         }
 
         private bool RequiresUpdate()