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, 9 insertions, 11 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
@@ -365,23 +365,21 @@ bool OpenEtext::checkConf()
 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"),
-                                      tr(msg),
-                                      tr("&Yes"), tr("&Cancel"), 0 ) )
-    {
-        case 0: // Yes clicked,
-            removeSelection();
-            QListBox_1->clear();
-            getTitles();
-            break;
-	case 1:
-        default:
+    unsigned short clickVal =
+        QMessageBox::information( this, tr("Remove Etext"),
+                                  tr(msg),
+                                  tr("&Yes"), tr("&Cancel"), 0 );
+    if ( clickVal == 0 ) {
+        // The yes button was clicked
+        removeSelection();
+        QListBox_1->clear();
+        getTitles();
     }
 }
 
 /*!
  * This removes selected title entry
  */