[PATCH, Fortran] [committed] PING: Extend gfc_code internal documentation

Daniel Kraft d@domob.eu
Sat Jun 28 15:13:00 GMT 2008


Tobias Burnus wrote:
> Hi Daniel,
> 
> Daniel Kraft wrote:
>> Are there any comments on that one?
> Looks OK. Thanks for your fantastic work. Sorry that we don't do timely 
> reviews, but somehow real life continues to interfere :-(
> Besides, I have the feeling that you know by now more about several part 
> of gfortran than some of us does ;-)
> 
> 
> Some comments from a non-native speaker:
> 
> +This statement-block will be represented in the internal gfortran tree 
> like
> +this, were the horizontal link-chains are those induced by the @code{next}
> 
> 
> Instead of "like this" I'd use "as follows".
> 
> +next @code{ELSEIF} or @code{ELSE} block if there's a chain of them.
> +
> +@subsection Loops
> 
> 
> I'd add another empty line to make it easier to find the @subsections.
> 
> +@code{op} member set to @code{EXEC_IF}.  This structure's @code{block}
> +member must point to another @code{gfc_code} node being the header of the
> +if-block.  This header's @code{op} member must be set to 
> @code{EXEC_IF}, too,
> 
> Instead of "being", I'd use "which is".

Checked in with the suggested changes as revision 137226.

Thanks,
Daniel

-- 
Done:     Bar-Sam-Val-Wiz, Dwa-Elf-Hum-Orc, Cha-Law, Fem-Mal
Underway: Ran-Gno-Neu-Fem
To go:    Arc-Cav-Hea-Kni-Mon-Pri-Rog-Tou



More information about the Fortran mailing list