Re: [boost] Boost.Process 0.5 released

Subject: Re: [boost] Boost.Process 0.5 released
From: Klaim - Joël Lamotte (mjklaim_at_[hidden])
Date: 2012-08-18 12:45:53

Ok, boost 1.51.0 release being targeted at monday I will try with the new
version before using Boost.Process 0.5.

I just finished reading the doc, here are quick remarks (so don't take them
as strong critics):

 1. It appears simple to understand by reading the doc, except the
suggested need for macros (but I am not a specialist so maybe I'm missing
knowledge here).
 2. I suspect that there are specific reasons why the 'child' class isn't
designed in the same way than std::thread? I think it might have been
discussed before but I don't remember.
     Is there a rational somewhere about this?
 3. Why do I need to repeat the executable name if I provide the full
command line? Can I just provide the command line?
 4. I see that some example use lambdas, so C++11 enabled. Why not use auto
too then? I think some of the macro use (related to return types in
particular) can be removed by using auto.
      Obviously that's true only if the example are meant to use C++11.
 5. It is not clear to me what you mean by "resources" in the  "Cleaning up
resources" part? I just read this and I'm guessing that it's for solving
the problem of zombie process? http://en.wikipedia.org/wiki/SIGCHLD

Otherwise good work.

Joel Lamotte