Johannes Altmanninger wrote:
Does anyone have time to review this? I think it's an evident bug. I can try to improve the clarity of the patch if needed.
It's not clarity that needs fixing, it's also correctness. A quick look suggests that the proposed fix can go into an infinite loop due to unsigned integer overflow. This is why the current code uses division and not multiplication.
