From 65753b5a2bca50bb9857feb41955af2748a5f27e Mon Sep 17 00:00:00 2001
From: Montel Laurent <montel@kde.org>
Date: Tue, 26 Apr 2016 13:19:41 +0200
Subject: [PATCH] Minor optimization

---
 .../src/htmlwriter/teehtmlwriter.cpp          | 21 ++++++++++++-------
 1 file changed, 14 insertions(+), 7 deletions(-)

diff --git a/messageviewer/src/htmlwriter/teehtmlwriter.cpp b/messageviewer/src/htmlwriter/teehtmlwriter.cpp
index d18d46f5..b32afbc8 100644
--- a/messageviewer/src/htmlwriter/teehtmlwriter.cpp
+++ b/messageviewer/src/htmlwriter/teehtmlwriter.cpp
@@ -61,49 +61,56 @@ void TeeHtmlWriter::addHtmlWriter(HtmlWriter *writer)
 
 void TeeHtmlWriter::begin(const QString &css)
 {
-    for (QList<HtmlWriter *>::Iterator it = mWriters.begin(); it != mWriters.end(); ++it) {
+    QList<HtmlWriter *>::const_iterator end(mWriters.constEnd());
+    for (QList<HtmlWriter *>::const_iterator it = mWriters.constBegin(); it != end; ++it) {
         (*it)->begin(css);
     }
 }
 
 void TeeHtmlWriter::end()
 {
-    for (QList<HtmlWriter *>::Iterator it = mWriters.begin(); it != mWriters.end(); ++it) {
+    QList<HtmlWriter *>::const_iterator end(mWriters.constEnd());
+    for (QList<HtmlWriter *>::const_iterator it = mWriters.constBegin(); it != end; ++it) {
         (*it)->end();
     }
 }
 
 void TeeHtmlWriter::reset()
 {
-    for (QList<HtmlWriter *>::Iterator it = mWriters.begin(); it != mWriters.end(); ++it) {
+    QList<HtmlWriter *>::const_iterator end(mWriters.constEnd());
+    for (QList<HtmlWriter *>::const_iterator it = mWriters.constBegin(); it != end; ++it) {
         (*it)->reset();
     }
 }
 
 void TeeHtmlWriter::write(const QString &str)
 {
-    for (QList<HtmlWriter *>::Iterator it = mWriters.begin(); it != mWriters.end(); ++it) {
+    QList<HtmlWriter *>::const_iterator end(mWriters.constEnd());
+    for (QList<HtmlWriter *>::const_iterator it = mWriters.constBegin(); it != end; ++it) {
         (*it)->write(str);
     }
 }
 
 void TeeHtmlWriter::queue(const QString &str)
 {
-    for (QList<HtmlWriter *>::Iterator it = mWriters.begin(); it != mWriters.end(); ++it) {
+    QList<HtmlWriter *>::const_iterator end(mWriters.constEnd());
+    for (QList<HtmlWriter *>::const_iterator it = mWriters.constBegin(); it != end; ++it) {
         (*it)->queue(str);
     }
 }
 
 void TeeHtmlWriter::flush()
 {
-    for (QList<HtmlWriter *>::Iterator it = mWriters.begin(); it != mWriters.end(); ++it) {
+    QList<HtmlWriter *>::const_iterator end(mWriters.constEnd());
+    for (QList<HtmlWriter *>::const_iterator it = mWriters.constBegin(); it != end; ++it) {
         (*it)->flush();
     }
 }
 
 void TeeHtmlWriter::embedPart(const QByteArray &contentId, const QString &url)
 {
-    for (QList<HtmlWriter *>::Iterator it = mWriters.begin(); it != mWriters.end(); ++it) {
+    QList<HtmlWriter *>::const_iterator end(mWriters.constEnd());
+    for (QList<HtmlWriter *>::const_iterator it = mWriters.constBegin(); it != end; ++it) {
         (*it)->embedPart(contentId, url);
     }
 }
-- 
GitLab