/* NOTREACHED */ belongs *before* the unreachable.

Definitely not *after* it.  It marks the start of the unreachable,
not the first unrechable line.  And if they are in that order,
it looks better to linebreak after the lint hint.

1 files changed, 2 insertions, 1 deletions

diff --git a/pp_pack.c b/pp_pack.c
index f877fe21af..a622d92dd0 100644
--- a/pp_pack.c
+++ b/pp_pack.c
@@ -567,7 +567,8 @@ S_group_end(pTHX_ const char *patptr, const char *patend, char ender)
     }
     Perl_croak(aTHX_ "No group ending character '%c' found in template",
                ender);
-    NOT_REACHED; /* NOTREACHED */
+    /* NOTREACHED */
+    NOT_REACHED;
 }