> On Nov. 7, 2015, 2:03 p.m., Sebastian Kügler wrote: > > src/kpackage/private/packagejobthread_p.h, line 58 > > <https://git.reviewboard.kde.org/r/125976/diff/3/?file=415594#file415594line58> > > > > As you note yourself, perhaps just make it an enum? It's not private > > API, OK, but the code here is getting really convoluted that I think > > another boolean trap won't help...
if the enum stays in the private packagejobthread class it stays private enough. as user facing api i would keep separate methods, like install, update and eventual new ones like updateUnconditionally or whatever - Marco ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125976/#review88138 ----------------------------------------------------------- On Nov. 6, 2015, 5:57 p.m., Marco Martin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/125976/ > ----------------------------------------------------------- > > (Updated Nov. 6, 2015, 5:57 p.m.) > > > Review request for KDE Frameworks, Plasma and Kai Uwe Broulik. > > > Repository: kpackage > > > Description > ------- > > new job based update() function that compared to install() > if a package with the same pluginId is already installed, > removes the old one before installing the new one, if > and only if the version of the new one is more recent > > > Diffs > ----- > > autotests/plasmoidpackagetest.h f730dce > autotests/plasmoidpackagetest.cpp cccc567 > src/kpackage/CMakeLists.txt 3696f37 > src/kpackage/package.h 4ada8da > src/kpackage/package.cpp 539b21a > src/kpackage/packagestructure.h 9427b42 > src/kpackage/packagestructure.cpp 0070514 > src/kpackage/private/packagejob.cpp 0d2241b > src/kpackage/private/packagejob_p.h 267429f > src/kpackage/private/packagejobthread.cpp ca523b3 > src/kpackage/private/packagejobthread_p.h bf8a266 > src/kpackage/private/versionparser.cpp PRE-CREATION > src/kpackagetool/CMakeLists.txt 78e0fb0 > > Diff: https://git.reviewboard.kde.org/r/125976/diff/ > > > Testing > ------- > > covered by autotests > > > Thanks, > > Marco Martin > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel