Re: [PATCH, Fortran, v1] Restructure initialization of allocatable components

2016-11-06 Thread Mikael Morin
Le 03/11/2016 à 14:16, Andre Vehreschild a écrit : @Dominique: Would you give it a go on your open patch collection? Maybe it fixes one PR, but I am not very hopeful, because the patch is merely removing complexity instead of doing new things. Hello, Since you asked: I think the patch fixes

Re: [PATCH, Fortran, v1] Restructure initialization of allocatable components

2016-11-06 Thread Andre Vehreschild
Hi Steve, hi Paul, hi Dominique, hi all, thanks for the reviews, Steve and Paul, thanks for the heavy testing, Dominique. Committed as r241885. Regards, Andre On Sat, 5 Nov 2016 10:46:46 +0100 Paul Richard Thomas wrote: > Dear Andre, > > The patch looks

Re: [PATCH, Fortran, v1] Restructure initialization of allocatable components

2016-11-05 Thread Paul Richard Thomas
Dear Andre, The patch looks fine to me. OK for trunk. Cheers Paul On 4 November 2016 at 01:57, Steve Kargl wrote: > On Thu, Nov 03, 2016 at 02:16:48PM +0100, Andre Vehreschild wrote: >> >> Bootstraps and regtests fine on x86_64-linux/F23. Ok for trunk? >> >>

Re: [PATCH, Fortran, v1] Restructure initialization of allocatable components

2016-11-03 Thread Steve Kargl
On Thu, Nov 03, 2016 at 02:16:48PM +0100, Andre Vehreschild wrote: > > Bootstraps and regtests fine on x86_64-linux/F23. Ok for trunk? > > @Dominique: Would you give it a go on your open patch collection? Maybe it > fixes one PR, but I am not very hopeful, because the patch is merely removing >

[PATCH, Fortran, v1] Restructure initialization of allocatable components

2016-11-03 Thread Andre Vehreschild
Hi all, the attached patch restructures gfortran's way of initializing components of derived types in ALLOCATE. The old way was to generate a new gfc_code-node and add it after the ALLOCATE node to initialize the the derived type on certain conditions (like initializer or allocatable components