desktop-packages team mailing list archive

[Christopher-leary <908508@xxxxxxxxxxxxxxxxxx>]

(In reply to Landry Breuil from comment #18)
> Another annoyance was the landing of bug #673188, which right now #errors
> out if ENABLE_YARR_JIT is unset (see RegExpObject.cpp, function
> checkSyntax()), pretty unfriendly. So far i've replaced it by a return true,
> but it's of course ugly.

It's required for lazy regular expression compilation -- you have to
detect errors in the syntax immediately for error reporting purposes.
Continuing to use the YARR parser to report errors when you're using the
PCRE backend could result in a mismatch where the PCRE parser believes
there to be an error in the pattern that YARR did not detect, which
would result in incorrect semantics (because you have no way of
reporting the error when you're doing the lazy compilation).

Sorry, I should have put a comment in if I didn't!

-- 
You received this bug notification because you are a member of Desktop
Packages, which is subscribed to thunderbird in Ubuntu.
https://bugs.launchpad.net/bugs/908508

Title:
  Firefox/Thunderbird 10 FTBFS on powerpc in js/src/yarr/pcre

Status in The Mozilla Firefox Browser:
  In Progress
Status in “firefox” package in Ubuntu:
  Triaged
Status in “thunderbird” package in Ubuntu:
  Triaged

Bug description:
  Debian has the fix, so I'll add it to the branch and it should work
  with the next upload.

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/908508/+subscriptions