diff options
author | Pierre Rossi <pierre.rossi@gmail.com> | 2011-07-28 11:04:07 +0200 |
---|---|---|
committer | Andrea Diamantini <adjam7@gmail.com> | 2011-07-28 11:04:07 +0200 |
commit | e9770b468474dbb1e2774d76b5309d729f35f4b8 (patch) | |
tree | 04c1056aa45f8e823ba398efbf68a3c99208fc58 /src/application.cpp | |
parent | Merge branch 'HOOKS' (diff) | |
download | rekonq-e9770b468474dbb1e2774d76b5309d729f35f4b8.tar.xz |
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
Diffstat (limited to 'src/application.cpp')
-rw-r--r-- | src/application.cpp | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/src/application.cpp b/src/application.cpp index fc4e81fc..a62d548e 100644 --- a/src/application.cpp +++ b/src/application.cpp @@ -238,12 +238,10 @@ int Application::newInstance() case 0: // new tab page loadUrl(KUrl("about:home") , Rekonq::NewWindow); break; - case 1: // blank page - loadUrl(KUrl("about:blank") , Rekonq::NewWindow); - break; case 2: // homepage loadUrl(KUrl(ReKonfig::homePage()) , Rekonq::NewWindow); break; + case 1: // blank page default: loadUrl(KUrl("about:blank") , Rekonq::NewWindow); break; @@ -632,6 +630,7 @@ void Application::updateConfiguration() break; default: // non extant case + ASSERT_NOT_REACHED(); break; } |