Re: [boost] Signals review

From: Darin Adler (darin_at_[hidden])
Date: 2002-02-22 01:30:47

On 2/21/02 6:36 PM, "Douglas Gregor" <gregod_at_[hidden]> wrote:

> "connection" (and soon "scoped_connection") definately need to be in the
> signals namespace. I'm not as sure about 'signal' and 'slot', because they
> don't seem to have as many uses outside the library. One comment on this:
> might it be confusion if signal is boost::signals::signal and function is
> boost::function, since they are very similar concepts.

My initial thoughts about name clashes were that signals from <signal.h> are
something quite different (POSIX?), and slot had a different meaning in the
Newton operating system, something like a named member. I think your
judgment here is good, though, and I was probably mistaken.

I feel more strongly about making sure the locations of the header files
match the namespace nesting, and I hope you'll consider that.

> Hmmm. Are you referring to actually overloading the '&&' and '||' operators,
> or to combiners such as:
> 
> struct stop_on_first_true {

The latter, combiners. What you said.

>> Perhaps the interface for finding trackable objects inside Boost.Bind
>> should be documented. It's part of the public interface of Boost.Signals,
>> in a way, no?
> 
> I was hoping that the visit_each documentation would suffice, because that is
> essentially the only link between the libraries.

My mistake, I missed that. I don't think I read the visit_each documentation
at all. I'll look again.

    -- Darin