cleaned-up dispose TODO (#22703)

src/common/interop/interop-tests/InteropTests.cs

@@ -66,8 +66,6 @@ namespace Microsoft.Interop.Tests
                     ClientPipe.Dispose();
                 }
 
-                // TODO: free unmanaged resources (unmanaged objects) and override finalizer
-                // TODO: set large fields to null
                 disposedValue = true;
             }
         }

src/modules/PowerOCR/PowerOCR/Keyboard/KeyboardMonitor.cs

@@ -151,12 +151,9 @@ public class KeyboardMonitor : IDisposable
         {
             if (disposing)
             {
-                // TODO: dispose managed state (managed objects)
                 _keyboardHook?.Dispose();
             }
 
-            // TODO: free unmanaged resources (unmanaged objects) and override finalizer
-            // TODO: set large fields to null
             disposedValue = true;
         }
     }

src/modules/colorPicker/ColorPickerUI/App.xaml.cs

@@ -89,8 +89,6 @@ namespace ColorPickerUI
 
                 _themeManager?.Dispose();
 
-                // TODO: free unmanaged resources (unmanaged objects) and override finalizer
-                // TODO: set large fields to null
                 disposedValue = true;
             }
         }

src/modules/colorPicker/ColorPickerUI/Keyboard/KeyboardMonitor.cs

@@ -162,12 +162,9 @@ namespace ColorPicker.Keyboard
             {
                 if (disposing)
                 {
-                    // TODO: dispose managed state (managed objects)
                     _keyboardHook?.Dispose();
                 }
 
-                // TODO: free unmanaged resources (unmanaged objects) and override finalizer
-                // TODO: set large fields to null
                 disposedValue = true;
             }
         }

src/modules/fancyzones/editor/FancyZonesEditor/App.xaml.cs

@@ -209,8 +209,6 @@ namespace FancyZonesEditor
                     _themeManager?.Dispose();
                 }
 
-                // TODO: free unmanaged resources (unmanaged objects) and override finalizer
-                // TODO: set large fields to null
                 _isDisposed = true;
                 Logger.LogInfo("FancyZones Editor disposed");
             }

src/modules/imageresizer/tests/Models/ResizeOperationTests.cs

@@ -550,8 +550,6 @@ namespace ImageResizer.Models
                     _directory.Dispose();
                 }
 
-                // TODO: free unmanaged resources (unmanaged objects) and override finalizer
-                // TODO: set large fields to null
                 disposedValue = true;
             }
         }

src/modules/imageresizer/tests/Test/TestDirectory.cs

@@ -64,8 +64,6 @@ namespace ImageResizer
                     }
                 }
 
-                // TODO: free unmanaged resources (unmanaged objects) and override finalizer
-                // TODO: set large fields to null
                 disposedValue = true;
             }
         }

src/modules/imageresizer/ui/App.xaml.cs

@@ -67,8 +67,6 @@ namespace ImageResizer
                     _themeManager?.Dispose();
                 }
 
-                // TODO: free unmanaged resources (unmanaged objects) and override finalizer
-                // TODO: set large fields to null
                 _isDisposed = true;
             }
         }

src/modules/imageresizer/ui/ViewModels/ProgressViewModel.cs

@@ -100,8 +100,6 @@ namespace ImageResizer.ViewModels
                     _cancellationTokenSource.Dispose();
                 }
 
-                // TODO: free unmanaged resources (unmanaged objects) and override finalizer
-                // TODO: set large fields to null
                 disposedValue = true;
             }
         }

src/modules/launcher/PowerLauncher/App.xaml.cs

@@ -293,18 +293,10 @@ namespace PowerLauncher
                     _themeManager?.Dispose();
                 }
 
-                // TODO: free unmanaged resources (unmanaged objects) and override finalizer
-                // TODO: set large fields to null
                 Log.Info("End PowerToys Run Exit ----------------------------------------------------  ", GetType());
             });
         }
 
