Subject: Re: [boost] [release][poly_collection] Permission to merge to release
From: Daniel James (dnljms_at_[hidden])
Date: 2017-11-22 11:23:04


On 22 November 2017 at 10:23, Joaquin M López Muñoz via Boost
<boost_at_[hidden]> wrote:
> Hello,
>
> I'm asking for permission to merge the following commit chain into release:
>
> https://github.com/boostorg/poly_collection/commit/3609895bd01e5add5971345a3e7d2d220e654020
> https://github.com/boostorg/poly_collection/commit/18b7d88df360001c2c006f021136a95bceacd29b
> https://github.com/boostorg/poly_collection/commit/ce7e7aac60747c6fad5e97d2cbc22f814b14d886
> https://github.com/boostorg/poly_collection/commit/ab32df47f8811dc7f136a166f3b957a0325fde60
> https://github.com/boostorg/poly_collection/commit/df754527917b7be013ce558f8c0286aa7a0cc5b9
>
> Fixes bring compatibility with Clang 3.3 and solve a potential UB problem
> related to CWG 232 and
> detected at
>
> http://www.boost.org/development/tests/develop/developer/Marshall-UBSAN-poly_collection-clang-darwin-darwin-ubsan-test_construction-variants_.html
>
> I'm waiting for Marshall-UBSAN to cycle through these but all recently
> updated test runners are
> working fine. Clang 3.3 is already verified to work by
> teeks99-02-dc3.3-11-Docker-64on64.

The test results for poly_collection aren't working at the moment, so
I'll take your word for it. Go ahead.