Re: [boost] Master branch open for merging

Subject: Re: [boost] Master branch open for merging
From: Marshall Clow (mclow.lists_at_[hidden])
Date: 2019-03-28 18:18:07

On Thu, Mar 28, 2019 at 10:48 AM Raffi Enficiaud via Boost <
boost_at_[hidden]> wrote:

> On 25.03.19 14:48, Marshall Clow via Boost wrote:
> > On Sun, Mar 24, 2019 at 10:40 PM Raffi Enficiaud via Boost <
> > boost_at_[hidden]> wrote:
> >
> >> On 21.03.19 14:27, Marshall Clow via Boost wrote:
> >>> On Wed, Mar 20, 2019 at 11:07 PM Raffi Enficiaud via Boost <
> >>>>
> >>>> Thanks, I would like to merge this additional changes/fixes. The diff
> is
> >>>> visible here:
> >>>>
> >>>>
> >>>>
> >>
> https://github.com/boostorg/test/compare/master..b9d74d1f55be2d70cdaee9bdb3ebe9445ce5c7b3
> >>>>
> >>>> Changes:
> >>>> - Windows.h -> windows.h
> >>>> - comment unused variable and remove dead code
> >>>> - the rest is about trying to fix a test that should not be run on
> >>>> GCC4.8 and causing noise on Travis.
> >>>>
> >>>> It will be certainly another permission to merge later this week, only
> >>>> about documentation.
> >>>>
> >>
> >> I would like to merge the following to master:
> >>
> >> *
> >>
> >>
> https://github.com/boostorg/test/commit/6a8773f62693877cfc9ac19df9990445a4ff3616
> >> : documentation changes only
> >>
> >> *
> >>
> >>
> https://github.com/boostorg/test/commit/9ffe07ebd0fc1684b5f0fa8890872c91952beedd
> >> : change in a cast
> >>
> >> Tests are green on my CI on develop.
> >>
>
> Thanks!
> Now ... :) I would like to merge this to master:
>
>
> https://github.com/boostorg/test/commit/ec3aeac46b7f3ec287e1655c63cb553145048b0b
>
> This is to fix the warning issues mentioned in another thread. Tests
> green on my side (VS2008/2013/2015/2017, appleclang6 C++03/11, gcc5
> C++03/11, UBSAN gcc8/C++14, UBSAN clang). Warning seems to be solved
> until the next visibility issue comes up.
>

Approved.
-- Marshall