Re: a push @lines2 in pass1

Adrian Aichner <[email protected]> 05 Feb 2003 22:05:14 +0100
Newsgroups gmane.comp.tex.texi2html.devel
Organization Teradyne, Inc. -- Consumer Business Unit
Message-ID <[email protected]>
>>>>> "Dumas" == Dumas Patrice <[email protected]> writes:

    Dumas> Hi,
    Dumas> I found that on some code signed by APA (Adrian ?).

APA is me, yes.

    Dumas> There is:

    Dumas>     push (@lines2, ...)

    Dumas> line 2024 in pass1. I believe this is a typo and should be instead @lines.

2024 is what file?

Is this related to the following?

revision 1.21
date: 2003/02/03 15:33:23;  author: pertusus;  state: Exp;  lines: +1 -21
Remove handling of setref which is a TeX macro. The corresponding texinfo
macro is anchor.
Remove an unneeded protect_texi

I'm confused.

Oh, is it that?

                # APA: <table> implicitly ends paragraph, so let's do it
                # explicitly to keep our HTML stack in sync.
                if ($start =~ /\A\s*<table>/i)
                {
                    if ($html_element eq 'p')
                    {
                        push (@lines2, &debug("</p>\n", __LINE__));
                        &html_pop();
                    }
                }

@lines2 looks wrong to me too.

I did a lot towards making texi2html output valid HTML.

Please use http://validator.w3.org/file-upload.html
or some such when you change texi2html output.

Best regards,

Adrian

    Dumas> My tests shows that with this change adds quite a lot of /p before some
    Dumas> <tables>, which seems to confirm my finding.

    Dumas> I changed it in the cvs.

    Dumas> Pat

    Dumas> ---------------------------------------------------------------------
    Dumas> To unsubscribe, e-mail: [email protected]
    Dumas> For additional commands, e-mail: [email protected]



-- 
Adrian Aichner
 mailto:[email protected]
 http://www.xemacs.org/