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/ardour/ardour/types.h b/libs/ardour/ardour/types.h
index 7444a54a7c..165afd0dd9 100644
--- a/libs/ardour/ardour/types.h
+++ b/libs/ardour/ardour/types.h
@@ -267,8 +267,7 @@ namespace ARDOUR {
 				return seconds != 0;
 			}
 
-			/* NOTREACHED */
-			assert (false);
+			abort(); /* NOTREACHED */
 			return false;
 		}
 	};