This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [patch, fortran, doc] Improve docs of OpenMP, add contributors
- From: Tobias Burnus <burnus at net-b dot de>
- To: Daniel Franke <franke dot daniel at gmail dot com>
- Cc: fortran at gcc dot gnu dot org, gcc-patches at gcc dot gnu dot org
- Date: Thu, 24 Jan 2008 18:14:30 +0100
- Subject: Re: [patch, fortran, doc] Improve docs of OpenMP, add contributors
- References: <200801232022.43943.franke.daniel@gmail.com>
Daniel Franke wrote:
Tested doc, dvi and html targets on i686-pc-linux-gnu. Ok for trunk?
+The OpenMP (Open Multi-Processing) is an application programming
I think this should simply be "OpenMP [...] is" without the "The".
+To enable processing of the OpenMP directives @code{!$omp} in free
+form, @code{c$omp}, @code{*$omp} and @code{!$omp} directives in fixed
+form, @code{!$} conditional compilation sentinels in free form and
+@code{c$}, @code{*$} and @code{!$} sentinels in fixed form,
I would disperse some "the":
"To enable the processing of the OpenMP directive @{!$omp} in free-form
source code; the @code{c$omp}, @code{*$omp} and @code{!$omp} directives
in fixed form; the code{!$} conditional compilation sentinels in free
form; and the @code{c$}, @code{*$} and @code{!$} sentinels in fixed form,"
Otherwise it looks OK. Thanks for the patch.
Tobias