[PATCH, gomp] Adding projects/gomp to documentation

Scott Robert Ladd coyote@coyotegulch.com
Mon Oct 18 19:54:00 GMT 2004


Gerald Pfeifer wrote:
>> Index: htdocs/projects/gomp/index.html
>> ===================================================================
>> +<title>GOMP --An OpenMP implementation for GCC</title>
> 
> Here you'll probably want to use — and add a space before "An".

Ah. Some browsers used to have a problem with — in the title; some 
still render it wrong in body text (ugh!). I'll change it.

>> +<h1>Welcome to the home of <b>GOMP</b></h1>
> 
> Please omit the <b> here (as it is physical markup which we try to avoid).

Done.

> Please make the two GCC links relative, that is, "../../fortran" and
> "../../".

Done.

>> +<p>Traditionally, programmers have used architecture-specific
>> +methods to effectively program tightly-parallelized
>> +computers—high band-width clusters, SMP machines, or
> 
> Spaces around &mdash?

A stylistic "thing"; some people want spaces, some people don't. You 
wouldn't believe how many arguments I've had about my use of serial 
commas...

>> +<p>To remain relevent, free software development tools must
>                      ^
> relevant

Missed that one; fixed.

>> +<p>The GOMP release will include a support library, libGOMP, and
> Will the library really spelt that we (uppercase GOMP)?  If so, this
> is fine, else please provide the exact name.

I'm changing it to libgomp; some of the early documents have libgomp, 
some libGomp, and others libGOMP. "libgomp" is more consistent with 
existing practice.

> Also this new page is fine, with the changes I described above (or
> equivalent ones).

Thanks for the editorial pass.

> PS: I didn't find your name in our MAINTAINERS file, but it seems you
> have CVS write access.  Please add yourself to the "write after approval"
> section in the MAINTAINERS file.

Will do.

-- 
Scott Robert Ladd
site: http://www.coyotegulch.com
blog: http://chaoticcoyote.blogspot.com



More information about the Gcc-patches mailing list