maxmustermann/PowerToys
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

[Hotfix] PowerToys Awake - fixing CPU usage (#11978)

Browse source 
* Change how background threads operate
This should reduce CPU usage.

* Well there is just no need for the console here

* Need to keep the full name sanitized
This commit is contained in:
Den Delimarsky 2021-06-30 04:04:31 -07:00 committed by crutkas
parent 15ba3d57be
commit 5e26a0a3af
2 changed files with 6 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
src/modules/awake/Awake/Core/APIHelper.cs
View file

@ -184,13 +184,10 @@ namespace Awake.Core
if (success)
{
_log.Info($"Initiated indefinite keep awake in background thread: {GetCurrentThreadId()}. Screen on: {keepDisplayOn}");
while (true)
{
if (_threadToken.IsCancellationRequested)
{
_threadToken.ThrowIfCancellationRequested();
}
}
WaitHandle.WaitAny(new[] { _threadToken.WaitHandle });
return success;
}
else
{

4
src/modules/awake/Awake/Program.cs
View file

@ -28,6 +28,7 @@ namespace Awake
{
private static Mutex? _mutex = null;
private const string AppName = "Awake";
private const string FullAppName = "PowerToys " + AppName;
private static FileSystemWatcher? _watcher = null;
private static SettingsUtils? _settingsUtils = null;
@ -127,7 +128,6 @@ namespace Awake
private static void ForceExit(string message, int exitCode)
{
_log.Info(message);
Console.ReadKey();
Environment.Exit(exitCode);
}
@ -180,7 +180,7 @@ namespace Awake
.Select(e => e.EventArgs)
.Subscribe(HandleAwakeConfigChange);
TrayHelper.SetTray(AppName, new AwakeSettings());
TrayHelper.SetTray(FullAppName, new AwakeSettings());
// Initially the file might not be updated, so we need to start processing
// settings right away.