perf(device::cache): Rework the refresh job to be always in task using the underlying lib

Belphemur · Belphemur · commit 7802f63290e8 · 2023-12-14T10:38:13.000-05:00
Fixes #1323
diff --git a/SoundSwitch/Framework/Audio/Lister/Job/DebounceRefreshJob.cs b/SoundSwitch/Framework/Audio/Lister/Job/DebounceRefreshJob.cs
@@ -25,7 +25,8 @@ public DebounceRefreshJob(DeviceState state, IAudioDeviceLister lister, ILogger
 
     public Task ExecuteAsync(CancellationToken cancellationToken)
     {
-        return Task.Factory.StartNew(() => _lister.Refresh(cancellationToken), cancellationToken);
+        _lister.Refresh(cancellationToken);
+        return Task.CompletedTask;
     }
 
     public Task OnFailure(JobException exception)
diff --git a/SoundSwitch/SoundSwitch.csproj b/SoundSwitch/SoundSwitch.csproj
@@ -50,7 +50,7 @@
         <PackageReference Include="AuthenticodeExaminer" Version="0.3.0" />
         <PackageReference Include="ContribSentry" Version="4.0.0" />
         <PackageReference Include="JetBrains.Annotations" Version="2023.3.0" />
-        <PackageReference Include="Job.Scheduler" Version="3.1.6" />
+        <PackageReference Include="Job.Scheduler" Version="3.1.8" />
         <PackageReference Include="Markdig" Version="0.33.0" />
         <PackageReference Include="Microsoft.CSharp" Version="4.7.0" />
         <PackageReference Include="RailSharp" Version="1.0.0" />

Original file line number	Diff line number	Diff line change
`@@ -25,7 +25,8 @@ public DebounceRefreshJob(DeviceState state, IAudioDeviceLister lister, ILogger`
`25`	`25`
`26`	`26`	`public Task ExecuteAsync(CancellationToken cancellationToken)`
`27`	`27`	`{`
`28`		`- return Task.Factory.StartNew(() => _lister.Refresh(cancellationToken), cancellationToken);`
	`28`	`+ _lister.Refresh(cancellationToken);`
	`29`	`+ return Task.CompletedTask;`
`29`	`30`	`}`
`30`	`31`
`31`	`32`	`public Task OnFailure(JobException exception)`