bug-texinfo
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Patch against texinfo-4.0b


From: jerome . zago
Subject: Patch against texinfo-4.0b
Date: Tue, 25 Dec 2001 23:04:57 +0100
User-agent: Mutt/1.2.5i

Merry Christmas !

First, I have to assume that the latest released version of texinfo is
4.0b, since it's what is shipped by Redhat Linux 7.2 and the closest
release I can find from ftp://ftp.gnu.org/gnu/texinfo/ is 4.0 (which is
quite dated).

I stepped across this bug by reading
http://gcc.gnu.org/install/configure.html with Konqueror 2.2.1, which
doesn't render the page correctly. I discovered it was because of
unclosed <a> tags, which was confirmed by http://validator.w3c.org/.
Indeed, the end tag of the A element is required
(http://www.w3.org/TR/html4/struct/links.html#h-12.2). Hence this patch.

After applying this patch, the tested page becomes
http://insomnia.nexen.net/tmp/configure.html. W3C's validator doesn't
report the <a> errors anymore, although it would appreciate a good
character encoding (iso-8859-1 for instance) and a proper DOCTYPE
declaration, like this for instance :

<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN"
   "http://www.w3.org/TR/html4/loose.dtd";>
<HTML>
<HEAD>
<TITLE>Free your mind</TITLE>
<META http-equiv="Content-Type" content="text/html; charset=ISO-8859-1">
</HEAD>
<BODY>
</BODY>
</HTML>

----------------------------- CUT HERE ------------------------------
diff -uNr texinfo-4.0b.orig/ChangeLog texinfo-4.0b/ChangeLog
--- texinfo-4.0b.orig/ChangeLog Tue May  1 21:46:05 2001
+++ texinfo-4.0b/ChangeLog      Tue Dec 25 22:27:01 2001
@@ -1,3 +1,7 @@
+2001-12-25    <address@hidden>
+
+       * makeinfo/sectioning.c: </a> is required even for <a name="[...]">
+
 2001-05-01    <address@hidden>
 
        * configure.in: 4.0b
--- texinfo-4.0b.orig/makeinfo/sectioning.c     Tue May  1 18:28:28 2001
+++ texinfo-4.0b/makeinfo/sectioning.c  Tue Dec 25 21:46:35 2001
@@ -420,7 +420,7 @@
   if (!current_node || !*current_node)
     {
       starting_pos = output_paragraph + output_paragraph_offset;
-      add_word_args ("<a name=\"TOC%d\">", toc_ref_count++);
+      add_word_args ("<a name=\"TOC%d\"></a>", toc_ref_count++);
       toc_anchor = substring (starting_pos + 9,
                               output_paragraph + output_paragraph_offset);
     }
----------------------------- CUT HERE ------------------------------



reply via email to

[Prev in Thread] Current Thread [Next in Thread]