------- Comment #2 from kargl at gcc dot gnu dot org 2010-08-04 18:09 -------
(In reply to comment #1)
> PATCH - lightly tested. Now regtesting.
>
> Index: gcc/fortran/resolve.c
> ===================================================================
> --- gcc/fortran/resolve.c (Revision 162868)
> +++ gcc/fortran/resolve.c (Arbeitskopie)
> @@ -936,11 +936,26 @@ resolve_structure_cons (gfc_expr *expr)
> p = gfc_constructor_first (cons->expr->value.constructor);
> if (cons->expr->ts.u.cl != p->expr->ts.u.cl)
> {
> - gfc_free_expr (cons->expr->ts.u.cl->length);
> - gfc_free (cons->expr->ts.u.cl);
> + gfc_charlen *cl, *cl2;
> +
> + cl2 = NULL;
> + for (cl = gfc_current_ns->cl_list; cl; cl = cl->next)
> + {
> + if (cl == cons->expr->ts.u.cl)
> + break;
> + cl2 = cl;
> + }
> +
> + gcc_assert (cl);
> +
> + if (cl2)
> + cl2->next = cl->next;
> +
> + gfc_free_expr (cl->length);
> + gfc_free (cl);
> }
>
> - cons->expr->ts.u.cl = gfc_get_charlen ();
> + cons->expr->ts.u.cl = gfc_new_charlen (gfc_current_ns, NULL);
> cons->expr->ts.u.cl->length_from_typespec = true;
> cons->expr->ts.u.cl->length = gfc_copy_expr
> (comp->ts.u.cl->length);
> gfc_resolve_character_array_constructor (cons->expr);
>
I must be missing something here. What does cl2 do in the above
patch? You set it, but it is never used.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45183