search

boostorg / mpl

Boost.org mpl module
http://boost.org/libs/mpl
Boost Software License 1.0
52 stars 78 forks source link

It normally should not be necessary to use the _Z form of the BOOST_P… #52

Closed eldiener closed 4 years ago

eldiener commented 4 years ago

…P_ENUM macro, although it is perfectly legal and helpful to do so, but in this case it is necessary as a workaround for a bug in the new VC++ standard conforming preprocessor. The bug manifests itself when testing the TTI library, which internally uses the Boost MPL code. The bug in the new VC++ standard conforming compiler is fixed in the VS2019 preview product, so that fix is sure to find its way to the official VS2019 product sometime soon. In the meantime this "fix" shortens the macro expansion somewhat and, while it should not be necessary, is still helpful.

See https://github.com/boostorg/preprocessor/issues/33 for a report and response to the VC++ new preprocessor bug.

codecov[bot] commented 4 years ago

Codecov Report

Merging #52 into develop will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop      #52   +/-   ##
========================================
  Coverage    73.33%   73.33%           
========================================
  Files            5        5           
  Lines           15       15           
  Branches         3        3           
========================================
  Hits            11       11           
  Misses           1        1           
  Partials         3        3

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 9c0f98f...4209712. Read the comment docs.