[PATCH] PR fortran/78479 -- allocate a charlen

Steve Kargl sgk@troutmask.apl.washington.edu
Tue Nov 22 21:46:00 GMT 2016


On Tue, Nov 22, 2016 at 10:21:52PM +0100, Janus Weil wrote:
> 
> > The patch and ChangeLog shuod be sufficient to explain the change.
> > Regression tested on x86_64-*-freebsd.  OK to commit?
> 
> the patch itself looks good.
> 

Thanks.

> For the test case, I'd prefer a somewhat more meaningful name (e.g.
> char_component_initializer_3.f90 or similar) and a mention of the PR
> number in a comment inside the test case.

At one time, I also preferred a meaningful name, but have
changed over time to using the PR number as the name.  There
are various pros and cons for using the PR number as the name.
In this, as char*_[1,2].f90 already exist, I rename the testcase.

-- 
Steve



More information about the Gcc-patches mailing list