Boost logo

Boost :

Subject: Re: [boost] [Review:Algorithms] Order of args to clamp
From: Marshall Clow (mclow.lists_at_[hidden])
Date: 2011-09-24 12:09:27

On Sep 24, 2011, at 7:31 AM, Olaf van der Spek wrote:

> On Fri, Sep 23, 2011 at 6:21 PM, Phil Endecott
> <spam_from_boost_dev_at_[hidden]> wrote:
>> template<typename V>
>> V clamp ( V val, V lo, V hi );
>> I have a clamp function that orders the args low - middle - high, which
>> seems like a more natural ordering to me. Is there some rationale or
>> precedent for this middle - low - high ordering? I think the confusion
> lo and hi will often be short literals, while val might be longer.
> Having lo and hi close together seems like a good idea.
> If val is really long, it might even be better to have lo and hi
> before val, as is sometimes seen with (if X == f()) where X is a
> constant.
>> I note that this function takes its args by value, while std::min & max take
>> const references. What is the rationale for this?
> Probably just an oversight.


I'm keeping track of issues at Github, and this one is there:

Feel free to add your own issues there (Brent has already done so).

-- 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

Boost list run by bdawes at, gregod at, cpdaniel at, john at