[PATCH] Fortran: PR/34908, PR/36276: do not free current_file->filename if it changes

Tobias Burnus burnus@net-b.de
Fri Jun 20 13:51:00 GMT 2008


Laurynas Biveinis wrote:
> Hi,
>
> The "old" current_file->filename is referenced by linemap, thus it
> cannot be freed at this point. The proper fix probably would be to
> update linemap with the new filename, but in any case this trivial
> patch is an improvement. This change is already suggested by a comment
> in PR/36276, I arrived at it independently by investigating PR/34908.
> Bootstrapped/regtested Fortran on trunk on x86_64-unknown-linux-gnu,
> OK for trunk? OK for 4.3 branch after bootstrapping/regtesting there?
>
> 2008-06-20  Laurynas Biveinis  <laurynas.biveinis@gmail.com>
>
>         PR fortran/34908, fortran/36276
>         * scanner.c (preprocessor_line):  Do not call gfc_free for
>         current_file->filename if it differs from filename.
>   
My patch, err  your patch*  is OK. :-)

But as Daniel already wrote, please change PR lines in the change log to:  

        PR fortran/34908
	PR fortran/36276


Tobias

(* I did the functionally identical patch in  PR 36276.)



More information about the Fortran mailing list