Review Request 123146: calligra sheets: do not use VerPattern twice for cell pattern

Boris Egorov egorov at linux.com
Fri Mar 27 03:27:18 GMT 2015


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

Review request for Calligra.


Repository: calligra


Description
-------

Do not use VerPattern twice for cell pattern

There are two brush entries with VerPattern style used for cell format.
Change one of them to SolidPattern style (the only unused BrushStyle for
cell pattern). See image: http://i.imgur.com/JGtP5Ew.png

Issue detected by Cppcheck: "[sheets/dialogs/LayoutDialog.cpp:3456]:
Expression is always false because 'else if' condition matches previous
condition at line 3432."


Diffs
-----

  sheets/dialogs/LayoutDialog.cpp 48ec5e6 

Diff: https://git.reviewboard.kde.org/r/123146/diff/


Testing
-------

Build Calligra, run Sheets, try to use SolidPattern for cell pattern.


Thanks,

Boris Egorov

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20150327/4a6f4575/attachment.htm>


More information about the calligra-devel mailing list