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
>> 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 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 acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk