On Thu, Nov 5, 2015 at 12:27 PM, Johannes Sixt <[email protected]> wrote: > However, I think that the infrastructure can be simplified even further > to a level that we do not need additional emulation on Windows. > > First let me say that I find it very questionable that the callbacks > receive a struct child_process.
ok, I try to clean that up as well. However I made this choice as we need to set the working dir as well as the environment variables. I guess these will go into an extra argument then > Here is a prototype patch. Feel free to pick it up. It marks a process > whose EOF we have found by setting .err to -1. It's probably better to > extend the meaning of the in_use indicator for this purpose. Thanks for the proposal, I'll take that and make in_use a tristate for now (an enum consisting of FREE, WORKING, WAIT_CLEANUP) Thanks, Stefan -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
