Boost C++ Libraries: Ticket #9286: signals2 compile error with MSVC12 and variadic templates

attachment set

raad@… — Tue, 22 Oct 2013 07:44:27 GMT

attachment → variadic_slot.hpp.patch

Make variadic slot contructor explicit

raad@… — Tue, 22 Oct 2013 08:17:35 GMT

Sorry, it actually fixes only my use case, but of course breaks code using the implicit BindArgs... constructor.

raad@… — Tue, 22 Oct 2013 09:43:51 GMT

The problem is that VC++12 prefers the variadic slot constructor over the non-variadic constructor, so calling slot(boost::bind(...)) always calls boost::bind(boost::bind(...)), which doesn't compile.

If I understand the example in 14.8.2.4.8 of the January 2012 C++ working draft correctly, this is a compiler bug, so I have filed a bug on Microsoft Connect (ID 806150).

Frank Mori Hess — Thu, 24 Oct 2013 02:17:14 GMT

(In [86412]) As per Peter Dimov's suggestion, the variadic binding constructor for slots shouldn't accept zero arguments. There is also no need for it to accept a single argument, and not doing so avoids a bug in MSVC12. Refs #9286

raad@… — Thu, 24 Oct 2013 07:23:02 GMT

The code compiles file now. Thank you!

raad@… — Thu, 24 Oct 2013 07:23:40 GMT

The code compiles fine now. Thank you!

status changed; resolution set

Frank Mori Hess — Fri, 25 Oct 2013 04:43:05 GMT

status new → closed
resolution → fixed

(In [86424]) Merged changeset 86412 from trunk. Fixes #9286