Opened 10 years ago

Closed 10 years ago

#8346 closed Bugs (fixed)

Broken vc11 build at usage BOOST_STATIC_ASSERT_MSG after r82886

Reported by: Vladimir Merzliakov <wanderer@…> Owned by: John Maddock
Milestone: To Be Determined Component: static_assert
Version: Boost Development Trunk Severity: Regression
Keywords: Cc:

Description

After this commit my project code build fail with errors in boost headers at BOOST_STATIC_ASSERT_MSG uses.

What difference in then/else branches?

# ifndef BOOST_NO_CXX11_VARIADIC_MACROS # define BOOST_STATIC_ASSERT_MSG( ... ) static_assert(VA_ARGS) # else # define BOOST_STATIC_ASSERT_MSG( ... ) static_assert(VA_ARGS) # endif

Look like copy paste typo. Change else to expected from old version code # define BOOST_STATIC_ASSERT_MSG( B, Msg ) BOOST_STATIC_ASSERT( B ) fix build problem.

Attachments (1)

static_assert.hpp.patch (566 bytes ) - added by Vladimir Merzliakov <wanderer@…> 10 years ago.
Fix copy paste typo in BOOST_STATIC_ASSERT_MSG

Download all attachments as: .zip

Change History (5)

comment:1 by Vladimir Merzliakov <wanderer@…>, 10 years ago

Sorry i has been wrong about build fix :( Anyway look like copy paster and possuble problem in other part of commit. I will continue research problem.

by Vladimir Merzliakov <wanderer@…>, 10 years ago

Attachment: static_assert.hpp.patch added

Fix copy paste typo in BOOST_STATIC_ASSERT_MSG

comment:2 by Vladimir Merzliakov <wanderer@…>, 10 years ago

Buid problems with vc11 has been at my side. But I sure that suggested patch is required for fix copy-paste error that can create problems at platforms where variadic macroes not suported

comment:3 by viboes, 10 years ago

Component: Nonestatic_assert
Owner: set to John Maddock

comment:4 by John Maddock, 10 years ago

Resolution: fixed
Status: newclosed

(In [83806]) Fix for compilers that have no variadic macros but do have static_assert (are there any??) Fixes #8346.

Note: See TracTickets for help on using tickets.