Hi Tobias and Mikael,

> Something went wrong with the indentation of the last two lines.

Fixed.

> Additionally: How about simply returning with an "return false;"?

After some more thinking, I used the option that you suggested.  We'll
see if we get feedback from users who want something else, if any :-)

I think it should be:
>    for (i = 0; i < GFC_MAX_DIMENSIONS; i++)

Fixed with

+       for (i = 0; i < GFC_MAX_DIMENSIONS
+              && ref->u.ar.dimen_type[i] != 0; i++)

(I didn't want to loop over all dimensions unconditially).
I also updated the test case to reflect this.

Commit is at http://gcc.gnu.org/viewcvs?rev=201294&root=gcc&view=rev .

Thanks a lot for the reviews!

        Thomas


Reply via email to