> On March 12, 2015, 12:50 p.m., David Edmundson wrote: > > src/plasma/theme.cpp, line 424 > > <https://git.reviewboard.kde.org/r/122915/diff/1/?file=354535#file354535line424> > > > > Would it work to: > > > > QMetaObject::invoke(d->rectSaveTimer, SLOT("start()")); > > > > that will mean we always use the timer?
Seems so: does invokeMethod always call it from the own thread? - Marco ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122915/#review77358 ----------------------------------------------------------- On March 12, 2015, 10:34 a.m., Marco Martin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/122915/ > ----------------------------------------------------------- > > (Updated March 12, 2015, 10:34 a.m.) > > > Review request for Plasma. > > > Repository: plasma-framework > > > Description > ------- > > rectSaveTimer is used to event compress the save and sync of the > configuration file that is used as a cache of the rectangles of svg items (in > order to avoid svg renderers creation when possible) > there is exactly one situation where it seems to not work: during creation of > svgitems/framesvgitems, insertintorectcache ends up being called by another > thread than the Theme's qobject thread, and that is not allowed. so in this > case > > during normal svg resizes, all goes as before > > > Diffs > ----- > > src/plasma/theme.cpp 18d4ed6 > > Diff: https://git.reviewboard.kde.org/r/122915/diff/ > > > Testing > ------- > > > Thanks, > > Marco Martin > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel