Re: [boost] [Review:Algorithms] Order of args to clamp

Subject: Re: [boost] [Review:Algorithms] Order of args to clamp
From: Marshall Clow (mclow.lists_at_[hidden])
Date: 2011-09-26 13:17:54

On Sep 26, 2011, at 9:59 AM, Christian Holmquist wrote:

> On 26 September 2011 11:33, Dave Abrahams <dave_at_[hidden]> wrote:
> 
>> 
>> on Mon Sep 26 2011, Andrew Sutton <asutton.list-AT-gmail.com> wrote:
>> 
>>>> It seems to me that a clamp_range is just:
>>>> 
>>>>       std::transform ( begin, end, out, clamp ( _1, lo, hi ))
>>>> 
>>>> is that what you meant?
>>> 
>>> Sorry... this got lost in the thread. That looks like the right
>>> implementation, but it might be nice to provide an interface for it:
>>> 
>>> template<typename Iter, typename T>
>>> void clamp_range(Iter first, Iter last, cont T& high, const T& low)
>> 
>> IMO it clearly should be:
>> 
>> template<typename Iter>
>> void clamp_range(
>>    Iter first, Iter last,
>>     typename itertor_traits<Iter>::value_type const& high,
>>    typename itertor_traits<Iter>::value_type const& low)
>> 
> +1.
> Can you go suggest to the committee and change find as well? :)

if people think that it's worthwhile, we could make a boost::find like that.

-- Marshall

Marshall Clow     Idio Software   <mailto:mclow.lists_at_[hidden]>

A.D. 1517: Martin Luther nails his 95 Theses to the church door and is promptly moderated down to (-1, Flamebait).
        -- Yu Suzuki