----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105287/#review15680 -----------------------------------------------------------
Ship it! Except for the missing docs it looks good. libnepomukcore/query/resultiterator.h <http://git.reviewboard.kde.org/r/105287/#comment12202> Docs are missing. - Sebastian Trueg On July 11, 2012, 3:38 p.m., Vishesh Handa wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/105287/ > ----------------------------------------------------------- > > (Updated July 11, 2012, 3:38 p.m.) > > > Review request for Nepomuk. > > > Description > ------- > > Allow synchronous execution of queries > > This is done by returning a Nepomuk::Query::ResultIterator, and > allows the user to control the execution of the query with the > iterator. > > This lets us move away from Soprano. Plus, this avoids > code duplication as this code is being duplicated in both the > query service and the kioslave. > > > Diffs > ----- > > includes/CMakeLists.txt 3e07f83 > includes/Query/ResultIterator PRE-CREATION > libnepomukcore/CMakeLists.txt 066c898 > libnepomukcore/query/resultiterator.h PRE-CREATION > libnepomukcore/query/resultiterator.cpp PRE-CREATION > > Diff: http://git.reviewboard.kde.org/r/105287/diff/ > > > Testing > ------- > > > Thanks, > > Vishesh Handa > >
_______________________________________________ Nepomuk mailing list [email protected] https://mail.kde.org/mailman/listinfo/nepomuk
