/* 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, 10 insertions, 5 deletions

diff --git a/util.c b/util.c
index 4666233db9..f5573e186d 100644
--- a/util.c
+++ b/util.c
@@ -1571,7 +1571,8 @@ Perl_die_sv(pTHX_ SV *baseex)
 {
     PERL_ARGS_ASSERT_DIE_SV;
     croak_sv(baseex);
-    assert(0); /* NOTREACHED */
+    /* NOTREACHED */
+    assert(0);
     NORETURN_FUNCTION_END;
 }
 
@@ -1593,7 +1594,8 @@ Perl_die_nocontext(const char* pat, ...)
     va_list args;
     va_start(args, pat);
     vcroak(pat, &args);
-    assert(0); /* NOTREACHED */
+    /* NOTREACHED */
+    assert(0);
     va_end(args);
     NORETURN_FUNCTION_END;
 }
@@ -1605,7 +1607,8 @@ Perl_die(pTHX_ const char* pat, ...)
     va_list args;
     va_start(args, pat);
     vcroak(pat, &args);
-    assert(0); /* NOTREACHED */
+    /* NOTREACHED */
+    assert(0);
     va_end(args);
     NORETURN_FUNCTION_END;
 }
@@ -1705,7 +1708,8 @@ Perl_croak_nocontext(const char *pat, ...)
     va_list args;
     va_start(args, pat);
     vcroak(pat, &args);
-    assert(0); /* NOTREACHED */
+    /* NOTREACHED */
+    assert(0);
     va_end(args);
 }
 #endif /* PERL_IMPLICIT_CONTEXT */
@@ -1716,7 +1720,8 @@ Perl_croak(pTHX_ const char *pat, ...)
     va_list args;
     va_start(args, pat);
     vcroak(pat, &args);
-    assert(0); /* NOTREACHED */
+    /* NOTREACHED */
+    assert(0);
     va_end(args);
 }