[boost] Re: [Formal review] Collection Traits library

From: Thorsten Ottosen (nesotto_at_[hidden])
Date: 2004-04-29 19:31:43

"Pavel Vozenilek" <pavel_vozenilek_at_[hidden]> wrote in message
news:Sea2-DAV34aYaZlbB9u0000cd78_at_hotmail.com...
| > | It would be useful to have many more examples in documentation,
| > | showing how to use it together with other Boost libraries.
| >
| > Can you give an example of what you have in mind?
| >
| I mean snippet for each of function provided by the library,
| applied to more exotic containers - just to show it works
| with them and how.

ok.
|  _______________________________________________________
| > | 10. collection_traits.html, Semantics section, tables:
| > |     it doesn't make sense to me what middle column contains.
| >
| > ok, would a table with
| >
| > --------------Abbreviations-----------
| > SC = std container
| > T    = the type used in arrays
| > P   = std::pair
| > etc
| >
| > help
| >
| > ?
| >
| Yes.

ok.

|  _______________________________________________________
| > | 12. collection_traits.html, Portability section:
| > |
| > | bcc6 ==>> BCB 6.4
| >
| > I'm just curious, did you compile the test with that compiler?
| >
| Tried now. I got error:
[snip]

| fixed with:
|
| #if BOOST_WORKAROUND( __BORLANDC__, BOOST_TESTED_AT(0x564))
| #  undef BOOST_CT_NO_ARRAY_SUPPORT
| #endif
|
| in config.hpp (or something like this).

thanks.

| ------------------
| BCB doesn't like to move function via using directive. E.g. in begin.hpp:
|
| } // namespace 'collection_traits'
| using collection_traits::begin;  <<<=== this makes BCB crash
| } // namespace boost
|
| Either BCB will be unsupported or the inner namespace needs to be
| conditionally removed for this compiler. If you want to try the port,
| it should maybe wait all other problems get fixed.

yes, lets wait a bit.

| > Maybe that could justify having reverse_iterator_of<> and
| > const_reverse_iterator_of<>
| > + rbegin(), rend(). ?
| >
| Yes, this looks as good idea.

ok.

| > hm...I think you're looking in a file that is depricated..
| > detail/function.hpp
| > is not officially part of the dist.
| >
| ???  I use the zip from Boost Files section, downloaded just today.

well, its my mistake that the file is part of the distribution, but
its not intended to be there.

br

Thorsten