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

Reply via email to