Re: [boost] Subject: Formal Review of Proposed Boost.Process library starts tomorrow

Subject: Re: [boost] Subject: Formal Review of Proposed Boost.Process library starts tomorrow
From: Steven Watanabe (watanabesj_at_[hidden])
Date: 2011-02-07 23:14:37

AMDG

On 2/7/2011 11:09 AM, Vicente Botet wrote:
>
> private functions like invalid_handle don't need to be documented. Shouldn't
> the nested impl class be private?
>

impl is private.  I usually wrap private members
in \cond \endcond to avoid this.  It should probably
be fixed in the doxygen to boostbook conversion, however.

> [pistream/postream]
> class pistream : public std::istream, public boost::noncopyable
>
> * inheritance missing on the documentation.

It's lost by doxygen.  I don't know whether there's an
easy way to fix it.

> * constructor must be documented explicit
>

Fixed in trunk BoostBook.

In Christ,
Steven Watanabe