Re: [patch, fortran] Fix PR 50627

2013-02-02 Thread Thomas Koenig
Hi Paul, OK for trunk. The part in decl.c is exactly what I was thinking to do. Thanks for the patch Thanks a lot for your review! Committed to trunk as rev. 195684, 4.7 and 4.6 to follow. Thomas

Re: [patch, fortran] Fix PR 50627

2013-02-02 Thread Paul Richard Thomas
Dear Thomas, I wrote this almost immediately after you posted the patch, got disturbed and forgot about it - my apologies. Note Dominique's reply after you corrected the PR number. OK for trunk. The part in decl.c is exactly what I was thinking to do. Thanks for the patch Paul

Re: [patch, fortran] Fix PR 50627

2013-01-27 Thread Dominique Dhumieres
> Of course, that should be PR 50627 in the ChangeLog. I think you need both PR 50627 and 56054. The patch fixes these PRs, full testing in progress. Dominique

Re: [patch, fortran] Fix PR 50627

2013-01-27 Thread Thomas Koenig
Of course, that should be PR 50627 in the ChangeLog.

[patch, fortran] Fix PR 50627

2013-01-27 Thread Thomas Koenig
Hello world, the attached patch fixes the regression regarding freeing namespaces twice. The test cases where also run unter valgrind without producing errors. Regression-tested. OK for trunk? Thomas 2013-01-27 Thomas Koenig PR fortran/56054 * decl.c (gfc_match_en