This is a live mirror of the Perl 5 development currently hosted at https://github.com/perl/perl5
Add note about the -std=c89 needing -pedantic.
authorJarkko Hietaniemi <jhi@iki.fi>
Wed, 18 Jun 2014 15:49:40 +0000 (11:49 -0400)
committerJarkko Hietaniemi <jhi@iki.fi>
Wed, 18 Jun 2014 18:14:29 +0000 (14:14 -0400)
cflags.SH

index cdcbf4c..e2bf4b9 100755 (executable)
--- a/cflags.SH
+++ b/cflags.SH
@@ -159,9 +159,14 @@ case "$gccversion" in
 '') ;;
 [12]*) ;; # gcc versions 1 (gasp!) and 2 are not good for this.
 Intel*) ;; # # Is that you, Intel C++?
-# XXX if -pedantic (or -pedantic-errors!) is ever added,
+# XXX Note that -std=c89 without -pedantic is rather pointless.
+# Just -std=c89 means "if there is room for interpretation,
+# interpret the C89 way."  It does NOT mean "strict C89",
+# you would need to add the -pedantic to get that.
+#
+# XXX If -pedantic (or -pedantic-errors!) is ever added,
 # the -Werror=declaration-after-statement can be removed
-# since -std=c89 -pedantic implies the -Werror=...
+# since "-std=c89 -pedantic" implies the -Werror=d-a-s.
 *)  for opt in -ansi -std=c89 -Wextra -W \
                -Werror=declaration-after-statement \
                -Wc++-compat -Wwrite-strings