From 0655497762edb856301a8aa03e18f7052a762eab Mon Sep 17 00:00:00 2001 From: Yu Leng Date: Mon, 8 Dec 2025 13:37:06 +0800 Subject: [PATCH] Refactor SettingsUtils and update project ID Refactored `SettingsUtils` initialization across multiple files (`App.xaml.cs`, `MainWindow.xaml.cs`, `MainViewModel.cs`) to use `SettingsUtils.Default` instead of creating new instances. This improves consistency, reduces redundancy, and promotes better resource management. Updated the project ID for `PowerDisplayModuleInterface.vcxproj` in `PowerToys.slnx` to reflect a configuration or structural change in the project setup. --- PowerToys.slnx | 2 +- .../powerdisplay/PowerDisplay/PowerDisplayXAML/App.xaml.cs | 2 +- .../PowerDisplay/PowerDisplayXAML/MainWindow.xaml.cs | 2 +- .../powerdisplay/PowerDisplay/ViewModels/MainViewModel.cs | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-) diff --git a/PowerToys.slnx b/PowerToys.slnx index 5516ed2ade..a79340db47 100644 --- a/PowerToys.slnx +++ b/PowerToys.slnx @@ -655,7 +655,7 @@ - + diff --git a/src/modules/powerdisplay/PowerDisplay/PowerDisplayXAML/App.xaml.cs b/src/modules/powerdisplay/PowerDisplay/PowerDisplayXAML/App.xaml.cs index 9668d16984..0e45f40269 100644 --- a/src/modules/powerdisplay/PowerDisplay/PowerDisplayXAML/App.xaml.cs +++ b/src/modules/powerdisplay/PowerDisplay/PowerDisplayXAML/App.xaml.cs @@ -32,7 +32,7 @@ namespace PowerDisplay /// private const string ProcessReadyEventName = "Local\\PowerToys_PowerDisplay_Ready"; - private readonly ISettingsUtils _settingsUtils = new SettingsUtils(); + private readonly ISettingsUtils _settingsUtils = SettingsUtils.Default; private Window? _mainWindow; private int _powerToysRunnerPid; private TrayIconService? _trayIconService; diff --git a/src/modules/powerdisplay/PowerDisplay/PowerDisplayXAML/MainWindow.xaml.cs b/src/modules/powerdisplay/PowerDisplay/PowerDisplayXAML/MainWindow.xaml.cs index 2fa8104997..08c28ad8f0 100644 --- a/src/modules/powerdisplay/PowerDisplay/PowerDisplayXAML/MainWindow.xaml.cs +++ b/src/modules/powerdisplay/PowerDisplay/PowerDisplayXAML/MainWindow.xaml.cs @@ -32,7 +32,7 @@ namespace PowerDisplay [DynamicallyAccessedMembers(DynamicallyAccessedMemberTypes.PublicProperties | DynamicallyAccessedMemberTypes.PublicMethods)] public sealed partial class MainWindow : WindowEx, IDisposable { - private readonly ISettingsUtils _settingsUtils = new SettingsUtils(); + private readonly ISettingsUtils _settingsUtils = SettingsUtils.Default; private MainViewModel? _viewModel; private AppWindow? _appWindow; private bool _isExiting; diff --git a/src/modules/powerdisplay/PowerDisplay/ViewModels/MainViewModel.cs b/src/modules/powerdisplay/PowerDisplay/ViewModels/MainViewModel.cs index d3b79a363f..734269008d 100644 --- a/src/modules/powerdisplay/PowerDisplay/ViewModels/MainViewModel.cs +++ b/src/modules/powerdisplay/PowerDisplay/ViewModels/MainViewModel.cs @@ -62,7 +62,7 @@ public partial class MainViewModel : INotifyPropertyChanged, IDisposable _isScanning = true; // Initialize settings utils - _settingsUtils = new SettingsUtils(); + _settingsUtils = SettingsUtils.Default; _stateManager = new MonitorStateManager(); // Initialize the monitor manager