This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [Patch,wwwdocs] libquadmath: Mention in the news, add to changes.html


Hi Gerald,

thanks for the review.

On 11/18/2010 09:25 PM, Gerald Pfeifer wrote:
Index: index.html
===================================================================
+<dt>November 16, 2010</dt>
+<dd><a href="gcc-4.6/changes.html">GCC 4.6.0</a>  will include the
+<tt>libquadmath</tt>  library, which provides quad-precision mathematical
+functions on targets supporting the<tt>__float128</tt>  datatype. The
+library is used to provide on such targets the<tt>REAL(16)</tt>  type in GNU
+Fortran compiler. The library has been contributed by Fran&ccedil;ois-Xavier
+Coudert.</dd>

How about changing the last two sentences

   The library is used to provide on such targets the<tt>REAL(16)</tt>
   type in GNU Fortran compiler. The library has been contributed by
   Fran&ccedil;ois-Xavier Coudert.
to
   The library is used to provide the<code>REAL(16)</code>  type on
   such targets.  It has been contributed by Fran&ccedil;ois-Xavier
   Coudert.

I like the "on such targets", however, I think one should mention Fortran there; for instance: "The library is used to provide the <code>REAL(16)</code> type in GNU Fortran on such targets."


I think without mentioning Fortran, not only half of the message is lost but it also looks redundant and talking about "REAL(16)" might be unclear to non-Fortran users.

I have now committed gcc-4.6/changes.html with your proposed changes and index.html with your changes - except for the added "in GNU Fortran".

Tobias


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]