Boost logo

Boost :

Subject: Re: [boost] [Boost-users] Subject: Formal Review of Proposed Boost.Process library starts tomorrow
From: Vicente Botet (vicente.botet_at_[hidden])
Date: 2011-02-08 02:06:14


AMDG

Review part 1, documentation:

* Introduction: eg. should be e.g.

> * "On POSIX systems you must use the macros defined
> in sys/wait.h to interpret exit codes."
>
> This forces me to use #ifdefs in my code. Isn't
> there a way to abstract this better? At least
> you could wrap the macros and have a trivial
> implementation for windows (WIFEXITED -> true)
>

+1

>
> * context::setup: The reference specifies boost::function< void()>,
> but this is obviously only true for posix. It should
> describe the two different behaviors.
>

Is this yet a limitation of Doxigen?

Vicente

-- 
View this message in context: http://boost.2283326.n4.nabble.com/Subject-Formal-Review-of-Proposed-Boost-Process-library-starts-tomorrow-tp3263607p3273421.html
Sent from the Boost - Dev mailing list archive at Nabble.com.

Boost list run by bdawes at acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk