14 Jul
2023
14 Jul
'23
6:33 p.m.
Do we know that the thread actually has to die for this condition to kick in? Maybe this is evident somewhere in the tests, but I'm having a hard time finding it [they are pretty large and complex, though this is probably unavoidable].
I.e. I'm seeing two conditions for asyncs to get terminated:
(1) APC port + no event + thread dies
(2) no handles to the object left in originating process [+ thread dies?]
We have (1) already implemented, and this patch series is adding (2), but it's not clear to me just from this patch that the "thread dies" part is necessary for (2). Is it? Am I missing a test somewhere?