46173 - mimetpfl fails to detect end of tags.

Status: RESOLVED FIXED

(MailNews Core :: MIME, defect, P3)

Description

[Daniel Bratell]

There are code in mimetpfl to detect when a tag starts, but there's no code to 
detect where it ends making the full line being treated like being inside a tag 
which prevents SPACE ->  -conversion, which causes multiple consecutive 
whitespace to collapse.

Comment 1

[Daniel Bratell]

Fixed this (and a glitch in bug 39728). Also factored out some duplicated code. 
There are more code that are duplicated for the flowed and fixed cases, but that 
will have to wait.

Comment 2

[Daniel Bratell]

Attachment: Fix for this bug and the glitch in bug 39728

Comment 3

[Daniel Bratell]

Rich, could you review this? What I did was to keep track of if we are inside a 
tag or not. There are also some code to detect if we are inside a quote inside a 
tag so that we don't think the tag has ended when it hasn't. For instance in 
<a href="hjh:>>>" we are still in   tag>. 

The fix for bug 39728 is to move the initializing of a variable outside the loop 
so that it doesn't get reset every iteration.

Comment 4

[rhp (gone)]

Looks good to me - r: rhp.

Comment 5

[Daniel Bratell]

The related bug is no longer bug 39728 but rather bug 46188.

Comment 6

[Daniel Bratell]

Checked in. 

Comment 7

[esther]

changing qa