remove pointless conversions to QString when converting to int

this doesn't change a lot, but is cleaner.

Change-Id: Ia89c399d6df7cc6264954cf309d2c971a5592d6b
Reviewed-by: Joerg Bornemann <joerg.bornemann@qt.io>
This commit is contained in:
Oswald Buddenhagen 2016-05-13 13:44:42 +02:00
parent e8bdca74b5
commit a7a3cd643a

View File

@ -478,9 +478,9 @@ ProStringList QMakeEvaluator::evaluateBuiltinExpand(
} else { } else {
var = args[0]; var = args[0];
sep = args.at(1).toQString(); sep = args.at(1).toQString();
beg = args.at(2).toQString(m_tmp2).toInt(); beg = args.at(2).toInt();
if (args.count() == 4) if (args.count() == 4)
end = args.at(3).toQString(m_tmp2).toInt(); end = args.at(3).toInt();
} }
} else { } else {
if (args.count() != 1) { if (args.count() != 1) {
@ -639,7 +639,7 @@ ProStringList QMakeEvaluator::evaluateBuiltinExpand(
const ProStringList &var = values(map(args.at(0))); const ProStringList &var = values(map(args.at(0)));
int start = 0, end = 0; int start = 0, end = 0;
if (args.count() >= 2) { if (args.count() >= 2) {
const QString &start_str = args.at(1).toQString(m_tmp1); const ProString &start_str = args.at(1);
start = start_str.toInt(&ok); start = start_str.toInt(&ok);
if (!ok) { if (!ok) {
if (args.count() == 2) { if (args.count() == 2) {
@ -652,11 +652,11 @@ ProStringList QMakeEvaluator::evaluateBuiltinExpand(
} }
if (!ok) if (!ok)
evalError(fL1S("member() argument 2 (start) '%2' invalid.") evalError(fL1S("member() argument 2 (start) '%2' invalid.")
.arg(start_str)); .arg(start_str.toQString(m_tmp1)));
} else { } else {
end = start; end = start;
if (args.count() == 3) if (args.count() == 3)
end = args.at(2).toQString(m_tmp1).toInt(&ok); end = args.at(2).toInt(&ok);
if (!ok) if (!ok)
evalError(fL1S("member() argument 3 (end) '%2' invalid.") evalError(fL1S("member() argument 3 (end) '%2' invalid.")
.arg(args.at(2).toQString(m_tmp1))); .arg(args.at(2).toQString(m_tmp1)));
@ -1292,7 +1292,7 @@ QMakeEvaluator::VisitReturn QMakeEvaluator::evaluateBuiltinConditional(
return ReturnFalse; return ReturnFalse;
} }
int cnt = values(map(args.at(0))).count(); int cnt = values(map(args.at(0))).count();
int val = args.at(1).toQString(m_tmp1).toInt(); int val = args.at(1).toInt();
if (args.count() == 3) { if (args.count() == 3) {
const ProString &comp = args.at(2); const ProString &comp = args.at(2);
if (comp == QLatin1String(">") || comp == QLatin1String("greaterThan")) { if (comp == QLatin1String(">") || comp == QLatin1String("greaterThan")) {