Boost logo

Boost :

From: Robert Ramey (ramey_at_[hidden])
Date: 2008-03-03 11:35:34


Edward Diener wrote:
> Robert Ramey wrote:
>> I've started to look at the signals/slots libraries for a task I
>> have to do and have found it interesting.
>>
>> I'm curious about the header placement of headers and namespaces
>> used by this library.
>>
>> Specifically I find the following.
>>
>> boost/signal.hpp containing boost::signal
>> boost/visit_each.hpp containing boost::visit_each
>> boost/last_value.hpp containnig boost::last_value
>>
>> as these are part of the signals library, should they be in
>> boost/signals/... with namespaces of boost/signal/signal ... etc.?
>>
>> This is especially confusing in the case of signal.hpp which is not
>> at all were one would expect
>> to find it.
>
> I do not know where you would expect to find signal.hpp but do not
> many libraries place their main header file directly in the boost
> directory ?

I would expect to find it in boost/signals/signal.hpp where all the rest
of the signals include files are found. Did you look at the contents
of this header? boost/signal.hpp is not a "convenience header"
 including all the other headers of the library. There is a
boost/signals.hpp file in there
for that purpose.

>> Are these things where they are due to some oversight? Or were they
>> placed there for some
>> special reason. Were their separate reviews of these things. They
>> seem to be part of
>> the signals/slots libraries.

> Both visit_each.hpp and last_value.hpp seem to have been put directly
> in the boost directory because they may be used by other implementations,
> even if currently they may be only used by signals.

Hmmm - this identical to the circumstance of a few components of the
serialization library. For this I was subjected to ridiculously
disproportionate
criticism a few months ago. I wondering if you will be prevailed upon
to address this in the same manner.

Hmmm, I see that a thread-safe version of the
signals library has been placed in the review queue. Is this issue
considered sufficiently important enough in this case to expect
that be addressed in this upgrade.

Robert Ramey

> _______________________________________________
> Unsubscribe & other changes:
> http://lists.boost.org/mailman/listinfo.cgi/boost


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