Instead of calling the count() method every time, assign to a variable

It's just a little cleaner to read this way
tsimonq2/null-packages-fix
Simon Quigley 11 months ago
parent 8cebb36a9c
commit cae134166b

@ -45,7 +45,9 @@ void MainWindow::setUpdateInfo(QList<QStringList> updateInfo)
bool installEnabled = false; bool installEnabled = false;
for (int i = 0;i < 4;i++) { for (int i = 0;i < 4;i++) {
if (updateInfo[i].count() > 0) { int count = updateInfo[i].count();
if (count > 0) {
installEnabled = true; installEnabled = true;
} }
@ -65,7 +67,7 @@ void MainWindow::setUpdateInfo(QList<QStringList> updateInfo)
break; break;
} }
for (int j = 0;j < updateInfo[i].count();j++) { for (int j = 0;j < count;j++) {
// TODO: Add security update detection here - security updates should be marked in some way // TODO: Add security update detection here - security updates should be marked in some way
installItem->addChild(new QTreeWidgetItem(QStringList() << updateInfo[i][j])); installItem->addChild(new QTreeWidgetItem(QStringList() << updateInfo[i][j]));
} }

Loading…
Cancel
Save