Fix sync instance check (#58)

OsuSync · Nov 30, 2018 · 0af24be · 0af24be
1 parent 8fb8105
commit 0af24be
Showing 1 changed file with 12 additions and 16 deletions.
diff --git a/Sync/Tools/StartupHelper.cs b/Sync/Tools/StartupHelper.cs
@@ -51,7 +51,7 @@ private void SyncInstanceLocker(MemoryMappedFile syncMappedFile, bool forceExit)
                             {
                                 CurrentIO.WriteColor(DefaultI18n.LANG_Instance_Exist, ConsoleColor.Red);
                                 int limit = 50, current = 0;
-                                while (oldSync.WaitForExit(100))
+                                while (!oldSync.WaitForExit(100))
                                 {
                                     current++;
                                     if (current > limit) break;
@@ -73,14 +73,6 @@ private void SyncInstanceLocker(MemoryMappedFile syncMappedFile, bool forceExit)
                 }
             }
         }
-        private void PerLuanchChecker(bool forceStart)
-        {
-            //Check sync.exe is run
-            using (var syncMappedFile = MemoryMappedFile.CreateOrOpen(SYNC_GUID, 4))
-            {
-                SyncInstanceLocker(syncMappedFile, forceStart);
-            }
-        }
 
         static void InitSync()
         {
@@ -122,15 +114,19 @@ internal StartupHelper(string[] args)
 
         internal void Start()
         {
-            PerLuanchChecker(ForceStart);
+            //Check sync.exe is run
+            using (var syncMappedFile = MemoryMappedFile.CreateOrOpen(SYNC_GUID, 4))
+            {
+                SyncInstanceLocker(syncMappedFile, ForceStart);
 
-            InitSync();
-            CurrentIO.WriteWelcome();
+                InitSync();
+                CurrentIO.WriteWelcome();
 
-            while (true)
-            {
-                var cmd = CurrentIO.ReadCommand();
-                SyncHost.Instance.Commands.invokeCmdString(cmd);
+                while (true)
+                {
+                    var cmd = CurrentIO.ReadCommand();
+                    SyncHost.Instance.Commands.invokeCmdString(cmd);
+                }
             }
         }
     }