Opened 11 years ago

Closed 9 years ago

Last modified 9 years ago

#1614 closed Bugs (fixed)

vector workaround needed for MSVC10

Reported by: François Barel Owned by: Joel de Guzman
Milestone: Boost 1.43.0 Component: fusion
Version: Boost Development Trunk Severity: Problem
Keywords: MSVC VC10 vector Cc: frabar666@…



It seems the vector constructor workaround used for MSVC8 and earlier (ctor_helper in boost/fusion/container/vector/vector.hpp) is also required for MSVC9 RTM.

Unfortunately I don't have a simple test case ready. My observation is simply that my Spirit2 parser (using fusion tuples and vectors as rule attributes) doesn't compile on VC9 unless I enable that workaround with the attached patch.


Attachments (1)

fusion-MSVC9.diff (766 bytes) - added by François Barel 11 years ago.

Download all attachments as: .zip

Change History (8)

Changed 11 years ago by François Barel

Attachment: fusion-MSVC9.diff added

comment:1 Changed 10 years ago by Stjepan Rajko

[Bug Sprint]

Boost trunk still uses #if BOOST_WORKAROUND(BOOST_MSVC, <= 1400), so this patch has not been applied. Is it still needed?

comment:2 in reply to:  1 Changed 10 years ago by François Barel <frabar666@…>

Cc: frabar666@… added

Yes, this patch is still needed. Please see!-td22203420.html for a repro by Sergei Politov.

comment:3 Changed 10 years ago by Stjepan Rajko

[Bug Sprint]

Confirming that the referenced code fails on MSVC9 SP1 with boost trunk and that the attached patch fixes the problem. I spent some time trying to boil the test code down to a fusion-only test case but failed.

comment:4 Changed 10 years ago by anonymous

Resolution: fixed
Status: newclosed

comment:5 Changed 9 years ago by rwebb <richard.webb@…>

Keywords: VC10 added; VC9 removed
Milestone: Boost 1.36.0Boost 1.43.0
Resolution: fixed
Status: closedreopened
Summary: vector workaround needed for MSVC9vector workaround needed for MSVC10

According to the post @, this same issue effects the RTM version of VC10.

Fix is to alter the workarounds mentioned in the VC9 patch to check BOOST_MSVC against 1600 instead of 1500.

comment:6 Changed 9 years ago by Joel de Guzman

Resolution: fixed
Status: reopenedclosed

(In [61487]) fix #1614

comment:7 Changed 9 years ago by Joel de Guzman

(In [61488]) fix #1614

Note: See TracTickets for help on using tickets.