Fix clang-tidy errors

Blinkinlabs · Oct 8, 2024 · 59d5bf5 · 59d5bf5
1 parent 109037c
commit 59d5bf5
Showing 1 changed file with 8 additions and 3 deletions.
diff --git a/src/app/gui/mainwindow.cpp b/src/app/gui/mainwindow.cpp
@@ -364,10 +364,11 @@ int MainWindow::getFrameCount()
 
 void MainWindow::drawTimer_timeout()
 {
-    if (getFrameCount() == 0)
+    const int framecount = getFrameCount();
+    if (framecount == 0)
         return;
 
-    setFrameIndex((getCurrentFrameIndex()+1)%getFrameCount());
+    setFrameIndex((getCurrentFrameIndex()+1)%framecount);
 }
 
 void MainWindow::connectionScannerTimer_timeout()
@@ -1253,7 +1254,11 @@ void MainWindow::on_actionStepForward_triggered()
     if (patternCollection.isEmpty())
         return;
 
-    setFrameIndex((getCurrentFrameIndex()+1)%getFrameCount());
+    const int framecount = getFrameCount();
+    if (framecount == 0)
+        return;
+
+    setFrameIndex((getCurrentFrameIndex()+1)%framecount);
 }
 
 void MainWindow::on_actionStepBackward_triggered()