7 Nov
2018
7 Nov
'18
8:44 a.m.
On Wed, 7 Nov 2018 at 03:40, Gerald Pfeifer <gerald(a)pfeifer.com> wrote:
Thanks for looking into this and the explanation. Since we are in unsigned land, the abs() really is a noop, so at a minimum should be removed? I'll submit a patch in a minute.
No, there's a type conversion when passing the unsigned value to abs(). I.e., abs(~0u) == 1.