[Edbrowse-dev] other bugs

[Karl Dahlke <eklhad@comcast.net>]

[-- Attachment #1: Type: text/plain, Size: 1399 bytes --]

The problem with using third party software is we sometimes find their  bugs, and then hope they fix them, or we work around them.
acid test hunts for corner cases so not surprising it uncovers bugs.
I already wrote about the tidy bug, not giving us the empty nodes, which test 0 expects to be there, and so I guess every other browser puts them there.
I sent this question to Geoff.
But turning to css, there are more problems.
The parser that Kevin found for me doesn't parse comments corectly.
If a comment contains a selector it is treated as a selector, not a comment, and it's all screwed up from there.
I reported this to jotform, no response yet.
Ok this one seems like an easy work around.
I don't need the comments preserved for any reason, so just strip them out with a preprocessing regexp.

css_string = css_string.replace(/\/\*(.|\s)*?\*\//g, '');

Yeah I know, it looks like line noise in the modem, but it's right, I'm pretty sure, and works on the modest test strings that I give it,
but it doesn't work on the style block that is part of acid test 0.
RangeError: regexp executor recursion limit
I used the nongreedy modifier to the wild card, so there really shouldn't be any recursion at all, so I don't get it.
I sent this to one of the duktape developers.

With all these issues hanging, I'm rather in a holding pattern as per the acid tests.

Karl Dahlke