-        // // TODO: override finalizer only if 'Dispose(bool disposing)' has code to free unmanaged resources
-        // ~App()
-        // {
-        //     // Do not change this code. Put cleanup code in 'Dispose(bool disposing)' method
-        //     Dispose(disposing: false);
-        // }
         public void Dispose()
         {
             // Do not change this code. Put cleanup code in 'Dispose(bool disposing)' method

src/modules/launcher/PowerLauncher/MainWindow.xaml.cs

@@ -4,7 +4,6 @@
 
 using System;
 using System.ComponentModel;
-using System.IO;
 using System.Linq;
 using System.Reactive.Linq;
 using System.Runtime.InteropServices;
@@ -733,19 +732,11 @@ namespace PowerLauncher
                     _hwndSource?.Dispose();
                 }
 
-                // TODO: free unmanaged resources (unmanaged objects) and override finalizer
-                // TODO: set large fields to null
                 _firstDeleteTimer = null;
                 _disposedValue = true;
             }
         }
 
-        // // TODO: override finalizer only if 'Dispose(bool disposing)' has code to free unmanaged resources
-        // ~MainWindow()
-        // {
-        //     // Do not change this code. Put cleanup code in 'Dispose(bool disposing)' method
-        //     Dispose(disposing: false);
-        // }
         public void Dispose()
         {
             // Do not change this code. Put cleanup code in 'Dispose(bool disposing)' method

src/modules/previewpane/GcodePreviewHandler/GcodePreviewHandler.cs

@@ -56,8 +56,6 @@ namespace Microsoft.PowerToys.PreviewHandler.Gcode
                     _gcodePreviewControl.Dispose();
                 }
 
-                // TODO: free unmanaged resources (unmanaged objects) and override finalizer
-                // TODO: set large fields to null
                 disposedValue = true;
             }
         }

src/modules/previewpane/MarkdownPreviewHandler/MarkdownPreviewHandler.cs

@@ -56,8 +56,6 @@ namespace Microsoft.PowerToys.PreviewHandler.Markdown
                     _markdownPreviewHandlerControl.Dispose();
                 }
 
-                // TODO: free unmanaged resources (unmanaged objects) and override finalizer
-                // TODO: set large fields to null
                 disposedValue = true;
             }
         }

src/modules/previewpane/MonacoPreviewHandler/MonacoPreviewHandler.cs

@@ -55,8 +55,6 @@ namespace Microsoft.PowerToys.PreviewHandler.Monaco
                     _monacoPreviewHandlerControl.Dispose();
                 }
 
-                // TODO: free unmanaged resources (unmanaged objects) and override finalizer
-                // TODO: set large fields to null
                 _disposedValue = true;
                 this.Unload();
             }

src/modules/previewpane/PdfPreviewHandler/PdfPreviewHandler.cs

@@ -56,8 +56,6 @@ namespace Microsoft.PowerToys.PreviewHandler.Pdf
                     _pdfPreviewHandlerControl.Dispose();
                 }
 
-                // TODO: free unmanaged resources (unmanaged objects) and override finalizer
-                // TODO: set large fields to null
                 _disposedValue = true;
             }
         }

src/modules/previewpane/SvgPreviewHandler/SvgPreviewHandler.cs

@@ -56,8 +56,6 @@ namespace Microsoft.PowerToys.PreviewHandler.Svg
                     _svgPreviewControl.Dispose();
                 }
 
-                // TODO: free unmanaged resources (unmanaged objects) and override finalizer
-                // TODO: set large fields to null
                 disposedValue = true;
             }
         }

src/modules/previewpane/common/examplehandler/TestCustomHandler.cs

@@ -45,8 +45,6 @@ namespace Common
                     _previewHandlerControl.Dispose();
                 }
 
-                // TODO: free unmanaged resources (unmanaged objects) and override finalizer
-                // TODO: set large fields to null
                 disposedValue = true;
             }
         }

src/settings-ui/Settings.UI/Controls/ShortcutControl/ShortcutControl.xaml.cs

@@ -373,12 +373,9 @@ namespace Microsoft.PowerToys.Settings.UI.Controls
             {
                 if (disposing)
                 {
-                    // TODO: dispose managed state (managed objects)
                     hook.Dispose();
                 }
 
-                // TODO: free unmanaged resources (unmanaged objects) and override finalizer
-                // TODO: set large fields to null
                 disposedValue = true;
             }
         }