> On April 13, 2013, 2:12 a.m., Aaron J. Seigo wrote: > > assetimporters/projectgutenberg/src/gutenbergdatabase.cpp, lines 328-338 > > <http://git.reviewboard.kde.org/r/109965/diff/1/?file=138225#file138225line328> > > > > this can be done once and then the value cached
now m_partnerId takes the value of the partnerId(). So do we still need to cache the value? - Giorgos ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/109965/#review30976 ----------------------------------------------------------- On April 20, 2013, 10:52 a.m., Giorgos Tsiapaliokas wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/109965/ > ----------------------------------------------------------- > > (Updated April 20, 2013, 10:52 a.m.) > > > Review request for Plasma. > > > Description > ------- > > This patch > > * removes the duplicated code in assetimporters > * adds asset's size into the db > * and fixes a few small issues > > > Diffs > ----- > > assetimporters/CMakeLists.txt 24e76a0 > assetimporters/database-common/channelscatalog.h 5d39c02 > assetimporters/database-common/channelscatalog.cpp 6ca0aef > assetimporters/database-common/database.h 9883216 > assetimporters/database-common/database.cpp e860bdd > assetimporters/kdeartwork-wallpapers/CMakeLists.txt 56d19b9 > assetimporters/kdeartwork-wallpapers/database.h 6991758 > assetimporters/kdeartwork-wallpapers/database.cpp d75cdda > assetimporters/kdeartwork-wallpapers/kdewallpapersdatabase.h PRE-CREATION > assetimporters/kdeartwork-wallpapers/kdewallpapersdatabase.cpp PRE-CREATION > assetimporters/kdeartwork-wallpapers/main.cpp 708a949 > assetimporters/obs/CMakeLists.txt 2dbcd42 > assetimporters/obs/channelscatalog.h PRE-CREATION > assetimporters/obs/channelscatalog.cpp PRE-CREATION > assetimporters/obs/packagedatabase.h 99f4e17 > assetimporters/obs/packagedatabase.cpp ae43b8e > assetimporters/projectgutenberg/CMakeLists.txt b86cc49 > assetimporters/projectgutenberg/src/CMakeLists.txt 2d48e9c > assetimporters/projectgutenberg/src/database.h 8dba0ba > assetimporters/projectgutenberg/src/database.cpp 75cba69 > assetimporters/projectgutenberg/src/gutenbergdatabase.h PRE-CREATION > assetimporters/projectgutenberg/src/gutenbergdatabase.cpp PRE-CREATION > assetimporters/projectgutenberg/src/main.cpp 46f2340 > sql/bodega.sql 44f8641 > > Diff: http://git.reviewboard.kde.org/r/109965/diff/ > > > Testing > ------- > > I haven't noticed regression. > > > Thanks, > > Giorgos Tsiapaliokas > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
