https://bugs.kde.org/show_bug.cgi?id=500015

--- Comment #7 from Rok F <r...@faith.si> ---
(In reply to Markus from comment #5)
> Changing this:
>             // sinces nextMoveResizeGeom is fractional, at best it is within
> 1 unit of currentMoveResizeGeom
>             if (std::abs(currentMoveResizeGeom.left() -
> nextMoveResizeGeom.left()) < 1.0
>                 && std::abs(currentMoveResizeGeom.right() -
> nextMoveResizeGeom.right()) < 1.0
>                 && std::abs(currentMoveResizeGeom.top() -
> nextMoveResizeGeom.top()) < 1.0
>                 && std::abs(currentMoveResizeGeom.bottom() -
> nextMoveResizeGeom.bottom()) < 1.0) {
>                 break; // Prevent lockup
>             }
> to all "<= 1.0" is preventing the lock-up for me.
> Is the calculation flawed or this exit criteria?

Given that the comment says "within 1 unit" it seems that the author forgot to
allow for equality. Seems a reasonable asumption if the comment is correct.

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to