On 24 April 2017 10:56:57 CEST, Janus Weil <ja...@gcc.gnu.org> wrote: >Hi Christophe, > >2017-04-24 10:25 GMT+02:00 Christophe Lyon ><christophe.l...@linaro.org>: >>>>> the patch in the attachment fixes a memory leak by >auto-deallocating >>>>> the allocatable components of an allocatable intent(out) argument. >>>>> >>>>> Regtests cleanly on x86_64-linux-gnu. Ok for trunk? >>>> >>>> OK for trunk. >>> >>> thanks for the review! Committed as r247083. >> >> This patch causes an error message from DejaGnu: >> (DejaGnu) proc "cleanup-tree-dump original" does not exist. > >thanks for letting me know. I didn't notice that ... > > >> I'm not familiar with fortran, so I'm not sure it is as obvious as >> removing cleanup-tree-dump as it is done in the other neighboring >tests? > >Yes, probably it should just be removed. I assume this kind of cleanup >is being done automatically now? I actually took it from this wiki >page:
Yes it is done automatically nowadays. > >https://gcc.gnu.org/wiki/TestCaseWriting > >So I guess this needs to be updated as well. Will take care of both >points tonight ... Obviously I did not think about updating the wiki back then. TIA for fixing the wiki. Cheers,
