[SCM] ktp-text-ui packaging branch, master, updated. debian/15.12.1-1-1918-gdf4b0ec
Maximiliano Curia
maxy at moszumanska.debian.org
Sat May 28 00:22:44 UTC 2016
Gitweb-URL: http://git.debian.org/?p=pkg-kde/applications/ktp-text-ui.git;a=commitdiff;h=d541544
The following commit has been merged in the master branch:
commit d5415444b8d32833893cd34d1538e8786d4d0c64
Author: Daniele E. Domenichelli <daniele.domenichelli at gmail.com>
Date: Fri Sep 21 10:27:12 2012 +0200
Use join instead of concatenating the string manually
---
lib/message.cpp | 13 ++++++-------
1 file changed, 6 insertions(+), 7 deletions(-)
diff --git a/lib/message.cpp b/lib/message.cpp
index d832011..451143c 100644
--- a/lib/message.cpp
+++ b/lib/message.cpp
@@ -55,7 +55,11 @@ void Message::appendMessagePart(const QString& part)
void Message::appendScript(const QString& script)
{
- m_scripts << script;
+ // Append the script only if it is not already appended to avoid multiple
+ // execution of the scripts.
+ if (!m_scripts.contains(script)) {
+ m_scripts << script;
+ }
}
QString Message::finalizedMessage() const
@@ -73,13 +77,8 @@ QString Message::finalizedScript() const
return QString();
}
- QString finalScript;
+ QString finalScript = m_scripts.join(QLatin1String(""));
- Q_FOREACH(const QString &script, m_scripts) {
- if (!finalScript.contains(script)) {
- finalScript.append(script);
- }
- }
if (!finalScript.isEmpty()) {
finalScript.append(QLatin1String("false;"));
}
--
ktp-text-ui packaging
More information about the pkg-kde-commits
mailing list