[prev in list] [next in list] [prev in thread] [next in thread] 

List:       gcc-fortran
Subject:    [Patch, Fortran, committed] Another C++ism + Fortran-dev update
From:       Tobias Burnus <burnus () net-b ! de>
Date:       2012-07-21 10:10:36
Message-ID: 500A801C.90209 () net-b ! de
[Download RAW message or body]

I build the Fortran-dev branch with C, and it failed because of the 
previous Boolean issue, but also because decl-ordering issue, addressed 
by the attachment. I initially thought that the latter is a merge issue, 
but as it isn't, I have now fixed it first on the trunk – and then 
backported the fix to the Fortran-dev branch.

Trunk commit: Rev. 189740.
Fortran-branch: Rev. 189741.

Tobias

["commit.diff" (text/x-patch)]

Index: gcc/fortran/resolve.c
===================================================================
--- gcc/fortran/resolve.c	(Revision 189738)
+++ gcc/fortran/resolve.c	(Arbeitskopie)
@@ -257,9 +257,10 @@ resolve_formal_arglist (gfc_symbol *proc)
 
   for (f = proc->formal; f; f = f->next)
     {
-      sym = f->sym;
       gfc_array_spec *as;
 
+      sym = f->sym;
+
       if (sym == NULL)
 	{
 	  /* Alternate return placeholder.  */
Index: gcc/fortran/ChangeLog
===================================================================
--- gcc/fortran/ChangeLog	(Revision 189739)
+++ gcc/fortran/ChangeLog	(Arbeitskopie)
@@ -1,5 +1,10 @@
 2012-07-21  Tobias Burnus  <burnus@net-b.de>
 
+	* resolve.c (resolve_formal_arglist): Put variable
+	declaration before the first assignment.
+
+2012-07-21  Tobias Burnus  <burnus@net-b.de>
+
 	* trans-expr.c (gfc_conv_derived_to_class): Fix argument passed
 	to class_array_data_assign.
 


[prev in list] [next in list] [prev in thread] [next in thread]