Re: [patch, libgfortran] PR78351 comma not terminating READ of formatted input field

2018-11-08 Thread Thomas Koenig
Hi Jerry! Hi all, The attached patch adds code in read_sf_internal to handle early termination of reads in the presence of comma's. This is to support legacy codes which are not standard conforming as far as we can tell. The additions are executed only if -std=legacy is given at compile tim

Re: [patch, libgfortran] PR78351 comma not terminating READ of formatted input field

2018-11-04 Thread Jerry DeLisle
On 11/4/18 1:51 AM, Bernhard Reutner-Fischer wrote: On Sat, 3 Nov 2018 15:33:07 -0700 Jerry DeLisle wrote: diff --git a/libgfortran/io/transfer.c b/libgfortran/io/transfer.c index 31198a3cc39..0d26101cef0 100644 --- a/libgfortran/io/transfer.c +++ b/libgfortran/io/transfer.c @@ -260,22 +250

Re: [patch, libgfortran] PR78351 comma not terminating READ of formatted input field

2018-11-04 Thread Bernhard Reutner-Fischer
On Sat, 3 Nov 2018 15:33:07 -0700 Jerry DeLisle wrote: > diff --git a/libgfortran/io/transfer.c b/libgfortran/io/transfer.c > index 31198a3cc39..0d26101cef0 100644 > --- a/libgfortran/io/transfer.c > +++ b/libgfortran/io/transfer.c > @@ -260,22 +250,80 @@ read_sf_internal (st_parameter_dt *dtp,

Re: [patch, libgfortran] PR78351 comma not terminating READ of formatted input field

2018-11-03 Thread Andreas Schwab
On Nov 03 2018, Jerry DeLisle wrote: > + /* To support legacy code we have to scan the input string one byte > + at a time because we don't no where an early comma may be and the s/no/know/ Andreas. -- Andreas Schwab, sch...@linux-m68k.org GPG Key fingerprint = 7578 EB47 D4E5 4D69 2510

[patch, libgfortran] PR78351 comma not terminating READ of formatted input field

2018-11-03 Thread Jerry DeLisle
Hi all, The attached patch adds code in read_sf_internal to handle early termination of reads in the presence of comma's. This is to support legacy codes which are not standard conforming as far as we can tell. The additions are executed only if -std=legacy is given at compile time. It does