[Kdenlive-devel] [PATCH 04/46] docclipbase.cpp: Use char for single character

Mikko Rapeli mikko.rapeli at iki.fi
Sat Jul 14 07:55:51 UTC 2012


Should be faster than using a full string. Found by krazy.
---
 src/docclipbase.cpp |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/docclipbase.cpp b/src/docclipbase.cpp
index be5a321..4694e17 100644
--- a/src/docclipbase.cpp
+++ b/src/docclipbase.cpp
@@ -66,7 +66,7 @@ DocClipBase::DocClipBase(ClipManager *clipManager, QDomElement xml, const QStrin
     }
 
     if (xml.hasAttribute("cutzones")) {
-        QStringList cuts = xml.attribute("cutzones").split(";", QString::SkipEmptyParts);
+        QStringList cuts = xml.attribute("cutzones").split(';', QString::SkipEmptyParts);
         for (int i = 0; i < cuts.count(); i++) {
             QString z = cuts.at(i);
             addCutZone(z.section('-', 0, 0).toInt(), z.section('-', 1, 1).toInt(), z.section('-', 2, 2));
@@ -243,7 +243,7 @@ QDomElement DocClipBase::toXML(bool hideTemporaryProperties) const
     QMapIterator<QString, QString> i(m_properties);
     while (i.hasNext()) {
         i.next();
-        if (hideTemporaryProperties && i.key().startsWith("_")) continue;
+        if (hideTemporaryProperties && i.key().startsWith('_')) continue;
         if (!i.value().isEmpty()) clip.setAttribute(i.key(), i.value());
     }
     doc.appendChild(clip);
-- 
1.7.10.4





More information about the Kdenlive mailing list