[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