Re: TrimEmptyElement(): don't remove empty TR, imply required TD instead

At 6:13 PM -0500 8/15/99, Advocate wrote:
>> Correction to last patch.  Where the variable "row" is mentioned,
>> replace with "element".  Side effect of where the patch was originially
>> applied before it was moved.  These would be in the ReportWarning and
>> InsertNode arguments.
>
>I see that a test case is preferred, even if a patch is supplied.
>Very well.  When given the input:

I prefer test cases, but that's just me - you guys know HTML better than I
do <grin>.

Is this the same bug as Randi Waki reported in "7-Jul-99 bug: <tr> without
<td> is deleted" :

<http://lists.w3.org/Archives/Public/html-tidy/1999JulSep/0090.html>

>MacTidy outputs these errors (only because it is the platform edition
>most available to me at the moment):

><table border="1" summary="Demonstration of HTML Tidy's erroneous \
>elimination of empty rows.">

>(Hmm, I seem to have found a bug in MacTidy 1.0b2, specifically that
>trailing slash in the summary.  That can't be right, can it?  No, it
>isn't being inserted by my mailer, I've double-checked that already.

Well you have apparently found a bug (I reproduced it here) - the question
is it Mac OS specific? I don't modify the guts of Dave's code, so the only
thing I can think of platform specific line termination library code
causing a problem.

Would like to see what happens on other platforms.

>Additionally, when output is set to an existing file, it overwrites
>it instead of removing the old file before writing the new one; thus,
>if the old file was bigger, you'll have residual text from it at the
>end of the new file.)

Sounds like a bug - I was not aware of that behaviour (I'm using StdIO and
SIOUX). I should be able to fix that.

Thanks for the report.

Regards, Terry

Received on Monday, 16 August 1999 03:17:47 UTC