46 bytes, text/x-phabricator-request
|Details | Review|
This makes easier to keep the existing error reporting working in bug 1422225, and it's a prerequisite for bug 1469173 anyway.
This should make it easier to report errors, and also reduce codesize. The reason this was so generic is that error reporting was unconditionally enabled and was super-hot, but now that's no longer the case after bug 1452143, so we can afford the virtual call in the "error reporting enabled" case. This opens the possibility of simplifying a lot the error setup as well, though this patch doesn't do it.
Comment on attachment 8986551 [details] Bug 1469957: Move the error reporter into ParserContext. r=xidorn Xidorn Quan [:xidorn] UTC+10 has approved the revision. https://phabricator.services.mozilla.com/D1734
Attachment #8986551 - Flags: review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/mozilla-inbound/rev/4072740124c8 Move the error reporter into ParserContext. r=xidorn
You need to log in before you can comment on or make changes to this bug.