
On Wed, Oct 23, 2013 at 2:34 PM, Frank Mori Hess
On Wed, Oct 23, 2013 at 12:02 PM, Peter Dimov
wrote: Beman Dawes wrote:
Should we continue to define BOOST_NO_CXX11_VARIADIC_TEMPLATES for VC++ 2013, or should we not define it and get signals2 working with a problem specific workaround?
Get signals2 working, I'd say. The variadic constructor is already a bit broken in that it incorrectly accepts zero arguments, so why not just constrain it to two or more.
template
slot( A1 const& a1, A2 const& a2, Args const&... args ) { ... } That sounds fine by me. I can commit the change to trunk tonight, should I also commit it to release, or wait for tests to cycle?
I've just committed (r86408), the BOOST_NO_CXX11_VARIADIC_TEMPLATES removal for VS 2013, but will wait roughly 24-hours to give tests a chance to cycle before merging to trunk. Why don't you do the same? That still meets the Monday for merging to release without having to ask permission. Thanks, --Beman