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