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
@@ -367,19 +367,17 @@ 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:
     }
 }
 
 /*!