Review Request 123146: calligra sheets: do not use VerPattern twice for cell pattern
Thorsten Zachmann
t.zachmann at zagge.de
Fri Mar 27 04:11:21 GMT 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123146/#review78094
-----------------------------------------------------------
Ship it!
The patch looks good. Maybe change the order so that the solid one in just before the no brush pattern.
- Thorsten Zachmann
On March 27, 2015, 3:27 a.m., Boris Egorov wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/123146/
> -----------------------------------------------------------
>
> (Updated March 27, 2015, 3:27 a.m.)
>
>
> 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/c52ebf22/attachment.htm>
More information about the calligra-devel
mailing list