----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103044/#review7936 -----------------------------------------------------------
filters/tables/excel/import/ODrawClient.cpp <http://git.reviewboard.kde.org/r/103044/#comment6830> 1. while we're at this, const is not needed here 2. how about making this method const? filters/tables/excel/import/ODrawClient.cpp <http://git.reviewboard.kde.org/r/103044/#comment6829> maybe isEmpty() could be better? filters/tables/excel/sidewinder/cell.h <http://git.reviewboard.kde.org/r/103044/#comment6831> BTW, someone will have to transform these comments to Doxygen comments... filters/tables/excel/sidewinder/excel.h <http://git.reviewboard.kde.org/r/103044/#comment6832> let's return values not references; difference in performance is minimal (or none in new compilers?) while it's far safer - Jarosław Staniek On Nov. 4, 2011, 9:16 a.m., Lukáš Tvrdý wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/103044/ > ----------------------------------------------------------- > > (Updated Nov. 4, 2011, 9:16 a.m.) > > > Review request for Calligra, Marijn Kruisselbrink and Sebastian Sauer. > > > Description > ------- > > This patch uses more of the libmso in excel filter. > > So far the pictures were handled as PictureObject class. One problem was that > there was no support for groups for this object. So when you put two images > into groups, they > are not displayed because the anchoring information is saved in different > data-structure - in the > group structure. > > Now I let the records to be handled by libmso, which provides this > functionality. > It supports groups and even graphical styles like like borders etc. > > > This addresses bug 262865. > http://bugs.kde.org/show_bug.cgi?id=262865 > > > Diffs > ----- > > filters/libmso/pictures.cpp f8bfa7c > filters/tables/excel/import/ExcelImport.cpp 1bb7f17 > filters/tables/excel/import/ODrawClient.cpp 5a6507b > filters/tables/excel/import/excelimporttoods.cc 287aef7 > filters/tables/excel/sidewinder/cell.h bc080f9 > filters/tables/excel/sidewinder/cell.cpp 8b8c0e1 > filters/tables/excel/sidewinder/excel.h 2be1f5a > filters/tables/excel/sidewinder/excel.cpp b475e5a > filters/tables/excel/sidewinder/globalssubstreamhandler.h 77c6e3a > filters/tables/excel/sidewinder/globalssubstreamhandler.cpp f9c03b1 > filters/tables/excel/sidewinder/objects.h 2412f07 > filters/tables/excel/sidewinder/sheet.h 9ff346d > filters/tables/excel/sidewinder/sheet.cpp 440f44b > filters/tables/excel/sidewinder/workbook.h 2e3c2fa > filters/tables/excel/sidewinder/workbook.cpp 9ad61bc > filters/tables/excel/sidewinder/worksheetsubstreamhandler.cpp 879f76f > > Diff: http://git.reviewboard.kde.org/r/103044/diff/diff > > > Testing > ------- > > I did regression testing against master. My testing set consists of 976 xls > files. > I found only positive changes. > > Missing frames around pictures are displayed now, missing images are > displayed now. > You can test the document. > > > Thanks, > > Lukáš Tvrdý > >
_______________________________________________ calligra-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/calligra-devel
