Skip to content

Commit

Permalink
(#5) Connect TODOs with the issue
Browse files Browse the repository at this point in the history
  • Loading branch information
ForNeVeR committed Dec 27, 2023
1 parent f7a211d commit 5339e8d
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion ProcessDoctor.Backend.Windows/WmiProcessMonitor.cs
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ public WmiProcessMonitor(Lifetime lifetime, ILog logger)
"select * from __InstanceDeletionEvent within 1 where TargetInstance isa 'Win32_Process'",
e =>
{
// TODO: Optimize this
// TODO[#5]: Optimize this
var process = (ManagementBaseObject)e.NewEvent["TargetInstance"];
var processId = Convert.ToUInt32(process.Properties["ProcessID"].Value);
lock (_locker)
Expand Down
4 changes: 2 additions & 2 deletions ProcessDoctor/ViewModels/ProcessTreeViewModel.cs
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@

namespace ProcessDoctor.ViewModels;

// TODO: optimize all IndexOf/RemoveAt/linear Remove calls in this class, perhaps by removing them
// TODO[#5]: optimize all IndexOf/RemoveAt/linear Remove calls in this class, perhaps by removing them
public class ProcessTreeViewModel
{
private readonly ILog _logger;
Expand Down Expand Up @@ -74,7 +74,7 @@ private void OnAdd(IEnumerable<ProcessModel> processes)
?? Array.Empty<ProcessViewModel>();
foreach (var child in children)
{
// TODO: Optimize
// TODO[#5]: Optimize
_viewModels.RemoveAt(_viewModels.IndexOf(child));
}
_orphanedModels.Remove(process.Id);
Expand Down

0 comments on commit 5339e8d

Please sign in to comment.