On Mon, Nov 13, 2017 at 04:42:31PM -0500, Fritz Reese wrote: > On Thu, Nov 9, 2017 at 5:02 PM, Steve Kargl > <s...@troutmask.apl.washington.edu> wrote: > > The following patch fixes PR fortran/78240. It seems > > to me to be inelegant, but it does pass regression > > testing. [...] OK to commit? > > Upon closer analysis, the patch is insufficient to fix the PR. I will > explain below. At the bottom of this letter I explain and have > attached a new patch which fixes some more subtle issues in the code > which causes the PR.
Thanks for taking a look. You're correct that I should have looked more closely at the memory management that you noted. > Thanks for pointing me to this issue and allowing me time to review > it. The new patch passes all regression tests including its two tests > (one for each issue above). OK to commit this one? Yes. Thanks for the patch. -- steve
