Date: 2001-09-03 12:52:14
--- In boost_at_y..., Jeremy Siek <jsiek_at_c...> wrote:
> in condition.html:
> Template parameter "Pr" should be replaced with "Predicate"
> and linked to the definition:
Do we make external links like this in Boost documentation? I'm
willing to make the change, just want to make sure it's appropriate.
> Links should be added to the documentation for xtime
> and that documentation should be created.
Hopefully will be done today and checked in/uploaded to files section.
> Move xlock.hpp to a detail/ directory, or document it and
> make it "public".
Currently it is documented, but it's also in the detail namespace (I
expected others to make use of the templates). I guess I've left
these types in limbo, partially a detail item and partially public
concepts. A decision has to be made as to which direction it should
> in mutex.html:
> there are no examples here for try and timed mutexes.
> Perhaps put in links to the examples in the *_lock.html files.
I'll think about this.
> in index.html:
> the link to tss.html is broke, there is no such file.
Yes, pointed out earlier and will be corrected in the upload today.
> in thread_specific_ptr.html:
> in the example, the static cast "<" and ">" need to be
> changed to "<" and ">"
Ahh... good catch. I'll correct.
> in scoped_lock.html (and the other lock docs too)
> Header, just use mutex.hpp here, xlock.hpp is an implementation
> detail, right?
We can't use just mutex.hpp, we'd have to use recursive_mutex.hpp as
well. However, the type isn't defined in either of these, but is
defined in xlock.hpp. Even if it's fully turned into a detail I
don't think this should be changed here.
> more comments to come...
I'm happily awaiting :).
Boost list run by bdawes at acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk