Re: smalleset() ( Re: [eigen] Householder.h: ::min() and operator<=)

[ Thread Index | Date Index | More Archives ]

Dear Christoph,

Am 12.09.2016 um 10:57 schrieb Christoph Hertzberg:

If nobody objects against that, I'll patch this into the dev-branch. Especially, are there objections against or alternative suggestions for the naming? `lowest()` probably would be a bad choice, since this is what std calls what
we call `min()`.

The problem with std::min() is, that it has to be understood historically. The name is a bad choice,
since the behaviour is different for integer and floating point values .

> N.B.: The 1 must be T(1) here (in case T is not implicitly constructible from int). And perhaps we should document it as "lowest non-denormalized positive value".

Which would suggest lowest() as a name, which is however already used.

Best regards,

Mail converted by MHonArc 2.6.19+