----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102562/#review6419 -----------------------------------------------------------
Ship it! Please changes the small additional thing I commented on, then it is good to commit. libs/odf/KoOdfStylesReader.cpp <http://git.reviewboard.kde.org/r/102562/#comment5706> Better use d->drawStyles.value(drawType); - Jan On Sept. 11, 2011, 4:49 a.m., Thorsten Zachmann wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/102562/ > ----------------------------------------------------------- > > (Updated Sept. 11, 2011, 4:49 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/odf/KoOdfStylesReader.h acab80f > libs/odf/KoOdfStylesReader.cpp 5a8324e > plugins/chartshape/Surface.cpp e22a6b2 > libs/flake/KoPatternBackground.cpp 4210ab48 > libs/odf/KoOdfGraphicStyles.cpp 9e9653f > > 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 > >
_______________________________________________ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.org/mailman/listinfo/calligra-devel
