At the very least that would warrant an explicit std::abort() or std::unreachable() at the end (or the C equivalent).