Re: [Patch, Fortran] First ASSOCIATE patch and some questions / RFCs

[Daniel Kraft <d at domob dot eu>]

Tobias Burnus wrote:
> On 06/08/2010 04:38 PM, Daniel Kraft wrote:
> > here's an updated version of my patch addressing Tobias' comments.
> > Namely, I changed the co-indexed check to an error message
>
> s/co-indexed/coindexed/
> to be consistent with the Fortran standard. (At some point during the
> standardization, co-arrays and the other co-* terms lost their hyphen.)
>
> > I plan to re-regtest and commit tomorrow or Thursday, after the merge
> > is settled for some time. What's about the "variable definition
> > context" thing?  I think it should be ok to just remove the XXX
> > comment.  I can also change it to a FIXME / TODO, though, if we want
> > to investigate this further.
>
> I think it is sufficient to just remove the XXX. Thanks for the patch!

Committed with these changes and an added test-case for the coindexed 
error as rev. 160550.

> After committal, can you also update
> http://gcc.gnu.org/wiki/GFortran#news and 
> http://gcc.gnu.org/wiki/Fortran2003Status (No->Partial)?

Yes, I'll do that now.  And then work on the documentation and 
dump-parse-tree patch ;)

Yours,
Daniel

--
http://www.pro-vegan.info/
--
Done:  Arc-Bar-Cav-Ran-Rog-Sam-Tou-Val-Wiz
To go: Hea-Kni-Mon-Pri