On Wed, 7 Nov 2018 at 03:40, Gerald Pfeifer gerald@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.