properly wait if replay file is busy

hotsapi · Mar 17, 2016 · 2dfb671 · 2dfb671
1 parent ce86d79
commit 2dfb671
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 7 deletions.
diff --git a/StatsDisplay/Properties/AssemblyInfo.cs b/StatsDisplay/Properties/AssemblyInfo.cs
@@ -51,5 +51,5 @@
 // You can specify all the values or you can default the Build and Revision Numbers 
 // by using the '*' as shown below:
 // [assembly: AssemblyVersion("1.0.*")]
-[assembly: AssemblyVersion("0.4.1")]
-[assembly: AssemblyFileVersion("0.4.1")]
+[assembly: AssemblyVersion("0.4.2")]
+[assembly: AssemblyFileVersion("0.4.2")]
diff --git a/StatsDisplay/Settings.xaml.cs b/StatsDisplay/Settings.xaml.cs
@@ -107,9 +107,9 @@ private void ProcessLobbyFile(string path)
 				currentWindow.Show();
 		}
 
-		private void ProcessRejoinFile(string path)
+		private async void ProcessRejoinFile(string path)
 		{
-			FileProcessor.ProcessRejoin(path, App.game);
+			await FileProcessor.ProcessRejoin(path, App.game);
 			currentWindow?.Close();
 			currentWindow = new FullStatsWindow();
 		}

diff --git a/StatsFetcher/FileProcessor.cs b/StatsFetcher/FileProcessor.cs
@@ -28,7 +28,7 @@ public static async Task FetchProfiles(Game game)
 			game.TriggerPropertyChanged();
 		}
 
-		public static void ProcessRejoin(string path, Game game)
+		public static async Task ProcessRejoin(string path, Game game)
 		{
 			var tmpPath = Path.GetTempFileName();
 
@@ -39,10 +39,10 @@ public static void ProcessRejoin(string path, Game game)
 					break;
 				}
 				catch (IOException e) {
-					File.AppendAllText("log.txt", $"[{DateTime.Now}] Replay copy error ({i}): {e}\n\n");
 					if (i < 4) {
-						System.Threading.Thread.Sleep(1000); // todo: don't block UI thread
+						await Task.Delay(1000);
 					} else {
+						File.AppendAllText("log.txt", $"[{DateTime.Now}] Replay copy error ({i}): {e}\n\n");
 						throw new ApplicationException("Can't access replay file", e);
 					}
 				}