--- Comment #9 from burnus at gcc dot gnu dot org 2008-01-22 09:30 ---
Patch was: http://gcc.gnu.org/ml/fortran/2008-01/msg00261.html
FIXED on the trunk (4.3.0).
Manfred Schwarb wrote:
> I like the idea of being able to compile also legacy code with gfortran.
> However, I really see the
--- Comment #8 from manfred99 at gmx dot ch 2008-01-22 09:23 ---
Gaa, my example is BS, of course.
The really interesting thing is however, that g77 compiles it just fine
and happily treats "1" as continuation line.
What a can of worms!!
--
http://gcc.gnu.org/bugzilla/show_bug
--- Comment #7 from manfred99 at gmx dot ch 2008-01-22 08:41 ---
Hmm,
I like the idea of being able to compile also legacy code with gfortran.
However, I really see the point Steve is making. With this patch things
get confusing.
If you consider a modified version of Steve's example:
c2
--- Comment #6 from burnus at gcc dot gnu dot org 2008-01-22 07:34 ---
Subject: Bug 34899
Author: burnus
Date: Tue Jan 22 07:33:46 2008
New Revision: 131713
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=131713
Log:
2008-01-22 Tobias Burnus <[EMAIL PROTECTED]>
PR fort
--- Comment #5 from jvdelisle at gcc dot gnu dot org 2008-01-22 03:12
---
With Tobias patch, the example code in comment 4 compiles with or without a tab
preceding the 1 digit. It also works with 0 to 5 spaces preceding the tab and
fails with anything between the tab and the first digi
--- Comment #4 from kargl at gcc dot gnu dot org 2008-01-21 19:24 ---
(In reply to comment #3)
>
> Sorry, no! The code is non-conforming. The code should be fixed.
>
I downloaded the FITS package, and it's worse than I thought. Please
don't unfix gfortran. This is valid fixed-form
--- Comment #3 from kargl at gcc dot gnu dot org 2008-01-21 17:40 ---
(In reply to comment #0)
> The following program - which uses a very common extension - is rejected by
> gfortran. It is accepted by ifort, NAG f95 (!), sunf95, openf95, pgi, absoft.
> As it is a very common extension,
--- Comment #2 from burnus at gcc dot gnu dot org 2008-01-21 10:55 ---
Actually, this does not work as in scanner.c's "load_line" the '\t' is changed
into 6 spaces. And of cause, if one changes this, e.g., "\tPARAMETER" is no
longer recognized ...
+ if (c < 1 || c > 9)
And t
--- Comment #1 from burnus at gcc dot gnu dot org 2008-01-21 08:45 ---
Real-world test case:
http://www.chara.gsu.edu/~gudehus/fits_library_package.html
Untested:
Index: scanner.c
===
--- scanner.c (revision 131688)
+++
