diff options
| author | David Robillard <d@drobilla.net> | 2020-11-29 10:15:16 +0100 |
|---|---|---|
| committer | David Robillard <d@drobilla.net> | 2020-11-29 12:00:37 +0100 |
| commit | 19b657b9882e53863cbd80964fcea0e1213439bd (patch) | |
| tree | 8667363aefb833ac3fbc9b234bb53cee0d20f6fa | |
| parent | d4d86ae54ebe73ad481f84145002ed8fb425d34f (diff) | |
| download | patchage-19b657b9882e53863cbd80964fcea0e1213439bd.tar.gz patchage-19b657b9882e53863cbd80964fcea0e1213439bd.tar.bz2 patchage-19b657b9882e53863cbd80964fcea0e1213439bd.zip | |
Fix GCC return type warnings
It would be nice if compilers could agree on how to deal with this.
| -rw-r--r-- | src/PortID.hpp | 3 | ||||
| -rw-r--r-- | src/event_to_string.cpp | 6 | ||||
| -rw-r--r-- | src/warnings.hpp | 6 |
3 files changed, 15 insertions, 0 deletions
diff --git a/src/PortID.hpp b/src/PortID.hpp index e399220..d92f08e 100644 --- a/src/PortID.hpp +++ b/src/PortID.hpp @@ -19,6 +19,7 @@ #include "ClientID.hpp" #include "ClientType.hpp" +#include "warnings.hpp" #include <cassert> #include <iostream> @@ -68,6 +69,8 @@ struct PortID case Type::alsa: return ClientID::alsa(_alsa_client); } + + PATCHAGE_UNREACHABLE(); } Type type() const { return _type; } diff --git a/src/event_to_string.cpp b/src/event_to_string.cpp index 503144c..cdfeee4 100644 --- a/src/event_to_string.cpp +++ b/src/event_to_string.cpp @@ -43,6 +43,8 @@ struct EventPrinter case ClientType::alsa: return "ALSA"; } + + PATCHAGE_UNREACHABLE(); } std::string operator()(const DriverAttachmentEvent& event) @@ -80,6 +82,8 @@ struct EventPrinter case PortType::jack_cv: return "JACK CV"; } + + PATCHAGE_UNREACHABLE(); } std::string operator()(const SignalDirection direction) @@ -92,6 +96,8 @@ struct EventPrinter case SignalDirection::duplex: return "duplex"; } + + PATCHAGE_UNREACHABLE(); } std::string operator()(const PortCreationEvent& event) diff --git a/src/warnings.hpp b/src/warnings.hpp index 31cdf42..6931bee 100644 --- a/src/warnings.hpp +++ b/src/warnings.hpp @@ -54,4 +54,10 @@ #endif +#if defined(__GNUC__) +# define PATCHAGE_UNREACHABLE() __builtin_unreachable() +#else +# define PATCHAGE_UNREACHABLE() +#endif + #endif // PATCHAGE_WARNINGS_HPP |