[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Fix illegal tab character in Fortran language test.
From: |
Eric Blake |
Subject: |
Re: Fix illegal tab character in Fortran language test. |
Date: |
Fri, 04 Sep 2009 05:40:22 -0600 |
User-agent: |
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.23) Gecko/20090812 Thunderbird/2.0.0.23 Mnenhy/0.7.6.666 |
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
[adding the list, which would have been the more appropriate place to send
this]
According to Thomas Jahns on 9/4/2009 3:50 AM:
> Hi,
>
> in commit 58a97ecfca0650c90b5dffacd3565c2647f37f0d of the autoconf repository,
> you replaced several occurrences of blanks with tabs.
>
> Unfortunately TAB is not a legal character in Fortran source and one of the
> lines affected by your change is part of a Fortran print statement which
> therefore does no longer compile with e.g. gfortran -Werror. Thus I wanted to
> ask you to apply the attached patch.
Thanks for the report (it's especially nice when a bug report comes with a
patch, as well as a commit id where the regression was introduced). Shows
how little I know about Fortran (and I guess things like this show why
AC_FC_FREEFORM is so important in the first place, since modern Fortran
has followed the lead of C in its change to allow freeform whitespace). I
tweaked your patch to add a comment and quadrigraph to prevent my editor
from regressing again, added THANKS, NEWS, and ChangeLog edits, and
applied it.
- --
Don't work too hard, make some time for fun as well!
Eric Blake address@hidden
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (Cygwin)
Comment: Public key at home.comcast.net/~ericblake/eblake.gpg
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/
iEYEARECAAYFAkqg/KYACgkQ84KuGfSFAYDKlgCfVZFOzquUw8iO64ow3MkDtd9t
5MQAn155a5T+IHyhs4t1B4ywTF4C/8mD
=gEBt
-----END PGP SIGNATURE-----
>From 87c5dbd357ea16629300d46c93dc4b8239a01ccf Mon Sep 17 00:00:00 2001
From: Thomas Jahns <jahns@dkrz.de>
Date: Fri, 4 Sep 2009 11:48:44 +0200
Subject: [PATCH] Fix illegal tab character in Fortran source.
---
lib/autoconf/fortran.m4 | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/lib/autoconf/fortran.m4 b/lib/autoconf/fortran.m4
index 93b5f9d..99e4b6b 100644
--- a/lib/autoconf/fortran.m4
+++ b/lib/autoconf/fortran.m4
@@ -1204,7 +1204,7 @@ do
program freeform
! FIXME: how to best confuse non-freeform compilers?
print *, 'Hello ', &
- 'world.'
+ 'world.'
end],
[ac_cv_fc_freeform=$ac_flag; break])
done
--
1.5.6.5
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: Fix illegal tab character in Fortran language test.,
Eric Blake <=