Code review

This commit is contained in:
JustArchi
2016-03-22 11:21:45 +01:00
parent 9bc04640a1
commit df415c8db5
3 changed files with 10 additions and 10 deletions

View File

@@ -148,7 +148,7 @@ namespace ArchiSteamFarm {
bool? isLoggedIn = await IsLoggedIn().ConfigureAwait(false);
if (isLoggedIn.HasValue && !isLoggedIn.Value) {
Logging.LogGenericInfo("Reconnecting because our sessionID expired!", Bot.BotName);
Task.Run(async () => await Bot.RestartIfRunning().ConfigureAwait(false)).Forget();
Bot.RestartIfRunning().Forget();
return true;
}

View File

@@ -252,7 +252,7 @@ namespace ArchiSteamFarm {
}
// Start
Task.Run(async () => await Start().ConfigureAwait(false)).Forget();
Start().Forget();
}
internal bool IsMaster(ulong steamID) {
@@ -726,7 +726,7 @@ namespace ArchiSteamFarm {
return "This bot instance is not connected!";
}
Task.Run(async () => await CardsFarmer.RestartFarming().ConfigureAwait(false)).Forget();
CardsFarmer.RestartFarming().Forget();
return "Done!";
}
@@ -1572,7 +1572,7 @@ namespace ArchiSteamFarm {
}
if (BotConfig.DismissInventoryNotifications) {
Task.Run(async () => await ArchiWebHandler.MarkInventory().ConfigureAwait(false)).Forget();
ArchiWebHandler.MarkInventory().Forget();
}
if (BotConfig.SteamMasterClanID != 0) {
@@ -1591,7 +1591,7 @@ namespace ArchiSteamFarm {
Task.Run(() => Trading.CheckTrades()).Forget();
Task.Run(async () => await CardsFarmer.StartFarming().ConfigureAwait(false)).Forget();
CardsFarmer.StartFarming().Forget();
break;
case EResult.NoConnection:
case EResult.ServiceUnavailable:
@@ -1698,7 +1698,7 @@ namespace ArchiSteamFarm {
if (callback.PurchaseResult == ArchiHandler.PurchaseResponseCallback.EPurchaseResult.OK) {
// We will restart CF module to recalculate current status and decide about new optimal approach
Task.Run(async () => await CardsFarmer.RestartFarming().ConfigureAwait(false)).Forget();
CardsFarmer.RestartFarming().Forget();
}
}
}

View File

@@ -73,7 +73,7 @@ namespace ArchiSteamFarm {
await StopFarming().ConfigureAwait(false);
} else {
Logging.LogGenericInfo("Now running in Automatic Farming mode", Bot.BotName);
Task.Run(async () => await StartFarming().ConfigureAwait(false)).Forget();
StartFarming().Forget();
}
return true;
@@ -151,7 +151,7 @@ namespace ArchiSteamFarm {
// We finished our queue for now, make sure that everything is indeed farmed before proceeding further
// Some games could be added in the meantime
if (await IsAnythingToFarm().ConfigureAwait(false)) {
Task.Run(async () => await StartFarming().ConfigureAwait(false)).Forget();
StartFarming().Forget();
return;
}
@@ -255,7 +255,7 @@ namespace ArchiSteamFarm {
CheckPage(htmlDocument); // Because we fetched page number 1 already
} else {
byte currentPage = page; // We need a copy of variable being passed when in for loops
tasks.Add(Task.Run(async () => await CheckPage(currentPage).ConfigureAwait(false)));
tasks.Add(CheckPage(currentPage));
}
}
await Task.WhenAll(tasks).ConfigureAwait(false);
@@ -269,7 +269,7 @@ namespace ArchiSteamFarm {
tasks = new List<Task>(GamesToFarm.Count);
Logging.LogGenericInfo("Checking hours...", Bot.BotName);
foreach (uint appID in GamesToFarm.Keys) {
tasks.Add(Task.Run(async () => await CheckHours(appID).ConfigureAwait(false)));
tasks.Add(CheckHours(appID));
}
await Task.WhenAll(tasks).ConfigureAwait(false);
}