> On 2010-07-15 15:27:50, Aaron Seigo wrote: > > trunk/KDE/kdelibs/plasma/externalpluginloader.cpp, line 34 > > <http://reviewboard.kde.org/r/4554/diff/4/?file=31229#file31229line34> > > > > i wonder if we should guard this so it can only be set once, e.g.: > > > > if (!s_externalPluginLoader) { > > s_externalPluginLoader = loader; > > }
Submitted as of r1150391 - Ryan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/4554/#review6587 ----------------------------------------------------------- On 2010-07-14 22:43:15, Ryan Rix wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviewboard.kde.org/r/4554/ > ----------------------------------------------------------- > > (Updated 2010-07-14 22:43:15) > > > Review request for Plasma, Aaron Seigo, Robert Marmorstein, and Shaun Reich. > > > Summary > ------- > > Adds ability for applications which embed plasma to specify a class which > plasma will query for Applet, DataEngine, and Service creation, giving them > the chance to create these objects internally. > > The foremost use of this is to give Plasma the chance to ask applications > when restoring layouts from disk, for applets which were based on QWidgets, > such as the case in various Plasma dashboards which may embed QWidgets as > plasma::applets .... > > > Diffs > ----- > > trunk/KDE/kdelibs/plasma/CMakeLists.txt 1146180 > trunk/KDE/kdelibs/plasma/applet.cpp 1146180 > trunk/KDE/kdelibs/plasma/dataenginemanager.cpp 1146180 > trunk/KDE/kdelibs/plasma/externalpluginloader.h PRE-CREATION > trunk/KDE/kdelibs/plasma/externalpluginloader.cpp PRE-CREATION > trunk/KDE/kdelibs/plasma/service.cpp 1146180 > > Diff: http://reviewboard.kde.org/r/4554/diff > > > Testing > ------- > > Well, this thing is about 90% there right now, everything works, etc, afaict, > but I can't get the instance of the PluginManager outside of the Plasma > namespace, if someone could help that would be great. :) I can commit some > support into trunk/playground/base/shells/kpart > > > Thanks, > > Ryan > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
