From: Alexandre F. <ale...@gm...> - 2008-05-30 09:10:04
|
On Fri, May 30, 2008 at 10:42 AM, Donal K. Fellows <don...@ma...> wrote: > Alexandre Ferrieux wrote: >> >> This future cleanup will be orthogonal to the present patch, touching >> a lower layer, and optimizing simultaneously the Windows versions of >> [open |] and [chan pipe], by removing any extra threads for pipes, and >> the associated TerminateThread leak (fixing bug 1958930). > > I approve in general of rationalizing this mess. What's to discuss? Any > evil nuances in there that need airing? :-) See the recent comments on 1958930. David is providing oxygen. I need to study his IOCPSOCK implementation first, but basically I propose to first do a half-rationalization to remove the TerminateThread (but keep the thread). Then for the full-rationalization (remove the thread), much more work is needed but I know I can count on him. In the meantime, I would be very grateful for a review of 304 ;-) I also have a plan for overhauling [exec], but I'm a bit of a sequential guy... -Alex |