[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    [kphotoalbum] Exif: Use const iterator boundaries with a const iterator
From:       Luigi Toscano <null () kde ! org>
Date:       2017-01-01 3:08:15
Message-ID: E1cNWVL-0002xB-Qk () code ! kde ! org
[Download RAW message or body]

Git commit f1a7fd309119e29cf573830d752a632fc5466cbd by Luigi Toscano.
Committed on 01/01/2017 at 03:08.
Pushed by ltoscano into branch 'master'.

Use const iterator boundaries with a const iterator

Summary:
It does not seem to compile otherwise, at least on my system (current
Debian testing).

Test Plan: It compiles

Reviewers: tleupold, johanneszarl

Reviewed By: johanneszarl

Differential Revision: https://phabricator.kde.org/D3894

M  +1    -1    Exif/TreeView.cpp

https://commits.kde.org/kphotoalbum/f1a7fd309119e29cf573830d752a632fc5466cbd

diff --git a/Exif/TreeView.cpp b/Exif/TreeView.cpp
index fec7e6d0..5837ed1b 100644
--- a/Exif/TreeView.cpp
+++ b/Exif/TreeView.cpp
@@ -71,7 +71,7 @@ void Exif::TreeView::reload()
 
     QMap<QString, QTreeWidgetItem*> tree;
 
-    for( QStringList::const_iterator keysIt = keys.begin(); keysIt != keys.end(); ++keysIt ) {
+    for( QStringList::const_iterator keysIt = keys.constBegin(); keysIt != keys.constEnd(); ++keysIt ) {
         QStringList subKeys = (*keysIt).split(QLatin1String("."));
         QTreeWidgetItem* parent = nullptr;
         QString path;
[prev in list] [next in list] [prev in thread] [next in thread]