Re: [Patch, Fortran, OOP] PR 49638: [OOP] length parameter is ignored when overriding type bound character functions with constant length.

[Thomas Koenig]

Hi Janus,

Here is a preparational patch (basically a subset of the
previous one), which does not do any real changes yet, only some
preparation and cleanup:
* It moves check_typebound_override to interface.c and prefixes it
with gfc_ (I don't like moving and modifying it at the same time).
* It add the commutativity of multiplication in gfc_dep_compare_expr.
* It does some minor cleanup in dependency.c (making two routines
static and removing an unused argument).
Ok for trunk?

Cheers,
Janus


2011-08-06  Janus Weil<ja...@gcc.gnu.org>

        PR fortran/49638
        * dependency.h (gfc_is_same_range,gfc_are_identical_variables): Remove
        two prototypes.
        * dependency.c (gfc_are_identical_variables): Made static and renamed.
        (gfc_dep_compare_expr): Renamed 'gfc_are_identical_variables', handle
        commutativity of multiplication.
        (gfc_is_same_range): Made static and renamed, removed argument 'def'.
        (check_section_vs_section): Renamed 'gfc_is_same_range'.
        * gfortran.h (gfc_check_typebound_override): New prototype.
        * interface.c (gfc_check_typebound_override): Moved here from ...
        * resolv.c (check_typebound_override): ... here (and renamed).
        (resolve_typebound_procedure): Renamed 'check_typebound_override'.

OK from my side (given Mikael's comments), provided you spell resolve.c 
with two e :-)

Regards

        Thomas