diff options
| author | Robin Gareus <robin@gareus.org> | 2014-11-14 10:47:43 +0100 |
|---|---|---|
| committer | Robin Gareus <robin@gareus.org> | 2014-11-14 11:30:08 +0100 |
| commit | 6b3a8915f3c50f7220121fcb9202ec20144389c4 (patch) | |
| tree | 08e7ae373859e5ca15b8fa0ca075b18b65915a22 /libs/ardour/butler.cc | |
| parent | 5d5d9cb9e262498dc0831dc362fbcb3f92bcdb9f (diff) | |
add abort() to non-reached code
This cleans up a lot of false-positives in static analysis
and also helps compilers to optimize code paths in general.
(tagging the fatal stingstream operator as ‘noreturn’ is
far less trivial)
Diffstat (limited to 'libs/ardour/butler.cc')
| -rw-r--r-- | libs/ardour/butler.cc | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/libs/ardour/butler.cc b/libs/ardour/butler.cc index 327665441e..9bc232007b 100644 --- a/libs/ardour/butler.cc +++ b/libs/ardour/butler.cc @@ -202,7 +202,7 @@ Butler::dequeue_request (Request::Type& r) return false; } else { fatal << _("Error reading from butler request pipe") << endmsg; - /*NOTREACHED*/ + abort(); /*NOTREACHED*/ } #else r = (Request::Type) m_request_state.get(); @@ -242,7 +242,7 @@ Butler::thread_work () case Request::Quit: return 0; - /*NOTREACHED*/ + abort(); /*NOTREACHED*/ break; default: |