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 the PR number in a comment inside the test case. Thanks, Janus > 2016-11-22 Steven G. Kargl <ka...@gcc.gnu.org> > > PR fortran/78479 > * expr.c (gfc_apply_init): Allocate a charlen if needed. > > 2016-11-22 Steven G. Kargl <ka...@gcc.gnu.org> > > PR fortran/78479 > * gfortran.dg/pr78479.f90: New test. > > -- > Steve