Dear Paul,

On Tue, Jan 03, 2012 at 09:30:28PM +0100, Paul Richard Thomas wrote:
> Bootstrapped and regtested on x86_64/FC9 - OK for trunk?

OK. Thanks for the patch.


> +  /* If we find a deferred typebound procedure, check for derived types
> +     that an over-riding typebound procedure has not been missed.  */

s/over-riding/overriding/.
(I somehow miss something before "that" - such as "ensuring" or similar.)

> +  if (e->value.compcall.tbp->deferred
> +        && e->value.compcall.name
> +        && !e->value.compcall.tbp->non_overridable
> +        && e->value.compcall.base_object
> +        && e->value.compcall.base_object->ts.type == BT_DERIVED)

Indentation looks wrong.

While I also tend to reduce the number of run tests, it probably makes sense
to follow Thomas' suggestion and add a new test case instead of modifying
the old one.

Tobias

----- End forwarded message -----

Reply via email to