Web lists-archives.com

Re: Review Request 129182: CLazy fixes: use const ref in for loop instead object




This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129182/

Review request for Baloo.
By Christian Ehrlicher.

Updated Oct. 14, 2016, 9:11 p.m.

Changes

Use Q_FOREACH() on map directly instead create a temporary QList object with map.values()

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

Testing

Compiles fine

Diffs (updated)

  • src/engine/transaction.cpp (908a81f)
  • src/file/fileindexerconfig.cpp (ccb5d80)
  • src/file/pendingfilequeue.cpp (489f100)
  • tests/file/storagedevicestest.cpp (f269c81)

View Diff