Re: [boost] [release] 1.64.0 post-beta merges

Subject: Re: [boost] [release] 1.64.0 post-beta merges
From: Steven Watanabe (watanabesj_at_[hidden])
Date: 2017-04-10 14:51:16

AMDG

On 04/08/2017 08:35 PM, Rene Rivera via Boost wrote:
> 
> execution_context.cpp:
> -# if (defined(BOOST_EXECUTION_CONTEXT) && (BOOST_EXECUTION_CONTEXT == 1))
>  +# if (BOOST_EXECUTION_CONTEXT == 1)
> 
> I would consider that change dangerous. If users are not building with b2
> there is a high probability they will not define BOOST_EXECUTION_CONTEXT.
> Unless you define it to some default some place I can't immediately see in
> that commit.
> 

  Technically, the two are equivalent, but the
new version may generate warnings, depending
on compiler options.

> So I'd say the change is risky. And I would prefer if you find a way to
> make it safer.
> 

In Christ,
Steven Watanabe