Re: [boost] [typeof][scope_exit] VS2017 and option /permissive-

Subject: Re: [boost] [typeof][scope_exit] VS2017 and option /permissive-
From: Peter Dimov (lists_at_[hidden])
Date: 2017-04-15 21:52:28

Daniela Engert wrote:
> I've reworked my pull-request by incorporating your suggestions with some 
> minor additional tweaks required to make it work. The tests of 
> Boost.Typeof pass with msvc10, msvc12, msvc14, and msvc14.1, and a test 
> run of the full Boost testsuite with msvc14.1 is successful as well.
> Given the generality of the suggested change, I'd appreciate if someone 
> would test it on gcc and clang, too.

Tests pass for me on Cygwin g++ 6.3.0 (03/11/14/1z), clang++ 3.9.1 
(03/11/14/1z), msvc-8.0, 10.0, 11.0, 12.0, 14.1.

> +#define BOOST_TYPEOF(expr) 
> boost::type_of::remove_cv_ref_t<decltype((boost::type_of::ensure_obj(expr)))>

The ensure_obj helper shouldn't be necessary here. The above tests also pass 
with

+#define BOOST_TYPEOF(expr) boost::type_of::remove_cv_ref_t<decltype(expr)>

and the definition of ensure_obj removed.

I've also enabled Travis for typeof, so any future pushes in the PR should 
be CI-tested.

Tests fail with msvc-14.1 /std:c++latest because of use of binder1st, 
auto_ptr, unary_function and so on, but that's a separate issue and 
unrelated to these changes, although we'll probably have to fix it at some 
point.