Introducing ASSERT_NOT_REACHED

In a similar fashion as WebKit, this might make those cases
where we don't expect to go more obvious in debug builds.
I also took the liberty to revisit a lot of switch statements
but it could potentially be used elsewhere

Reviewed-by: andrea

1 files changed, 1 insertions, 5 deletions

diff --git a/src/mainwindow.cpp b/src/mainwindow.cpp
index d0d4d401..09e2ae1a 100644
--- a/src/mainwindow.cpp
+++ b/src/mainwindow.cpp
@@ -1190,17 +1190,13 @@ void MainWindow::notifyMessage(const QString &msg, Rekonq::Notify status)
 
     switch (status)
     {
-    case Rekonq::Url:
-        break;
     case Rekonq::Info:
         m_hidePopupTimer->start(500);
         break;
+    case Rekonq::Url:
     case Rekonq::Success:
-        break;
     case Rekonq::Error:
-        break;
     case Rekonq::Download:
-        break;
     default:
         break;
     }