> On May 6, 2015, 6:52 p.m., Vishesh Handa wrote: > > src/engine/postingdb.cpp, line 381 > > <https://git.reviewboard.kde.org/r/123662/diff/2/?file=366918#file366918line381> > > > > Maybe in a seperarate patch we want to use QByteArray::fromRawData or > > something and avoid the copy. Though it might just be too trivial.
we would have to use something like QByteArray term = QByteArray::fromRawData(arr.data() + prefixLen, arr.length() - prefixLen); - Pinak ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123662/#review79979 ----------------------------------------------------------- On May 6, 2015, 6:28 p.m., Pinak Ahuja wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/123662/ > ----------------------------------------------------------- > > (Updated May 6, 2015, 6:28 p.m.) > > > Review request for Baloo and Vishesh Handa. > > > Repository: baloo > > > Description > ------- > > This removes code dupication within different kinds of iterators namely: > prefix,regex, comparator iterators. > > > Diffs > ----- > > src/engine/postingdb.h fd48602 > src/engine/postingdb.cpp 8c76a72 > > Diff: https://git.reviewboard.kde.org/r/123662/diff/ > > > Testing > ------- > > Compiles and tests pass. > > > Thanks, > > Pinak Ahuja > >
>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<