Fix -Wimplicit-fallthrough warnings

libk3b/jobs/k3bdvdcopyjob.cpp

@@ -253,6 +253,7 @@ void K3b::DvdCopyJob::slotDiskInfoReady( K3b::Device::DeviceHandler* dh )
                     }
                 }
             }
+            Q_FALLTHROUGH();
         case K3b::Device::MEDIA_DVD_R:
         case K3b::Device::MEDIA_DVD_R_SEQ:
         case K3b::Device::MEDIA_DVD_RW:

libk3b/projects/datacd/k3bisoimager.cpp

@@ -163,7 +163,7 @@ void K3b::IsoImager::slotProcessExited( int exitCode, QProcess::ExitStatus exitS
 
                         break;
                     }
-                    // otherwise just fall through
+                    Q_FALLTHROUGH();
 
                 default:
                     if( !d->knownError && !mkisofsReadError() ) {

libk3bdevice/k3bdevice.cpp

@@ -2130,6 +2130,7 @@ K3b::Device::DiskInfo K3b::Device::Device::diskInfo() const
                     // done.
                     break;
                 }
+                Q_FALLTHROUGH();
 
             default:
             case MEDIA_CD_ROM:
@@ -2153,6 +2154,7 @@ K3b::Device::DiskInfo K3b::Device::Device::diskInfo() const
                         inf.d->usedCapacity = readToc().length();
                     }
                 }
+                break;
 
             case MEDIA_DVD_ROM: {
                 K3b::Msf readCap;
@@ -2238,6 +2240,7 @@ K3b::Device::DiskInfo K3b::Device::Device::diskInfo() const
 
             case MEDIA_DVD_RW_OVWR:
                 inf.d->numSessions = 1;
+                Q_FALLTHROUGH();
             case MEDIA_DVD_RW:
             case MEDIA_DVD_RW_SEQ:
                 // only one track on a DVD-RW media

src/k3b.cpp

@@ -904,8 +904,7 @@ bool K3b::MainWindow::canCloseDocument( K3b::Doc* doc )
                                               KStandardGuiItem::save(),
                                               KStandardGuiItem::dontSave() ) ) {
     case KMessageBox::Yes:
-        if ( !fileSave( doc ) )
-            return false;
+        return fileSave( doc );
     case KMessageBox::No:
         return true;
     default:

src/k3bprojectmanager.cpp

@@ -337,7 +337,8 @@ void K3b::ProjectManager::loadDefaults( K3b::Doc* doc )
         movixDoc->setEjectDisk( c.readEntry( "eject", false ) );
         movixDoc->setRandomPlay( c.readEntry( "random_play", false ) );
         movixDoc->setNoDma( c.readEntry( "no_dma", false ) );
-        // fallthrough
+
+        Q_FALLTHROUGH();
     }
 
     case K3b::Doc::DataProject: {

src/projects/k3baudioprojectmodel.cpp

@@ -269,8 +269,8 @@ QVariant K3b::AudioProjectModel::data( const QModelIndex& index, int role ) cons
             f.setPointSize( f.pointSize() - 2 );
             return f;
         }
-        break;
         }
+        break;
 
     case Qt::ForegroundRole:
         if ( index.column() == FilenameColumn ) {

src/projects/k3bdataurladdingdialog.cpp

@@ -468,13 +468,13 @@ void K3b::DataUrlAddingDialog::slotAddUrls()
                                                       KStandardGuiItem::cancel())  ) {
                 case 2:
                     m_bFolderLinksFollowAll = true;
-                    // fall-through
+                    Q_FALLTHROUGH();
                 case 1:
                     followLink = true;
                     break;
                 case 4:
                     m_bFolderLinksAddAll = true;
-                    // fall-through
+                    Q_FALLTHROUGH();
                 case 3:
                     followLink = false;
                     break;

src/projects/kostore/KoStore.cpp

@@ -127,6 +127,7 @@ KoStore* KoStore::createStore( QIODevice *device, Mode mode, const QByteArray &
   case Directory:
     qCCritical(KOSTORE) << "Can't create a Directory store for a memory buffer!" << endl;
     // fallback
+    Q_FALLTHROUGH();
   case Zip:
 #ifdef QCA2
     if( automatic && mode == Read ) {

src/rip/k3baudiotrackmodel.cpp

@@ -345,6 +345,7 @@ bool K3b::AudioTrackModel::setData( const QModelIndex& index, const QVariant& va
                 emit dataChanged( index, index );
                 return true;
             }
+            break;
 
         case Qt::CheckStateRole:
             setTrackChecked( index.row(), value.toInt() == Qt::Checked );