[Kdenlive-devel] [PATCH 10/16] markerdialog.cpp: add break to switch case

Mikko Rapeli mikko.rapeli at iki.fi
Fri Aug 10 12:17:11 UTC 2012


Fixes Coverity CID 709294: Missing break in switch (MISSING_BREAK)
This case (value 9) is not terminated by a 'break' statement.
 70        case PLAYLIST:
 71            connect(this, SIGNAL(updateThumb()), m_previewTimer, SLOT(start()));
The above case falls through to this one.
 72        case IMAGE:
 73        case TEXT:
 74            p = QPixmap::fromImage(KThumb::getFrame(m_producer, m_in->getValue(), swidth, width, 100));
 75            break;
---
 src/markerdialog.cpp |    1 +
 1 file changed, 1 insertion(+)

diff --git a/src/markerdialog.cpp b/src/markerdialog.cpp
index a3935e5..4c07961 100644
--- a/src/markerdialog.cpp
+++ b/src/markerdialog.cpp
@@ -69,6 +69,7 @@ MarkerDialog::MarkerDialog(DocClipBase *clip, CommentedTime t, Timecode tc, cons
         case SLIDESHOW:
         case PLAYLIST:
             connect(this, SIGNAL(updateThumb()), m_previewTimer, SLOT(start()));
+            break;
         case IMAGE:
         case TEXT:
             p = QPixmap::fromImage(KThumb::getFrame(m_producer, m_in->getValue(), swidth, width, 100));
-- 
1.7.10.4





More information about the Kdenlive mailing list