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

2016-11-22 Thread Steve Kargl
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 n

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

2016-11-22 Thread Janus Weil
Hi Steve, > 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. 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 th

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

2016-11-22 Thread Steve Kargl
The patch and ChangeLog shuod be sufficient to explain the change. Regression tested on x86_64-*-freebsd. OK to commit? 2016-11-22 Steven G. Kargl PR fortran/78479 * expr.c (gfc_apply_init): Allocate a charlen if needed. 2016-11-22 Steven G. Kargl PR fortran/7847