-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/109140/
-----------------------------------------------------------

Review request for Calligra.


Description
-------

SelectionRect::fixAspect recurses when the values of the rect don't exceed 
those of the borders. However, this can cause an infinite recursion when 
floating point errors occur, due to (for example) rect.top() being 
10.0000000001 which is larger than 10. This patch uses qFuzzyCompare to check 
for minor floating point errors before recursing.


Diffs
-----

  plugins/pictureshape/SelectionRect.cpp c3a9a2d 

Diff: http://git.reviewboard.kde.org/r/109140/diff/


Testing
-------

Compiled and run. Cropping on image rect still works.


Thanks,

Arjen Hiemstra

_______________________________________________
calligra-devel mailing list
calligra-devel@kde.org
https://mail.kde.org/mailman/listinfo/calligra-devel

Reply via email to