Re: [AD] Makedoc Wars: the return of backwards compatibility

[ Thread Index | Date Index | More lists.liballeg.org/allegro-developers Archives ]



On Sun, 5 May 2002, Grzegorz Adam Hankiewicz wrote:

> Hi.
>
> This is a modified version of Sven's patch, which brings backwards
> compatibility (with some warnings through stdout). I've tested it on
> the documentation of 4.0.0 and seems to be ok.  Sven, Stephan, could
> you both test this and see if it really works? If yes, I will apply it.

I tried it on almost all allegro._tx files starting with Allegro 2.2 and
it works without a problem. But I found that those old-formatted files
have errors in them (@xref pointing to @node, which seems to be illegal or
at least unsupported), so I propose this small patch to makedoc.c:

--- makedoc.c.orig	Sun May  5 13:52:05 2002
+++ makedoc.c	Sun May  5 14:10:29 2002
@@ -332,7 +332,7 @@
 	    _add_line("", INDEX_FLAG);
 	 else if (strincmp(buf+1, "node ") == 0) {
 	    _add_toc_line(buf+6, NULL, 0, line, 1, 0, 0);
-	    _add_line(buf+6, NODE_FLAG);
+	    _add_line(buf+6, NODE_FLAG | HTML_FLAG);
 	 }
 	 else if (strincmp(buf+1, "hnode ") == 0) {
 	    _add_toc_line(buf+7, NULL, 0, line, 1, 0, 0);

I don't know exactly why @node is in makedoc, because it is almost unused
(even in those old files). The only usage in latest documentation is as
'@node gui font' section. It should be removed or changed to @hnode.
@node itself should be made deprecated.

Have a nice day.

Stepan Roh



Mail converted by MHonArc 2.6.19+ http://listengine.tuxfamily.org/