Review Request 112263: EBN: Check for foreach loop issues.

Commit Hook null at kde.org
Mon Aug 26 10:11:50 UTC 2013


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

(Updated Aug. 26, 2013, 10:11 a.m.)


Status
------

This change has been marked as submitted.


Review request for Amarok.


Description
-------

More janitor work.

EBN: Check for foreach loop issues.

Fixing a number of issues from
http://www.englishbreakfastnetwork.org/krazy/reports/extragear/multimedia/amarok/index.html
[...]
13. Check for foreach loop issues [foreach]...
[...]

Calling values() isn't necessary here. foreach will iterate over the values by default.


Diffs
-----

  src/browsers/playlistbrowser/PlaylistBrowserCategory.cpp 4d2352d 
  src/context/applets/upcomingevents/UpcomingEventsStack.cpp dd9e149 
  src/core-impl/collections/db/sql/SqlScanResultProcessor.cpp f19ff4d 
  src/core-impl/collections/support/MemoryMatcher.cpp 6e94647 
  src/core-impl/podcasts/sql/SqlPodcastProvider.cpp 793358b 

Diff: http://git.reviewboard.kde.org/r/112263/diff/


Testing
-------

Unit tests still pass.


Thanks,

Frank Meerkoetter

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/amarok-devel/attachments/20130826/099fdc25/attachment-0001.html>


More information about the Amarok-devel mailing list