Review Request: Make it possible to get a all draw styles of a defined type.

[Jan Hambrecht]

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



libs/odf/KoOdfStylesReader.h
<http://git.reviewboard.kde.org/r/102562/#comment5675>

    I agree, most important is to mention which string is valid to pass, e.g. in most cases the style element name, but for gradients it is "gradient". It would probably even better to use an enum here?


- Jan


On Sept. 10, 2011, 6:28 a.m., Thorsten Zachmann wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/102562/
> -----------------------------------------------------------
> 
> (Updated Sept. 10, 2011, 6:28 a.m.)
> 
> 
> Review request for Calligra.
> 
> 
> Summary
> -------
> 
> This patch makes it possible to get all gradients, markers, ...  from the styles reader. It is needed when we don't want to load the same stuff over and over again. 
> I will use it in my marker branch to get all markers and load them in one go and just reuse them later.
> 
> There is no change in functionality for the extiting applications.
> 
> 
> Diffs
> -----
> 
>   libs/flake/KoPatternBackground.cpp 4210ab48 
>   libs/odf/KoOdfGraphicStyles.cpp 9e9653f 
>   libs/odf/KoOdfStylesReader.h acab80f 
>   libs/odf/KoOdfStylesReader.cpp 5a8324e 
>   plugins/chartshape/Surface.cpp e22a6b2 
> 
> Diff: http://git.reviewboard.kde.org/r/102562/diff
> 
> 
> Testing
> -------
> 
> Run cstester and no problems found in the documents of my test set (~1500 docs)
> 
> 
> Thanks,
> 
> Thorsten
> 
>

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