Browse files

Patch that fixes the skipping of the HTML_PARSE_NOIMPLIED flag

For https://bugzilla.gnome.org/show_bug.cgi?id=642916

I just noticed that the HTML_PARSE_NOIMPLIED flag that you can pass to the
HTML-Parser methods doesn't do anything. Its intended purpose is to stop the
HTML-parser from forcibly adding a pair of html/body tags if the stream does
not contain any.

This is highly useful when you don't need this level of strictness.
Unfortunately, specifying it doesn't work, because the option is not
copied into the parsing context.
  • Loading branch information...
1 parent 24464be commit b91111b47599b9b07830db5ae2291739d22c384b @HedAurabesh HedAurabesh committed with veillard May 10, 2012
Showing with 4 additions and 0 deletions.
  1. +4 −0 HTMLparser.c
View
4 HTMLparser.c
@@ -6587,6 +6587,10 @@ htmlCtxtUseOptions(htmlParserCtxtPtr ctxt, int options)
ctxt->options |= HTML_PARSE_IGNORE_ENC;
options -= HTML_PARSE_IGNORE_ENC;
}
+ if (options & HTML_PARSE_NOIMPLIED) {
+ ctxt->options |= HTML_PARSE_NOIMPLIED;
+ options -= HTML_PARSE_NOIMPLIED;
+ }
ctxt->dictNames = 0;
return (options);
}

0 comments on commit b91111b

Please sign in to comment.