----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129182/#review100012 -----------------------------------------------------------
Fix it, then Ship it! src/engine/transaction.cpp (line 458) <https://git.reviewboard.kde.org/r/129182/#comment67173> no space before the coma - Aleix Pol Gonzalez On Oct. 14, 2016, 11:11 p.m., Christian Ehrlicher wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/129182/ > ----------------------------------------------------------- > > (Updated Oct. 14, 2016, 11:11 p.m.) > > > Review request for Baloo. > > > Repository: baloo > > > Description > ------- > > While trying to figure out why indexing my files is so slow, I compiled baloo > on my system with clazy and found some unnecessary copies in foreach loops. > It's not related to my problem and not really a performance critical part but > imo should be fixed anyway > > > Diffs > ----- > > src/engine/transaction.cpp 908a81f > src/file/fileindexerconfig.cpp ccb5d80 > src/file/pendingfilequeue.cpp 489f100 > tests/file/storagedevicestest.cpp f269c81 > > Diff: https://git.reviewboard.kde.org/r/129182/diff/ > > > Testing > ------- > > Compiles fine > > > Thanks, > > Christian Ehrlicher > >