search

vezel-dev / cathode

A terminal-centric replacement for the .NET console APIs.
https://docs.vezel.dev/cathode
BSD Zero Clause License
91 stars 7 forks source link

`ChildProcess.Kill()` can let internal exceptions through #139

Closed alexrp closed 9 months ago

alexrp commented 9 months ago

https://github.com/vezel-dev/cathode/blob/69f1c657165bede3829d02b50b95e95894dcdf76/src/core/Processes/ChildProcess.cs#L172-L180

https://learn.microsoft.com/en-us/dotnet/api/system.diagnostics.process.kill#system-diagnostics-process-kill(system-boolean)

I suspect the documentation here may not match the current Process implementation...