BUG: My change to a switch statement made opie-gutenbrowser uncompilable.

FIX: The code there is just strange. I tore out the switch statement and
replaced it with a simple if block instead.

1 files changed, 5 insertions, 7 deletions

diff --git a/noncore/apps/opie-gutenbrowser/openetext.cpp b/noncore/apps/opie-gutenbrowser/openetext.cpp
index a5d855d..75edacd 100644
--- a/noncore/apps/opie-gutenbrowser/openetext.cpp
+++ b/noncore/apps/opie-gutenbrowser/openetext.cpp
@@ -359,35 +359,33 @@ bool OpenEtext::checkConf()
             return true;
         }
     }
     return false;
 }
 
 void OpenEtext::remove()
 {
     title_text = QListBox_1->text( QListBox_1->currentItem() );
     title_text=title_text.stripWhiteSpace();
     QString msg ="<p>Do you really want to REMOVE\n" + title_text +
                  "?\nThis will not delete the file.</P>";
-    switch( QMessageBox::information( this, tr("Remove Etext"),
+    unsigned short clickVal =
+        QMessageBox::information( this, tr("Remove Etext"),
                                       tr(msg),
-                                      tr("&Yes"), tr("&Cancel"), 0 ) )
-    {
-        case 0: // Yes clicked,
+                                  tr("&Yes"), tr("&Cancel"), 0 );
+    if ( clickVal == 0 ) {
+        // The yes button was clicked
             removeSelection();
             QListBox_1->clear();
             getTitles();
-            break;
-	case 1:
-        default:
     }
 }
 
 /*!
  * This removes selected title entry
  */
 void OpenEtext::removeSelection()
 {
     Config config("Gutenbrowser");
     config.setGroup( "Files" );
     s_numofFiles = config.readEntry("NumberOfFiles", "0");
     int rem=0;