> On May 7, 2015, 7:06 a.m., Milian Wolff wrote: > > src/engine/postingdb.cpp, line 380 > > <https://git.reviewboard.kde.org/r/123662/diff/3/?file=367309#file367309line380> > > > > auto, tyke comVal and com by value > > Pinak Ahuja wrote: > Same as above. > > Pinak Ahuja wrote: > ok got passing integers by value is faster, but what about comVal, its a > QByteArray?
Ok, I should probably stop flooding the reviewboard, but I think i got that too, is it because comVal is a reference to a QByteArray and taking references as values is faster? also I assume should be taking the reference to RegularExpression by value too? - Pinak ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123662/#review80012 ----------------------------------------------------------- On May 7, 2015, 4:53 a.m., Pinak Ahuja wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/123662/ > ----------------------------------------------------------- > > (Updated May 7, 2015, 4:53 a.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 <<