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)

1 files changed, 1 insertions, 2 deletions

diff --git a/libs/evoral/evoral/PatchChange.hpp b/libs/evoral/evoral/PatchChange.hpp
index e05157aa92..48ed0f9c13 100644
--- a/libs/evoral/evoral/PatchChange.hpp
+++ b/libs/evoral/evoral/PatchChange.hpp
@@ -147,8 +147,7 @@ public:
 		case 2:
 			return _program_change;
 		default:
-			abort ();
-			/*NOTREACHED*/
+			abort(); /*NOTREACHED*/
 			return _program_change;
 		}
 	}