On Mon, Dec 12, 2016 at 02:19:47PM -0700, Sandra Loosemore wrote:
> One small grammar nit at the end:
> 
> > > +It is recommended that code which is intended to be portable use the
> > > +@code{_Float16} type defined by ISO/IEC TS 18661-3:2015
> > > +(@xref{Floating Types}).
> 
> Either s/which/that/ or rewrite the beginning of the sentence as "It is
> recommended that portable code use..."
> 
> The patch is OK to commit with that fixed.

I've noticed
../../gcc/doc/extend.texi:1060: warning: `.' or `,' must follow @xref, not )
warning with this.  The following patch adjusts it similarly how other
(@xref{...}.) look like.  @xref actually emits
See Section 6.11 [Floating Types], page 415
in pdf or
*Note Floating Types::
in info.
Tested with building gcc.info and gcc.pdf and checking what is in there.

Ok for trunk?

2016-12-15  Jakub Jelinek  <ja...@redhat.com>

        * doc/extend.texi: Add a dot after @xref{...}.

--- gcc/doc/extend.texi.jj      2016-12-14 20:28:12.000000000 +0100
+++ gcc/doc/extend.texi 2016-12-15 10:56:14.470702563 +0100
@@ -1057,7 +1057,7 @@ implements conversions between @code{__f
 calls.
 
 It is recommended that portable code use the @code{_Float16} type defined
-by ISO/IEC TS 18661-3:2015 (@xref{Floating Types}).
+by ISO/IEC TS 18661-3:2015.  (@xref{Floating Types}.)
 
 @node Decimal Float
 @section Decimal Floating Types


        Jakub

Reply via email to