Thank you Tobias, I thought that "Change name in gfc_check_init_expr"
was sufficient.

2012/6/3 Tobias Burnus <bur...@net-b.de>:
> Hi Alessandro, hi all,
>
>
> Alessandro Fanfarillo wrote:
>>
>> in attachment the patch which includes the review comments provided by
>> Tobias.
>
>
> Thanks for the patch, which I committed as Rev. 188152. Congratulation to
> your second committed patch.
>
> Nit: You forgot twice to add the prefix "gfc_" in the ChangeLog; I corrected
> it before committal.
>
>  * * *
>
> If possible, use "-p" when you do a diff. With svn, simply pass "-x -p" (or
> --diff-cmd=diff -x '-p -u'); git does this already by default. [Some prefer
> "-c" to "-u", which is also fine.]  Without the "-p" flag, the result is:
>
> --- gcc/fortran/check.c (revisione 188147)
> +++ gcc/fortran/check.c
> @@ -163,7 +163,7 @@
>   if (scalar_check (k, n) == FAILURE)
>
>
> While with "-p" flag, one gets:
>
> --- gcc/fortran/check.c    (Revision 188123)
> +++ gcc/fortran/check.c
> @@ -163,7 +163,7 @@ kind_check (gfc_expr *k, int n, bt type)
>   if (scalar_check (k, n) == FAILURE)
>
>
> The difference is that the @@ line shows the function name (here:
> "kind_check"). That information makes it easier to review a patch as one
> then knows more about the context.
>
> Tobias

Reply via email to