code review 2039043: doc: various changes for validating HTML

doc: various changes for validating HTML
(Fixes missing tags, extra tags, missing attributes, and such found by tidy -e)

[bytbox, 2010-08-29 12:36:02]

Hello golang-dev@googlegroups.com (cc: golang-dev@googlegroups.com),

I'd like you to review this change.

[adg1, 2010-08-29 23:58:08]

LGTM

Thanks for these.

On 29 August 2010 22:36,  <bytbox@gmail.com> wrote:
> Reviewers: golang-dev_googlegroups.com,
>
> Message:
> Hello golang-dev@googlegroups.com (cc: golang-dev@googlegroups.com),
>
> I'd like you to review this change.
>
>
> Description:
> doc: various changes for validating HTML
> (Fixes missing tags, extra tags, missing attributes, and such found by
> tidy -e)
>
> Please review this at http://codereview.appspot.com/2039043/
>
> Affected files:
>  M doc/codereview_with_mq.html
>  M doc/gccgo_install.html
>  M doc/go_for_cpp_programmers.html
>  M doc/install.html
>  M doc/root.html
>
>
> Index: doc/codereview_with_mq.html
> ===================================================================
> --- a/doc/codereview_with_mq.html
> +++ b/doc/codereview_with_mq.html
> @@ -36,11 +36,11 @@
>  <pre>
>  [hooks]
>  # Prevent "hg pull" if MQ patches are applied.
> -prechangegroup.mq-no-pull = ! hg qtop > /dev/null 2>&1
> +prechangegroup.mq-no-pull = ! hg qtop > /dev/null 2>&amp;1
>  # Prevent "hg push" if MQ patches are applied.
> -preoutgoing.mq-no-push = ! hg qtop > /dev/null 2>&1
> +preoutgoing.mq-no-push = ! hg qtop > /dev/null 2>&amp;1
>  # Prevent "hg update" if MQ patches are applied.
> -preupdate.mq-no-update = ! hg qtop > /dev/null 2>&1
> +preupdate.mq-no-update = ! hg qtop > /dev/null 2>&amp;1
>  </pre>
>
>  <h2>Making a change</h2>
> Index: doc/gccgo_install.html
> ===================================================================
> --- a/doc/gccgo_install.html
> +++ b/doc/gccgo_install.html
> @@ -396,9 +396,9 @@
>  guarantee that it will not change in the future. It is more useful as a
>  starting point for real Go code than as a regular procedure.
>
> -<h2 id="RTEMS Port">RTEMS Port</h2>
> +<h2 id="RTEMS_Port">RTEMS Port</h2>
>  <p>
> -The <code>gccgo</code> compiler has been ported to <a
> href="http://www.rtems.com/"
> +The <code>gccgo</code> compiler has been ported to <a
> href="http://www.rtems.com/">
>  <code>RTEMS</code></a>. <code>RTEMS</code> is a real-time executive
>  that provides a high performance environment for embedded applications
>  on a range of processors and embedded hardware. The current
> <code>gccgo</code>
> Index: doc/go_for_cpp_programmers.html
> ===================================================================
> --- a/doc/go_for_cpp_programmers.html
> +++ b/doc/go_for_cpp_programmers.html
> @@ -198,7 +198,7 @@
>  Since semicolons do end statements, you may continue using them as in
>  C++.  However, that is not the recommended style.  Idiomatic Go code
>  omits unnecessary semicolons, which in practice is all of them other
> -than the initial <for> loop clause and cases where you want several
> +than the initial <code>for</code> loop clause and cases where you want
> several
>  short statements on a single line.
>
>  <p>
> Index: doc/install.html
> ===================================================================
> --- a/doc/install.html
> +++ b/doc/install.html
> @@ -293,7 +293,6 @@
>        <code>386</code> (32-bit x86), and
>        <code>arm</code> (32-bit ARM, an incomplete port).
>        The valid combinations of <code>$GOOS</code> and <code>$GOARCH</code>
> are:
> -       <p>
>        <table cellpadding="0">
>        <tr>
>        <th width="50"><th align="left" width="100"><code>$GOOS</code></th>
> <th align="left" width="100"><code>$GOARCH</code></th> <th
> align="left"></th>
> Index: doc/root.html
> ===================================================================
> --- a/doc/root.html
> +++ b/doc/root.html
> @@ -1,6 +1,6 @@
>  <!-- The Go Programming Language -->
>
> -<script>
> +<script type="text/javascript">
>   // On the frontpage we hide the header and navigation elements that other
>   // pages have.
>   document.getElementById('generatedHeader').style.display = 'none';
>
>
>

[adg, 2010-08-29 23:58:37]

*** Submitted as http://code.google.com/p/go/source/detail?r=cacf292f4a12 ***

doc: various changes for validating HTML
(Fixes missing tags, extra tags, missing attributes, and such found by tidy -e)

R=golang-dev, adg1, adg
CC=golang-dev
http://codereview.appspot.com/2039043

Committer: Andrew Gerrand <adg@golang.org>