Re: [PATCH] Fix the fix for PR fortran/90988

2019-11-01 Thread Steve Kargl
On Fri, Nov 01, 2019 at 08:49:58AM +0100, Tobias Burnus wrote: > On 10/31/19 11:16 PM, Steve Kargl wrote: > > > Jeff Law sent me an email … caused a regression for fixed-form > > source code. > > Interesting, what kind of code is used in the real world! > Jeff sent me the file from dl_poly that

Re: [PATCH] Fix the fix for PR fortran/90988

2019-11-01 Thread Jeff Law
On 11/1/19 1:49 AM, Tobias Burnus wrote: > On 10/31/19 11:16 PM, Steve Kargl wrote: > >> Jeff Law sent me an email … caused a regression for fixed-form >> source code. > > Interesting, what kind of code is used in the real world! It's from the dl_poly package in Fedora IIRC. It was the only inst

Re: [PATCH] Fix the fix for PR fortran/90988

2019-11-01 Thread Tobias Burnus
On 10/31/19 11:16 PM, Steve Kargl wrote: Jeff Law sent me an email … caused a regression for fixed-form source code. Interesting, what kind of code is used in the real world! OK to commit? OK. Thanks for the patch! Tobias 2019-10-31 Steven G. Kargl PR fortran/90988 *

[PATCH] Fix the fix for PR fortran/90988

2019-10-31 Thread Steve Kargl
Jeff Law sent me an email that showed my patch for PR fortran/90988 caused a regression for fixed-form source code. The attached patch address that regression. Briefly, the original patch fixed free-form souce code parsing for 'PUBLICX' where the required whitespace was missing (i.e., 'PUBLIC X')