Closed Bug 532652 Opened 15 years ago Closed 15 years ago

Compile-time errors do not trigger ErrorReporter

Categories

(Core :: JavaScript Engine, defect)

x86
Linux
defect
Not set
major

Tracking

()

RESOLVED DUPLICATE of bug 451732

People

(Reporter: lkundrak, Unassigned)

References

Details

Attachments

(1 file)

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
Blocks: js1.8
Comment on attachment 415856 [details] [diff] [review]
Patch that fixed things for me

jorendorff@, could you please review this?
Attachment #415856 - Flags: review?(jorendorff)
Attachment #415856 - Attachment is patch: true
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
Attachment #415856 - Flags: review?(jorendorff)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: