Fix Lock misuses

Note to myself: Please read Microsoft documentation more comprehensively before using a feature :derp:

Author: neon-nyan

CollapseLauncher/Classes/Helper/Background/Loaders/MediaPlayerLoader.cs

@@ -235,7 +235,7 @@ Playing background video with FFmpeg!
 
         private void UpdateCanvasOnSizeChangeEvent(object sender, SizeChangedEventArgs e)
         {
-            lock (_currentLock)
+            using (_currentLock.EnterScope())
             {
                 float scalingFactor = (float)WindowUtility.CurrentWindowMonitorScaleFactor;
                 float newWidth      = (float)(e.NewSize.Width * scalingFactor);
@@ -301,7 +301,7 @@ public void DisposeMediaModules()
 
             if (IsUseVideoBgDynamicColorUpdate)
             {
-                lock (_currentLock)
+                using (_currentLock.EnterScope())
                 {
                     _currentCanvasDrawingSession?.Dispose();
                     Interlocked.Exchange(ref _currentCanvasDrawingSession, null);
@@ -373,7 +373,7 @@ private static async Task<StorageFile> GetFileAsStorageFile(string filePath)
         private async void FrameGrabberEvent(MediaPlayer mediaPlayer, object args)
     #pragma warning restore CS1998 // Async method lacks 'await' operators and will run synchronously
         {
-            lock (_currentLock)
+            using (_currentLock.EnterScope())
             {
                 if (_currentCanvasVirtualImageSource is null)
                 {
@@ -391,7 +391,7 @@ private async void FrameGrabberEvent(MediaPlayer mediaPlayer, object args)
 
             try
             {
-                lock (_currentLock)
+                using (_currentLock.EnterScope())
                 {
                     mediaPlayer.CopyFrameToVideoSurface(_currentCanvasBitmap);
                     _currentCanvasDrawingSession = _currentCanvasVirtualImageSource
@@ -413,7 +413,7 @@ private async void FrameGrabberEvent(MediaPlayer mediaPlayer, object args)
 #endif
             finally
             {
-                lock (_currentLock)
+                using (_currentLock.EnterScope())
                 {
                     CurrentDispatcherQueue.TryEnqueue(() =>
                     {

CollapseLauncher/Classes/Helper/Hash.StringAndBytes.cs

@@ -117,7 +117,7 @@ public static bool TryGetHashFromBytes<T>(
             where T : NonCryptographicHashAlgorithm
         {
             // Lock the thread and append the span to the hash algorithm
-            lock (hashSource.Lock)
+            using (hashSource.Lock.EnterScope())
             {
                 // Append and calculate the hash of the span
                 hashSource.Hash.Append(source);
@@ -235,7 +235,7 @@ public static bool TryGetCryptoHashFromBytes<T>(
             where T : HashAlgorithm
         {
             // Lock the thread and compute the hash of the span
-            lock (hashSource.Lock)
+            using (hashSource.Lock.EnterScope())
             {
                 // Reset the hash instance state.
                 hashSource.Hash.Initialize();

CollapseLauncher/Classes/Helper/HttpClientBuilder.cs

@@ -112,7 +112,7 @@ public HttpClientBuilder<THandler> UseLauncherConfig(int maxConnections = MaxCon
                 UseExternalProxy(lProxyUri, lHttpProxyUsername, proxyPassword);
             }
 
-            lock (HttpClientBuilderSharedLock)
+            using (HttpClientBuilderSharedLock.EnterScope())
             {
                 if (!skipDnsInit && lIsUseExternalDns && ExternalDnsServers == null)
                 {

CollapseLauncher/Classes/Helper/Metadata/DataCooker.cs

@@ -108,7 +108,7 @@ internal static void ServeV3Data(ReadOnlySpan<byte> data,             Span<byte>
                         : new byte[dataRawBuffer.Length];
                     isDecryptPoolUsed = isDecryptPoolAllowed;
 
-                    lock (RsaDecryptLock)
+                    using (RsaDecryptLock.EnterScope())
                     {
                         if (RsaInstance == null)
                         {

CollapseLauncher/Classes/InstallManagement/Base/InstallManagerBase.SophonPatch.cs

@@ -318,7 +318,7 @@ async ValueTask ImplDownload(Tuple<SophonPatchAsset, Dictionary<string, int>> ct
                 SophonPatchAsset        patchAsset     = ctx.Item1;
                 Dictionary<string, int> downloadedDict = ctx.Item2;
 
-                lock (dictionaryLock)
+                using (dictionaryLock.EnterScope())
                 {
                     _ = downloadedDict.TryAdd(patchAsset.PatchNameSource, 0);
                     downloadedDict[patchAsset.PatchNameSource]++;
@@ -347,7 +347,7 @@ async ValueTask ImplPatchUpdate(Tuple<SophonPatchAsset, Dictionary<string, int>>
                 SophonPatchAsset patchAsset = ctx.Item1;
                 Dictionary<string, int> downloadedDict = ctx.Item2;
 
-                lock (dictionaryLock)
+                using (dictionaryLock.EnterScope())
                 {
                     if (!string.IsNullOrEmpty(patchAsset.PatchNameSource) &&
                         downloadedDict.Remove(patchAsset.PatchNameSource))

CollapseLauncher/Classes/RegistryMonitor/RegistryMonitor.cs

@@ -164,7 +164,7 @@ public RegChangeNotifyFilter RegChangeNotifyType
             get { return _regFilter; }
             set
             {
-                lock (_threadLock)
+                using (_threadLock.EnterScope())
                 {
                     if (IsMonitoring)
                         throw new InvalidOperationException("Monitoring thread is already running");
@@ -247,7 +247,7 @@ public void Start()
             if (_disposed)
                 throw new ObjectDisposedException(null, "This instance is already disposed");
 
-            lock (_threadLock)
+            using (_threadLock.EnterScope())
             {
                 if (IsMonitoring)
                 {
@@ -270,7 +270,7 @@ public void Stop()
         {
             if (_disposed) return;
 
-            lock (_threadLock)
+            using (_threadLock.EnterScope())
             {
                 Thread thread = _thread;
                 if (thread == null)

CollapseLauncher/XAMLs/MainApp/Pages/FileCleanupPage.xaml.cs

@@ -386,7 +386,7 @@ await Task.Factory.StartNew(state =>
                                                                                          catch (Exception ex)
                                                                                          {
                                                                                              Interlocked.Increment(ref deleteFailed);
-                                                                                             lock (failedListLock)
+                                                                                             using (failedListLock.EnterScope())
                                                                                              {
                                                                                                  failedList.Add(fileInfo);
                                                                                              }

Hi3Helper.Core/Classes/Logger/LoggerBase.cs

@@ -51,7 +51,7 @@ public void SetFolderPathAndInitialize(string folderPath, Encoding logEncoding)
             }
 #endif
 
-            lock (_lockObject)
+            using (_lockObject.EnterScope())
             {
                 // Try dispose the _logWriter even though it's not initialized.
                 // This will be used if the program need to change the log folder to another location.
@@ -76,7 +76,7 @@ public void SetFolderPathAndInitialize(string folderPath, Encoding logEncoding)
 #nullable enable
         public void ResetLogFiles(string? reloadToPath, Encoding? encoding = null)
         {
-            lock (_lockObject)
+            using (_lockObject.EnterScope())
             {
                 DisposeBase();