|
Boost-Maint : |
Subject: Re: [Boost-maint] [date_time] Fix Boost.Sync tests
From: Daniel James (dnljms_at_[hidden])
Date: 2014-05-31 10:13:17
On 31 May 2014 14:54, Andrey Semashev <andrey.semashev_at_[hidden]> wrote:
> On Saturday 31 May 2014 14:36:40 Daniel James wrote:
>> On 31 May 2014 14:25, Andrey Semashev <andrey.semashev_at_[hidden]> wrote:
>> > As a side note, the mentioned commit also removes to_time_t() from
>> > boost/date_time/posix_time/conversion.hpp. Is that intentional? Seems like
>> > a useful function to me.
>> >
>> > It also removes a few tests from test/posix_time/testtime.cpp.
>>
>> Looks like it accidentally reverts some of the changes from before the
>> merge point.
>
> Should I prepare a request to restore these reverted parts? Can I have two
> pending pull requests?
Yes, you just need to create a new branch in your repo. But I merged
your pull request so you don't have to, it's trivial enough that I
don't think anyone would object, and it does look like it was reverted
accidentally.
Boost-Maint list run by bdawes at acm dot org