Yes, I like that more. Essentially we parse those tags, be it in PRE or non-PRE, but don't do anything with them, which makes sense, especially for <!-- -->.
Neither of them should have formatting effect, in theory the xml tag could drive the parser, but for our crude parser it makes currently no sense anyway.

> According to the specs only these tags are excluded in PRE:
> <!ENTITY % pre.exclusion "IMG|OBJECT|BIG|SMALL|SUB|SUP">

My understanding it is for content creators that is for a rss writer. Grr is a rss reader.

I attached examples with special treatment:



and without



The <pre> is precisely to show the content without a modification by the parser.

The xml-configuration snippets can be copy-pasted by the user to check them.

Added an example of special treatment inside <pre>

Looks to work fine at a first test, nice examples.

I wonder why you make a special treatment for comments inside PRE. Now I wonder if they should be displayed or not. Tags are treated.

According to the specs only these tags are excluded in PRE:
<!ENTITY % pre.exclusion "IMG|OBJECT|BIG|SMALL|SUB|SUP">

I am also unsure if
// <?xml.....?>

should have special treatments for PRE, I think not, but I couldn't certify the opposite.

I excluded special treatment for now until we understand it better or have a proof.

Please review the attached patch.

A brief description:
- removed extra -[retain] (memory leak);
- changed return values of -[foundOpeningTagName:attributes] and -[foundClosingTagName:attributes] to BOOL because otherwise any unknown tag is eaten by the -[parseHTML]... RedHat' feed has those xml-configuration snippets inside the <pre></pre>... see one of attached screenshots;
- added straightforward handling of <?..?> and <!-- --> because there is no reason to sieve them throw the code chunk seeking attribute name/value pairs;
- I didn't get the comment "// TODO: People use <br/> inside <pre>, parse that!" so left it intact;
- added the missed HTML entity 'apos'

Take a look at the nice ASCII now is visible.


(file #52797,