----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105065/#review16292 -----------------------------------------------------------
This review has been submitted with commit 9a528c92781a39882df24ab2a4c1282d222adca2 by Vishesh Handa to branch KDE/4.9. - Commit Hook On July 24, 2012, 6:36 a.m., Vishesh Handa wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/105065/ > ----------------------------------------------------------- > > (Updated July 24, 2012, 6:36 a.m.) > > > Review request for Nepomuk and Sebastian Trueg. > > > Description > ------- > > If people want them, they should enable them. There is no advantage of > providing them by default. Specially since no one seems to be using them. It > also complicates the queries when you're trying to debug stuff. > > Also, this breaks the existing API a little bit. However, since no one has > packaged Nepomuk2 so far, I'm allowed to break stuff. > > > Diffs > ----- > > libnepomukcore/query/query.h f4360b0 > libnepomukcore/query/query.cpp d67b6ce > libnepomukcore/query/queryserializer.cpp 0d8bc7e > libnepomukcore/test/queryserializertest.cpp ccc8af6 > libnepomukcore/test/querytest.cpp 2a8f685 > > Diff: http://git.reviewboard.kde.org/r/105065/diff/ > > > Testing > ------- > > Not tested. This also breaks almost all the tests in querytest. I'm waiting > to see if trueg is okay with this patch. > > > Thanks, > > Vishesh Handa > >
_______________________________________________ Nepomuk mailing list [email protected] https://mail.kde.org/mailman/listinfo/nepomuk
