This is an automatically generated e-mail. To reply, visit: http://svn.reviewboard.kde.org/r/5431/

Making the methods non-private makes me think we should no longer have those methods be inlined as that would mean the implementation would be copied in the library of all users.
Can you please move the implementation to the cpp file?

trunk/koffice/libs/odf/KoGenStyle.h (Diff revision 1)
public:
398
        if (type == DefaultType) {
What is this part about?

- Thomas


On September 23rd, 2010, 2:31 p.m., Marijn Kruisselbrink wrote:

Review request for KOffice.
By Marijn Kruisselbrink.

Updated 2010-09-23 14:31:27

Description

This patch makes the attribute() and property() methods in KoGenStyle public instead of private. The main use-case for this class is of course only to set properies and attributes, then insert styles in a KoGenStyles and forget about the actual contents of the style, but sometimes when creating odf files it is useful to be able to later lookup properties of earlier generated styles, without having to double-store that information.
For example in the xlsx filter this is used because the cell style for merged cells is stored in one cell style in odf, but in the xlsx file it is a combination of properties from the cell styles for all cells that are merged. With this change I can simply merge the cell styles when I know that cells are merged, without having to keep the cell-styles around in another format than the KoGenStyle that already contains all relevant information anyway.

Diffs

  • trunk/koffice/libs/odf/KoGenStyle.h (1178191)

View Diff