Review Request: more little things in libs

Casper Boemann cbr at boemann.dk
Sat Aug 20 18:24:21 BST 2011


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


i'm fine with the text parts

- Casper


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/20110820/72e3692c/attachment.htm>


More information about the calligra-devel mailing list