Misc Rider enhancements

ArchiSteamFarm.CustomPlugins.PeriodicGC/PeriodicGCPlugin.cs

@@ -39,7 +39,7 @@ namespace ArchiSteamFarm.CustomPlugins.PeriodicGC;
 internal sealed class PeriodicGCPlugin : IPlugin {
 	private const byte GCPeriod = 60; // In seconds
 
-	private static readonly object LockObject = new();
+	private static readonly Lock Lock = new();
 	private static readonly Timer PeriodicGCTimer = new(PerformGC);
 
 	[JsonInclude]
@@ -55,7 +55,7 @@ public Task OnLoaded() {
 
 		ASF.ArchiLogger.LogGenericWarning($"Periodic GC will occur every {timeSpan.ToHumanReadable()}. Please keep in mind that this plugin should be used for debugging tests only.");
 
-		lock (LockObject) {
+		lock (Lock) {
 			PeriodicGCTimer.Change(timeSpan, timeSpan);
 		}
 
@@ -65,7 +65,7 @@ public Task OnLoaded() {
 	private static void PerformGC(object? state = null) {
 		ASF.ArchiLogger.LogGenericWarning($"Performing GC, current memory: {GC.GetTotalMemory(false) / 1024} KB.");
 
-		lock (LockObject) {
+		lock (Lock) {
 			GCSettings.LargeObjectHeapCompactionMode = GCLargeObjectHeapCompactionMode.CompactOnce;
 			GC.Collect(GC.MaxGeneration, GCCollectionMode.Forced, true, true);
 		}

ArchiSteamFarm.OfficialPlugins.Monitoring/TradeStatistics.cs

@@ -22,12 +22,13 @@
 // limitations under the License.
 
 using System;
+using System.Threading;
 using ArchiSteamFarm.Steam.Exchange;
 
 namespace ArchiSteamFarm.OfficialPlugins.Monitoring;
 
 internal sealed class TradeStatistics {
-	private readonly object Lock = new();
+	private readonly Lock Lock = new();
 
 	internal int AcceptedOffers { get; private set; }
 	internal int BlacklistedOffers { get; private set; }

ArchiSteamFarm/Core/AprilFools.cs

@@ -28,7 +28,7 @@
 namespace ArchiSteamFarm.Core;
 
 internal static class AprilFools {
-	private static readonly object LockObject = new();
+	private static readonly Lock Lock = new();
 
 	// We don't care about CurrentCulture global config property, because April Fools are never initialized in this case
 	private static readonly CultureInfo OriginalCulture = CultureInfo.CurrentCulture;
@@ -49,7 +49,7 @@ internal static void Init(object? state = null) {
 
 			TimeSpan aprilFoolsEnd = TimeSpan.FromDays(1) - now.TimeOfDay;
 
-			lock (LockObject) {
+			lock (Lock) {
 				Timer.Change(aprilFoolsEnd + TimeSpan.FromMilliseconds(100), Timeout.InfiniteTimeSpan);
 			}
 
@@ -72,7 +72,7 @@ internal static void Init(object? state = null) {
 		// Timer can accept only dueTimes up to 2^32 - 2
 		uint dueTime = (uint) Math.Min(uint.MaxValue - 1, (ulong) aprilFoolsStart.TotalMilliseconds + 100);
 
-		lock (LockObject) {
+		lock (Lock) {
 			Timer.Change(dueTime, Timeout.Infinite);
 		}
 	}

ArchiSteamFarm/IPC/Integration/SwaggerItemsMinMaxAttribute.cs

@@ -30,12 +30,12 @@ namespace ArchiSteamFarm.IPC.Integration;
 [PublicAPI]
 public sealed class SwaggerItemsMinMaxAttribute : CustomSwaggerAttribute {
 	public uint MaximumUint {
-		get => BackingMaximum.HasValue ? decimal.ToUInt32(BackingMaximum.Value) : default(uint);
+		get => BackingMaximum.HasValue ? decimal.ToUInt32(BackingMaximum.Value) : 0;
 		set => BackingMaximum = value;
 	}
 
 	public uint MinimumUint {
-		get => BackingMinimum.HasValue ? decimal.ToUInt32(BackingMinimum.Value) : default(uint);
+		get => BackingMinimum.HasValue ? decimal.ToUInt32(BackingMinimum.Value) : 0;
 		set => BackingMinimum = value;
 	}
 

ArchiSteamFarm/Steam/Integration/ArchiWebHandler.cs

@@ -749,7 +749,7 @@ public async Task<bool> JoinGroup(ulong groupID) {
 				Bot.ArchiLogger.LogGenericWarning(Strings.WarningFailed);
 				Bot.ArchiLogger.LogGenericDebug(Strings.FormatErrorFailingRequest(request));
 
-				return default(ObjectResponse<T>?);
+				return null;
 			}
 		}
 
@@ -759,7 +759,7 @@ public async Task<bool> JoinGroup(ulong groupID) {
 		ObjectResponse<T>? response = await WebLimitRequest(host, async () => await WebBrowser.UrlGetToJsonObject<T>(request, headers, referer, requestOptions, maxTries, rateLimitingDelay, cancellationToken).ConfigureAwait(false), cancellationToken).ConfigureAwait(false);
 
 		if (response == null) {
-			return default(ObjectResponse<T>?);
+			return null;
 		}
 
 		if (IsSessionExpiredUri(response.FinalUri)) {