On Wed, 06 Jun 2007 23:38:36 +0900, David Klein wrote:

Boris wrote:

...

void Hello() { } boost::signal sig; sig.connect(&Hello); // OK sig.disconnect(&Hello); // Error [...]Hi Boris,

http://www.boost.org/doc/html/signals/tutorial.html#id1627590

Thanks for the link! But do I really have to collect and maintain various connection objects for a signal? There is out-of-the-box support for attaching callback functions to a signal. But in order to detach them I have to maintain the connection objects for all the callback functions myself? What's the rationale for this? Or is there is any utility class in Boost.Signal I miss? Boris

On Jun 6, 2007, at 11:10 AM, Boris wrote:

On Wed, 06 Jun 2007 23:38:36 +0900, David Klein wrote:

...

Boris wrote:

...

void Hello() { } boost::signal sig; sig.connect(&Hello); // OK sig.disconnect(&Hello); // Error [...]http://www.boost.org/doc/html/signals/tutorial.html#id1627590

According to http://www.boost.org/doc/html/signals/ tutorial.html#id1627841 the code above should work? However I can't compile it with VC++ 2005 SP1. Anyone with the same problem? The error message I get is:

The code supporting this feature was accidentally disabled for VC++ in Boost 1.33.1. This oversight was fixed in Boost 1.34.0. - Doug

Hi Boris, On Jun 7, 2007, at 12:23 AM, Boris wrote:

On Thu, 07 Jun 2007 09:34:32 +0900, Douglas Gregor wrote:

...

The code supporting this feature was accidentally disabled for VC++ in Boost 1.33.1. This oversight was fixed in Boost 1.34.0.

Oh, thanks, good to know. Is there anything I can do about it (like setting a preprocessor directive to enable the feature)? Or is it something more difficult to change in the headers? I wonder as while I plan to upgrade to Boost 1.34.0 in the future I'm not yet ready to do so.

It's actually really easy to fix. If you poke through the headers in boost/function and boost/function/detail, you'll find a couple references to a mysterious constant 0x1700 in some BOOST_WORKAROUND uses... change that to "1300" (decimal, not hex) and it should enable this feature. - Doug

On Fri, 08 Jun 2007 10:55:39 +0900, Douglas Gregor wrote:

On Jun 7, 2007, at 9:53 PM, Douglas Gregor wrote:

...

It's actually really easy to fix. If you poke through the headers in boost/function and boost/function/detail, you'll find a couple references to a mysterious constant 0x1700 in some BOOST_WORKAROUND uses... change that to "1300" (decimal, not hex) and it should enable this feature.

Ugh, sorry. "Function" on the brain today... the offending 0x1700 is in boost/signals/signal_template.hpp.

Thanks, I found it! As it's a change in a template I suppose I don't need to rebuild the library? Boris

On Thu, 07 Jun 2007 09:34:32 +0900, Douglas Gregor wrote:

[...]

...

...

...

void Hello() { } boost::signal sig; sig.connect(&Hello); // OK sig.disconnect(&Hello); // Error [...]http://www.boost.org/doc/html/signals/tutorial.html#id1627590

According to http://www.boost.org/doc/html/signals/ tutorial.html#id1627841 the code above should work? However I can't compile it with VC++ 2005 SP1. Anyone with the same problem? The error message I get is:

The code supporting this feature was accidentally disabled for VC++ in Boost 1.33.1. This oversight was fixed in Boost 1.34.0.

Doug, I upgraded now to Boost 1.34.0 only to run into another problem with Boost.Signal: C:\Boost\include\boost-1_34\boost/signals/signal_template.hpp(242) : error C2666: 'boost::operator ==' : 4 overloads have similar conversions C:\Boost\include\boost-1_34\boost/function/function_template.hpp(699): could be 'void boost::operator ==(const boost::function1 &,const boost::function1 &)' [...] C:\Boost\include\boost-1_34\boost/function/function_base.hpp(647): or 'bool boost::operator ==(Functor,const boost::function_base &)' [...] C:\Boost\include\boost-1_34\boost/function/function_base.hpp(638): or 'bool boost::operator ==<Function>(const boost::function_base &,Functor)' [...] or 'built-in C++ operator==(void (__thiscall boost::function1::dummy::* )(void), void (__thiscall boost::function1::dummy::* )(void))' [...] while trying to match the argument list '(boost::function<Signature>, const boost::function<Signature>)' [...] C:\Boost\include\boost-1_34\boost/signals/signal_template.hpp(224) : see reference to function template instantiation 'void boost::signal1::do_disconnect<T>(const Function &,boost::mpl::bool_)' being compiled [...] If required I can copy&paste the full error message (I only dropped the template parameters for better readability). Is this a known problem? Anything I can do about it? Boris

On Wed, 04 Jul 2007 00:32:31 +0900, Doug Gregor wrote:

[...]It appears that we're running into a bug in Visual C++ 7.1's handling of partial ordering of function templates. We'll need to find a workaround and put it into Signals itself; I don't think it can be done without changing the Signals library. Could you report this as a bug at http://svn.boost.org/trac/boost/report ?

Done! See http://svn.boost.org/trac/boost/ticket/1076 Boris