Patchwork [wwwdocs] Fix grammar in Android entry of 4.6 release notes

login
register
mail settings
Submitter Gerald Pfeifer
Date Nov. 6, 2010, 9:10 p.m.
Message ID <alpine.LNX.2.00.1011062150220.10295@acrux.dbai.tuwien.ac.at>
Download mbox | patch
Permalink /patch/70336/
State New
Headers show

Comments

Gerald Pfeifer - Nov. 6, 2010, 9:10 p.m.
I must have missed this when this entry was added in May; the patch
below, which I just applied, adds "the" in a couple of places.  It
also removes one use of "Linux" where it's not necessary (since this
is the Linux subsection).

Maxim, I am wondering, since when not refering to just the kernel we
are supposed to use GNU/Linux, should this be changed here, or is the
kernel the only thing Android uses from the GNU side?  Or, how about
just changing this to

   <h3 id="android">Android</h3>

?

Gerald
Maxim Kuvyrkov - Nov. 7, 2010, 7:21 a.m.
On 11/07/2010 12:10 AM, Gerald Pfeifer wrote:
> I must have missed this when this entry was added in May; the patch
> below, which I just applied, adds "the" in a couple of places.  It
> also removes one use of "Linux" where it's not necessary (since this
> is the Linux subsection).

Thank you for fixing the entry.

>
> Maxim, I am wondering, since when not refering to just the kernel we
> are supposed to use GNU/Linux, should this be changed here, or is the
> kernel the only thing Android uses from the GNU side?
>  Or, how about
> just changing this to
>
>     <h3 id="android">Android</h3>
>
> ?

Kernel is not the only thing Android uses from GNU, but it's close to 
that.  I think changing the header to "Android" would look better, I 
can't say why I didn't write so in the first place.

Thanks,

Patch

Index: changes.html
===================================================================
RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-4.6/changes.html,v
retrieving revision 1.55
diff -u -3 -p -r1.55 changes.html
--- changes.html	5 Nov 2010 20:58:41 -0000	1.55
+++ changes.html	6 Nov 2010 20:53:36 -0000
@@ -342,12 +342,13 @@  supported on several Xilinx Spartan and 
 
 <h3 id="Linux">Linux</h3>
   <ul>
-    <li>GCC now supports Bionic C library and provides convenient way of
-      building native libraries and applications for Android platform.
-      Refer to documentation of <code>-mandroid</code>
+    <li>GCC now supports the Bionic C library and provides a convenient
+      way of building native libraries and applications for the Android
+      platform.
+      Refer to the documentation of the <code>-mandroid</code>
       and <code>-mbionic</code> options for details on building native code
-      for Android platform.
-      At the moment, Android support is enabled only for ARM Linux target.</li>
+      for the Android platform.
+      At the moment, Android support is enabled only for ARM.</li>
   </ul>
 
 <h2>Documentation improvements</h2>