[Core.IO] Modernize code

Author: Kryptos-FR

sources/core/Stride.Core.IO/DirectoryWatcher.Desktop.cs

@@ -1,45 +1,49 @@
 // Copyright (c) .NET Foundation and Contributors (https://dotnetfoundation.org/ & https://stride3d.net) and Silicon Studio Corp. (https://www.siliconstudio.co.jp)
 // Distributed under the MIT license. See the LICENSE.md file in the project root for more information.
-using System;
-using System.Collections.Generic;
-using System.Threading;
+namespace Stride.Core.IO;
 
-namespace Stride.Core.IO
+/// <summary>
+/// Track file system events from several directories.
+/// </summary>
+public partial class DirectoryWatcher : IDisposable
 {
+    private const int SleepBetweenWatcherCheck = 200;
+    private Thread? watcherCheckThread;
+    private bool exitThread;
+
+    /// <summary>
+    /// Occurs when a file/directory change occurred.
+    /// </summary>
+    public event EventHandler<FileEvent>? Modified;
+
     /// <summary>
-    /// Track file system events from several directories.
+    /// Initializes a new instance of the <see cref="DirectoryWatcher"/> class.
     /// </summary>
-    public partial class DirectoryWatcher : IDisposable
+    /// <param name="fileFilter">The file filter By default null default to *.*</param>
+    public DirectoryWatcher(string? fileFilter = null)
     {
-        private const int SleepBetweenWatcherCheck = 200;
-        private Thread watcherCheckThread;
-        private bool exitThread;
-
-        /// <summary>
-        /// Occurs when a file/directory change occurred.
-        /// </summary>
-        public event EventHandler<FileEvent> Modified;
-
-        /// <summary>
-        /// Initializes a new instance of the <see cref="DirectoryWatcher"/> class.
-        /// </summary>
-        /// <param name="fileFilter">The file filter By default null default to *.*</param>
-        public DirectoryWatcher(string fileFilter = null)
-        {
-            FileFilter = fileFilter ?? "*.*";
-            InitializeInternal();
-        }
+        FileFilter = fileFilter ?? "*.*";
+        InitializeInternal();
+    }
+
+    /// <summary>
+    /// Gets the file filter used by this instance.
+    /// </summary>
+    /// <value>The file filter.</value>
+    public string FileFilter { get; private set; }
 
-        /// <summary>
-        /// Gets the file filter used by this instance.
-        /// </summary>
-        /// <value>The file filter.</value>
-        public string FileFilter { get; private set; }
+    /// <summary>
+    /// Performs application-defined tasks associated with freeing, releasing, or resetting unmanaged resources.
+    /// </summary>
+    public void Dispose()
+    {
+        Dispose(true);
+        GC.SuppressFinalize(this);
+    }
 
-        /// <summary>
-        /// Performs application-defined tasks associated with freeing, releasing, or resetting unmanaged resources.
-        /// </summary>
-        public virtual void Dispose()
+    protected virtual void Dispose(bool disposing)
+    {
+        if (disposing)
         {
             exitThread = true;
             if (watcherCheckThread != null)
@@ -49,72 +53,72 @@ public virtual void Dispose()
             }
             DisposeInternal();
         }
+    }
 
-        /// <summary>
-        /// Gets a list of current directories being tracked.
-        /// </summary>
-        /// <returns>A list of current directories being tracked</returns>
-        public List<string> GetTrackedDirectories()
-        {
-            return GetTrackedDirectoriesInternal();
-        }
+    /// <summary>
+    /// Gets a list of current directories being tracked.
+    /// </summary>
+    /// <returns>A list of current directories being tracked</returns>
+    public List<string> GetTrackedDirectories()
+    {
+        return GetTrackedDirectoriesInternal();
+    }
 
-        /// <summary>
-        /// Tracks the specified path.
-        /// </summary>
-        /// <param name="path">The path.</param>
-        /// <remarks>
-        /// If path is a file, this will used the parent directory. If the path is invalid, it will not fail but just skip it.
-        /// </remarks>
-        public void Track(string path)
-        {
-            TrackInternal(path);
-        }
+    /// <summary>
+    /// Tracks the specified path.
+    /// </summary>
+    /// <param name="path">The path.</param>
+    /// <remarks>
+    /// If path is a file, this will used the parent directory. If the path is invalid, it will not fail but just skip it.
+    /// </remarks>
+    public void Track(string path)
+    {
+        TrackInternal(path);
+    }
 
-        /// <summary>
-        /// UnTracks the specified path.
-        /// </summary>
-        /// <remarks>
-        /// If path is a file, this will used the parent directory. If the path is invalid, it will not fail but just skip it.
-        /// </remarks>
-        public void UnTrack(string path)
-        {
-            UnTrackInternal(path);
-        }
+    /// <summary>
+    /// UnTracks the specified path.
+    /// </summary>
+    /// <remarks>
+    /// If path is a file, this will used the parent directory. If the path is invalid, it will not fail but just skip it.
+    /// </remarks>
+    public void UnTrack(string path)
+    {
+        UnTrackInternal(path);
+    }
 
-        /// <summary>
-        /// Called when a file event occurred.
-        /// </summary>
-        /// <param name="sender">The sender.</param>
-        /// <param name="e">The file event.</param>
-        protected virtual void OnModified(object sender, FileEvent e)
-        {
-            Modified?.Invoke(sender, e);
-        }
+    /// <summary>
+    /// Called when a file event occurred.
+    /// </summary>
+    /// <param name="sender">The sender.</param>
+    /// <param name="e">The file event.</param>
+    protected virtual void OnModified(object sender, FileEvent e)
+    {
+        Modified?.Invoke(sender, e);
+    }
 
 #if !STRIDE_PLATFORM_DESKTOP
-        // Doesn't throw any exceptions on other platforms
+    // Doesn't throw any exceptions on other platforms
 
-        private void InitializeInternal()
-        {
-        }
+    private void InitializeInternal()
+    {
+    }
 
-        private void DisposeInternal()
-        {
-        }
+    private void DisposeInternal()
+    {
+    }
 
-        private void TrackInternal(string path)
-        {
-        }
+    private void TrackInternal(string path)
+    {
+    }
 
-        private void UnTrackInternal(string path)
-        {
-        }
+    private void UnTrackInternal(string path)
+    {
+    }
 
-        private List<string> GetTrackedDirectoriesInternal()
-        {
-            return new List<string>();
-        }
-#endif
+    private List<string> GetTrackedDirectoriesInternal()
+    {
+        return new List<string>();
     }
+#endif
 }