This is the mail archive of the
libstdc++@gcc.gnu.org
mailing list for the libstdc++ project.
Re: [libstdc++] Add C++17clamp
- From: NightStrike <nightstrike at gmail dot com>
- To: Ed Smith-Rowland <3dw4rd at verizon dot net>
- Cc: "libstdc++ at gcc dot gnu dot org" <libstdc++ at gcc dot gnu dot org>, gcc-patches <gcc-patches at gcc dot gnu dot org>, Jonathan Wakely <jwakely at redhat dot com>
- Date: Thu, 21 Jul 2016 19:38:00 -0400
- Subject: Re: [libstdc++] Add C++17clamp
- Authentication-results: sourceware.org; auth=none
- References: <57882528.8010501@verizon.net>
On Thu, Jul 14, 2016 at 7:50 PM, Ed Smith-Rowland <3dw4rd@verizon.net> wrote:
> Here is an implementation of P0025
> An algorithm to "clamp" a value between a pair of boundary values.
>
> Testing is almost finished - looks good so far.
>
> OK if testing passes?
>
> I didn't see a feature test in any of the SD-6 papers or P0025.
>
This is not an efficient implementation. See here:
https://gcc.gnu.org/ml/gcc-help/2014-10/msg00112.html
Which I derived from this SO answer (which is sadly not the accepted
answer at this time):
http://stackoverflow.com/a/16659263
I suggest using the very efficient method that requires a temporary.