Subject: Re: [boost] [utility] prior(it, n) for n being an unsigned type
From: Adam Romanek (romanek.adam_at_[hidden])
Date: 2014-06-24 02:34:46
On 24.06.2014 08:30, Andrey Semashev wrote:
> On Tue, Jun 24, 2014 at 10:21 AM, Adam Romanek <romanek.adam_at_[hidden]> wrote:
>> On 23.06.2014 23:21, Andrey Semashev wrote:
>>> I've created a pull request:
>>> After I read that old discussion I decided to go with a slightly more
>>> complicated solution than I originally intended.
>> It became a bit complicated, for such a "simple" utility ;) But I'm glad
>> it's been fixed. Thanks!
> Not yet. I'll apply the pull request if noone objects in a few days.
I meant I'm glad there's some movement here after all these years. Hope
to see it merged.
>> Just out of curiosity - why haven't you chosen the std::reverse_iterator
>> -based approach described in the old thread?
> Dave Abrahams (the original author) stated that he intended this
> utility to be also usable with integers. You can see that in the
> discussion . I felt obligated to fulfill that requirement, and
> using std::advance and reverse_iterator contradicted that. I could
> have used reverse_iterator in the iterator-related part but, well,
> deducing the signed integer type looked more straightforward to me.
>  http://lists.boost.org/Archives/boost/2004/02/61583.php
Oh, I somehow forgot about that. Thanks for pointing this out.
Boost list run by bdawes at acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk