Help the compiler optimizing loops
authorLuís Pereira <luis.artur.pereira@gmail.com>
Thu, 5 Jan 2017 20:11:08 +0000 (20:11 +0000)
committerLuís Pereira <luis.artur.pereira@gmail.com>
Thu, 5 Jan 2017 20:12:56 +0000 (20:12 +0000)
Make the compiler aware that the loop condition is const and only needs to
be evaluated one time.

lxqtautostartentry.cpp
lxqtgridlayout.cpp
lxqtnotification.cpp

index 3fe0056..379f297 100644 (file)
@@ -38,7 +38,8 @@ AutostartEntry::AutostartEntry() :
 AutostartEntry::AutostartEntry(const QString& name):
     mLocalState(StateNone), mSystem(false)
 {
-    foreach (const QString& dir, XdgDirs::autostartDirs())
+    const QStringList& dirs = XdgDirs::autostartDirs();
+    foreach (const QString& dir, dirs)
     {
         const QString path = QString("%1/%2").arg(dir, name);
         if (QFile(path).exists())
index 94861fc..739178e 100644 (file)
@@ -131,7 +131,8 @@ void GridLayoutPrivate::updateCache()
     mCellMaxSize = QSize(0, 0);
     mVisibleCount = 0;
 
-    for (int i=0; i<mItems.count(); ++i)
+    const int N = mItems.count();
+    for (int i=0; i < N; ++i)
     {
         QLayoutItem *item = mItems.at(i);
         if (!item->widget() || item->widget()->isHidden())
index d26f25f..1dc3935 100644 (file)
@@ -158,7 +158,8 @@ void NotificationPrivate::setActions(QStringList actions, int defaultAction)
 {
     mActions.clear();
     mDefaultAction = defaultAction;
-    for (int ix = 0; ix < actions.size(); ix++)
+    const int N = actions.size();
+    for (int ix = 0; ix < N; ix++)
     {
         if (ix == defaultAction)
             mActions.append("default");