Review Request: more little things in libs
Thorsten Zachmann
t.zachmann at zagge.de
Sun Aug 21 05:07:45 BST 2011
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/102385/#review5863
-----------------------------------------------------------
Ship it!
After fixing the comments please push. Nice work. Thanks
libs/koproperty/editors/spinbox.cpp
<http://git.reviewboard.kde.org/r/102385/#comment5184>
should be moved into the if (se) block above
libs/koproperty/editors/spinbox.cpp
<http://git.reviewboard.kde.org/r/102385/#comment5185>
Should be moved in the if (le) block above
- Thorsten
On Aug. 20, 2011, 4 p.m., Jaime Torres Amate wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/102385/
> -----------------------------------------------------------
>
> (Updated Aug. 20, 2011, 4 p.m.)
>
>
> Review request for Calligra.
>
>
> Summary
> -------
>
> Use references por QRectF and QString.
>
> One //FIXME because I can not see where the error variable gets a value.
> One //TODO to avoid code duplication.
> safeguards for possible null pointer dereferences.
> Remove some unused variables.
> Remove a double break;
> Reduce the scope of variables.
>
> Create a destructor to avoid a memory leak in FrameIterator.
>
>
> Diffs
> -----
>
> libs/flake/KoShapeFactoryBase.cpp fbbab41
> libs/kopageapp/KoPACanvasBase.h 85b82e7
> libs/kopageapp/KoPACanvasBase.cpp 4ee4bd3
> libs/koproperty/editors/spinbox.cpp fbe49d5
> libs/koreport/renderer/KoReportPreRenderer.cpp ce809c4
> libs/koreport/wrtembed/KoReportDesignerItemRectBase.cpp 7de1ed6
> libs/kotext/opendocument/KoTextLoader.cpp ff1c793
> libs/kotext/opendocument/KoTextWriter.cpp 01cb31f
> libs/odf/KoXmlNS.h 41a1658
> libs/odf/KoXmlNS.cpp 65a6316
> libs/pigment/KoColorConversionSystem_p.h d87b05a
> libs/pigment/KoColorConversions.cpp 5c3f708
> libs/textlayout/FrameIterator.h 56138ae
> libs/textlayout/FrameIterator.cpp e7dd615
> libs/textlayout/KoTextShapeContainerModel.cpp 80ac07b
> libs/widgets/KoUniColorChooser.cpp 68a9e60
>
> Diff: http://git.reviewboard.kde.org/r/102385/diff
>
>
> Testing
> -------
>
> No differences in make tests
>
>
> Thanks,
>
> Jaime Torres
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20110821/91c546d5/attachment.htm>
More information about the calligra-devel
mailing list