> On Feb. 20, 2014, 9:13 p.m., Martin Gräßlin wrote: > > src/declarativeimports/core/svgitem.cpp, line 127 > > <https://git.reviewboard.kde.org/r/115923/diff/1/?file=245131#file245131line127> > > > > nitpick: looking at the surrounding coding style the * should be moved > > from type to name > > David Edmundson wrote: > I'm really not sure which Martin is worse... > /me grumbles and fixes it. > > Martin Klapetek wrote: > There is a coding style defined and it's there so the resulting code, > shared with the whole world and worked on by many people, does not look like > it went through set of earthquakes :P > > David Edmundson wrote: > If I ever learn how to add astyle to my pre-commit hooks, you are going > to be out of a job.
if you do, please share :-) - Martin ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115923/#review50402 ----------------------------------------------------------- On Feb. 21, 2014, 12:45 a.m., David Edmundson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/115923/ > ----------------------------------------------------------- > > (Updated Feb. 21, 2014, 12:45 a.m.) > > > Review request for Plasma. > > > Repository: plasma-framework > > > Description > ------- > > The rationale behind this patch is on the mailing list in the thread "Minutes > Monday" > > This doesn't boost performance or save memory much, but it paves the way for > texture sharing, faster resizing, and plenty of other things. > > Based on Frederick's comment I have reverted my changes to use QImage > everywhere, otherwise we lose out on the local QPixmap cache in KImageCache. > Changes to plasmacore are minimal. > > I'm currently porting FrameSVG which is where we should see more gains, but I > thought I should get this reviewed/merged in parallel. > > I have only seen one regression which is in the analog clock. > Some odd code in the analog clock (by me apparently!) means the width is > dependent on the current width, which due to some changes in this patch ends > up in a constant spiral getting to infinitely sized and explode. > > > Changelog (in reverse order): > Remove manual isDirty tracking in SvgItem > Always resize the node geometry on resizes > Update to paint to fill the size of the object, not the size of texture > Fix leaking texture > Add convenient QImage image() getter in SVG > Avoid repainting if node is not changed > Render SvgItem natively rather than going through QQuickPaintedItem > > > Diffs > ----- > > src/declarativeimports/core/svgitem.cpp e90751a > src/plasma/svg.h 01d98f8 > src/plasma/svg.cpp 9ec2aa5 > src/declarativeimports/core/svgitem.h c8be7cc > > Diff: https://git.reviewboard.kde.org/r/115923/diff/ > > > Testing > ------- > > > Thanks, > > David Edmundson > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
