Modify

Ticket #11157 (closed Bugs: fixed)

Opened 2 years ago

Last modified 13 months ago

MSVC 8 warnings for BOOST_PP_IS_EMPTY in Boost.Phoenix because of Boost.Fusion

Reported by: damien.buhl@… Owned by: djowel
Milestone: Boost 1.61.0 Component: fusion
Version: Boost 1.58.0 Severity: Cosmetic
Keywords: fusion warnings msvc8 Cc: eldiener@…, andrey.semashev@…

Description

Boost.Fusion in it's new facility to deduce memeber's types for BOOST_FUSION_ADAPT* macros mase use of BOOST_PP_IS_EMPTY, which should be kept for private use by Boost.Preprocessor and the newcoming VMD library.

This macro works really fine when used in conjunction to BOOST_PP_EMPTY. However it generates warnings in VS 2005 MSVC 8 compiler.

These warnings could be removed in that BOOST_PP_IS_EMPTY would be removed from the fusion codebase in favor to something like BOOST_MPL_PP_TOKEN_EQUAL.

Attachments

Change History

comment:1 Changed 2 years ago by damien.buhl@…

  • Owner set to djowel
  • Component changed from None to fusion

comment:2 Changed 2 years ago by damien.buhl@…

I've made a fix for this in the branch feature/ticket-11157-remove-BOOST_PP_IS_EMPTY on my github fork (daminetreg).

I still need to adapt the documentation as I also added "auto" as keyword as sysnonym for BOOST_FUSION_ADAPT_AUTO as proposed by Andrey Semashev on the mailing list.

comment:3 Changed 21 months ago by damien.buhl@…

The fix is done, I'm currently setting up alot of MSVC versions aside to perform the testing, but on my github branch feature/ticket-11157-remove-BOOST_PP_IS_EMPTY it's all in and all test passes.

I'll make the pull-request as soon as I'm sure with older MSVCs.

comment:4 Changed 14 months ago by Kohei Takahashi <flast@…>

Can I get minimal reproducible code? I'm trying to catch up Damien's patches.

comment:5 Changed 13 months ago by Kohei Takahashi <flast@…>

  • Milestone changed from To Be Determined to Boost 1.61.0

Fixed:  https://github.com/boostorg/fusion/pull/124

Joel, could you close this ticket?

comment:6 Changed 13 months ago by djowel

  • Status changed from new to closed
  • Resolution set to fixed
View

Add a comment

Modify Ticket

Change Properties
<Author field>
Action
as closed
The resolution will be deleted. Next status will be 'reopened'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.