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