From 466d78f961128fbbbb51a7bcc293bdb748043ab4 Mon Sep 17 00:00:00 2001 From: Karl Williamson Date: Sat, 20 Mar 2021 11:14:00 -0600 Subject: [PATCH] regexec.c: Refactor switch default() It seems clearer to me to have the panic at the end of the routine instead of as the default: of a switch(). --- regexec.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/regexec.c b/regexec.c index bd9df72b2da2..0d881c9aeb19 100644 --- a/regexec.c +++ b/regexec.c @@ -462,9 +462,13 @@ S_isFOO_lc(pTHX_ const U8 classnum, const U8 character) case CC_ENUM_WORDCHAR_: return isU8_WORDCHAR_LC(character); case CC_ENUM_XDIGIT_: return isU8_XDIGIT_LC(character); default: /* VERTSPACE should never occur in locales */ - Perl_croak(aTHX_ "panic: isFOO_lc() has an unexpected character class '%d'", classnum); + break; } + Perl_croak(aTHX_ + "panic: isFOO_lc() has an unexpected character class '%d'", + classnum); + NOT_REACHED; /* NOTREACHED */ return FALSE; }