Skip to content

Commit

Permalink
Fix typo in cancellation message (#4769)
Browse files Browse the repository at this point in the history
  • Loading branch information
nohwnd authored Nov 27, 2023
1 parent 643dca7 commit e32b169
Showing 1 changed file with 3 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -205,8 +205,9 @@ public void DetachFromTargetProcess(int targetProcessId)
// But not all versions of procdump have that parameter (definitely not the one we are getting from the Procdump 0.0.1 nuget package), and it works reliably.
// What was not reliable before was that we sent the message and immediately killed procdump, that caused testhost to crash occasionally, because procdump was not detached,
// and killing the process when it is not detached takes the observed process with it.
new Win32NamedEvent($"Procdump-{targetProcessId}").Set();
EqtTrace.Info($"ProcDumpDumper.DetachFromTargetProcess: Cancel event was sent to Procdump.");
var eventName = $"ProcDump-{targetProcessId}";
new Win32NamedEvent(eventName).Set();
EqtTrace.Info($"ProcDumpDumper.DetachFromTargetProcess: Cancel event '{eventName}' was sent to Procdump.");

var sw = Stopwatch.StartNew();
var exited = _procDumpProcess.WaitForExit(_timeout);
Expand Down

0 comments on commit e32b169

Please sign in to comment.