Should be faster than using a full string. Found by krazy.
---
src/clipitem.cpp | 24 ++++++++++++------------
1 file changed, 12 insertions(+), 12 deletions(-)
diff --git a/src/clipitem.cpp b/src/clipitem.cpp
index e50ba6e..281953a 100644
--- a/src/clipitem.cpp
+++ b/src/clipitem.cpp
@@ -343,7 +343,7 @@ const QString ClipItem::adjustKeyframes(QString keyframes,
int offset)
const QStringList list = keyframes.split(';', QString::SkipEmptyParts);
foreach(const QString &keyframe, list) {
int pos = keyframe.section(':', 0, 0).toInt() - offset;
- QString newKey = QString::number(pos) + ":" + keyframe.section(':', 1);
+ QString newKey = QString::number(pos) + ':' + keyframe.section(':', 1);
result.append(newKey);
}
return result.join(";");
@@ -823,7 +823,7 @@ void ClipItem::paint(QPainter *painter,
else {
#if KDE_IS_VERSION(4,5,0)
if (m_clip && m_clip->thumbProducer()) {
- QString path = m_clip->fileURL().path() + "_";
+ QString path = m_clip->fileURL().path() + '_';
QImage img;
QPen pen(Qt::white);
pen.setStyle(Qt::DotLine);
@@ -1539,14 +1539,14 @@ EffectsParameterList ClipItem::addEffect(QDomElement
effect, bool /*animate*/)
needInOutSync = true;
}
if (e.attribute("type") == "simplekeyframe") {
- QStringList values = e.attribute("keyframes").split(";",
QString::SkipEmptyParts);
+ QStringList values = e.attribute("keyframes").split(';',
QString::SkipEmptyParts);
double factor = locale.toDouble(e.attribute("factor", "1"));
double offset = e.attribute("offset", "0").toDouble();
if (factor != 1 || offset != 0) {
for (int j = 0; j < values.count(); j++) {
QString pos = values.at(j).section(':', 0, 0);
double val =
(locale.toDouble(values.at(j).section(':', 1, 1)) - offset) / factor;
- values[j] = pos + "=" + locale.toString(val);
+ values[j] = pos + '=' + locale.toString(val);
}
}
parameters.addParam(e.attribute("name"), values.join(";"));
@@ -1797,18 +1797,18 @@ void ClipItem::insertKeyframe(QDomElement effect, int
pos, int val)
newkfr.append(str);
} else if (!added) {
if (i == m_visibleParam)
- newkfr.append(QString::number(pos) + ":" +
QString::number(val));
+ newkfr.append(QString::number(pos) + ':' +
QString::number(val));
else
- newkfr.append(QString::number(pos) + ":" +
locale.toString(newval));
+ newkfr.append(QString::number(pos) + ':' +
locale.toString(newval));
if (kpos > pos) newkfr.append(str);
added = true;
} else newkfr.append(str);
}
if (!added) {
if (i == m_visibleParam)
- newkfr.append(QString::number(pos) + ":" +
QString::number(val));
+ newkfr.append(QString::number(pos) + ':' +
QString::number(val));
else
- newkfr.append(QString::number(pos) + ":" +
e.attribute("default"));
+ newkfr.append(QString::number(pos) + ':' +
e.attribute("default"));
}
e.setAttribute("keyframes", newkfr.join(";"));
}
@@ -1836,9 +1836,9 @@ void ClipItem::movedKeyframe(QDomElement effect, int
oldpos, int newpos, double
newpos = qMax(newpos, start);
newpos = qMin(newpos, end);
if (i == m_visibleParam)
- newkfr.append(QString::number(newpos) + ":" +
locale.toString(value));
+ newkfr.append(QString::number(newpos) + ':' +
locale.toString(value));
else
- newkfr.append(QString::number(newpos) + ":" +
str.section(':', 1, 1));
+ newkfr.append(QString::number(newpos) + ':' +
str.section(':', 1, 1));
}
}
e.setAttribute("keyframes", newkfr.join(";"));
@@ -2059,9 +2059,9 @@ void ClipItem::updateGeometryKeyframes(QDomElement
effect, int paramIndex, int w
if (keyframe.contains('=')) {
int pos = keyframe.section('=', 0, 0).toInt();
pos += offset;
- data.append(QString::number(pos) + "=" + keyframe.section('=',
1) + ";");
+ data.append(QString::number(pos) + '=' + keyframe.section('=',
1) + ";");
}
- else data.append(keyframe + ";");
+ else data.append(keyframe + ';');
}
}
Mlt::Geometry geometry(data.toUtf8().data(),
oldInfo.cropDuration.frames(m_fps), width, height);
--
1.7.10.4
------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and
threat landscape has changed and how IT managers can respond. Discussions
will include endpoint security, mobile security and the latest in malware
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
Kdenlive-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/kdenlive-devel
