Closed
Bug 532652
Opened 15 years ago
Closed 15 years ago
Compile-time errors do not trigger ErrorReporter
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
RESOLVED
DUPLICATE
of bug 451732
People
(Reporter: lkundrak, Unassigned)
References
Details
Attachments
(1 file)
|
1.22 KB,
patch
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.1.5) Gecko/20091105 Fedora/3.5.5-1.fc12 Firefox/3.5.5
Build Identifier: js-1.80-rc1
Regression against 1.7.0. boo() gets never called:
ErrorReporter () { boo() }
JS_SetErrorReporter (..., ErrorReporter);
JS_EvaluateScript (... "foo = \"bar" ...);
This breaks perl JavaScript::SpiderMonkey's test suite, 005error.t test.
Reproducible: Always
|
Reporter | |
Comment 1•15 years ago
|
||
|
|
Reporter | |
Comment 2•15 years ago
|
||
Comment on attachment 415856 [details] [diff] [review]
Patch that fixed things for me
jorendorff@, could you please review this?
Attachment #415856 -
Flags: review?(jorendorff)
|
||
Updated•15 years ago
|
Attachment #415856 -
Attachment is patch: true
|
|
||
Comment 3•15 years ago
|
||
Please look for fixes in newer repos and dup accordingly. Then the patch can be back-ported for minimal re-work cost and delta between repos.
/be
Status: UNCONFIRMED → RESOLVED
Closed: 15 years ago
Resolution: --- → DUPLICATE
|
||
Updated•15 years ago
|
Attachment #415856 -
Flags: review?(jorendorff)
You need to log in
before you can comment on or make changes to this bug.
Description
•