[multimedia/kdenlive] src/timeline2/view/qml: Fix confusing-expression-statement qml error with Qt 6.12

Julius Künzel <[email protected]>
Newsgroups gmane.comp.kde.cvs
Message-ID <[email protected]>
Git commit 2a40ef90e699ae270fc8c1e4b0db36c0c9b9a029 by Julius Künzel.
Committed on 09/08/2026 at 21:50.
Pushed by jlskuz into branch 'master'.

Fix confusing-expression-statement qml error with Qt 6.12

M  +8    -9    src/timeline2/view/qml/ClipThumbs.qml

https://invent.kde.org/multimedia/kdenlive/-/commit/2a40ef90e699ae270fc8c1e4b0db36c0c9b9a029

diff --git a/src/timeline2/view/qml/ClipThumbs.qml b/src/timeline2/view/qml/ClipThumbs.qml
index 0758fe63ff..338a514e59 100644
--- a/src/timeline2/view/qml/ClipThumbs.qml
+++ b/src/timeline2/view/qml/ClipThumbs.qml
@@ -31,29 +31,28 @@ Row {
         id: thumbRepeater
         // switching the model allows one to have different view modes.
         // We set the model to the number of frames we want to show
-        model: switch (thumbRow.thumbsFormat) {
+        model: {
+            switch (thumbRow.thumbsFormat) {
                    case 0:
                        // in/out
                        if (parent.width > thumbRow.thumbWidth) {
-                           2 // 2: will display start / end thumbs
+                           return 2 // 2: will display start / end thumbs
                        } else {
-                           1 // 1: if the width of the container is to small, only show first thumbnail
+                           return 1 // 1: if the width of the container is to small, only show first thumbnail
                        }
-                       break;
                    case 1:
                        // All frames
                        // display as many thumbnails as can fit into the container
-                       Math.ceil(parent.width / thumbRow.thumbWidth)
-                       break;
+                       return Math.ceil(parent.width / thumbRow.thumbWidth)
                    case 2:
                        // In frame only
-                       1 // 1: only show first thumbnail
-                       break;
+                       return 1 // 1: only show first thumbnail
                    case 3:
                    default:
                        // No thumbs
-                       0 // 0: will disable thumbnails
+                       return 0 // 0: will disable thumbnails
                }
+        }
         property int startFrame: thumbRow.parentClip.inPoint
         property int endFrame: thumbRow.parentClip.outPoint
         property real imageWidth: Math.max(thumbRow.thumbWidth, parent.width / thumbRepeater.count)
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.