Review Request: more little things in libs
Commit Hook
null at kde.org
Sun Aug 21 11:16:42 BST 2011
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/102385/#review5873
-----------------------------------------------------------
This review has been submitted with commit 814902e5fde287e2e96a2b7eb8ab2453eb6da227 by Jaime Torres to branch master.
- Commit
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/fb9db0b0/attachment.htm>
More information about the calligra-devel
mailing list