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 /gtk2_ardour/editor_audio_import.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 'gtk2_ardour/editor_audio_import.cc')
| -rw-r--r-- | gtk2_ardour/editor_audio_import.cc | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/gtk2_ardour/editor_audio_import.cc b/gtk2_ardour/editor_audio_import.cc index e5ac8c6c72..af1d41f91f 100644 --- a/gtk2_ardour/editor_audio_import.cc +++ b/gtk2_ardour/editor_audio_import.cc @@ -314,14 +314,14 @@ Editor::do_import (vector<string> paths, ImportDisposition disposition, ImportMo continue; case 0: fatal << "Updating existing sources should be disabled!" << endmsg; - /* NOTREACHED*/ + abort(); /* NOTREACHED*/ break; case 1: replace = false; break; default: fatal << "Illegal return " << check << " from check_whether_and_how_to_import()!" << endmsg; - /* NOTREACHED*/ + abort(); /* NOTREACHED*/ } /* have to reset this for every file we handle */ |