306633 - JSOPTION_STRICT: Compiler warnings are not reported from within eval()

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Michael Daumling]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7.10) Gecko/20050716 Firefox/1.0.6
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7.10) Gecko/20050716 Firefox/1.0.6

When setting JSOPTION_STRICT, all compiler warnings should be reported, but
warnings from within a scripted eval() call  are not. In jsscan.c, line 708, the
statement if (cx->interplevel != 0) turns off error erporting.

Reproducible: Always

Steps to Reproduce:
1. Launch jshell
2. type in: options ("strict")
3. set a C++ breakpoint at my_ErrorReporter in js.c
4. type in: eval ('super=5')

Actual Results:  
Breakpoint is not hit

Expected Results:  
breakpoint should be hit

Comment 1

[Michael Daumling]

Attachment: Patch provided by Brendan Eich.

Fixes the bug - I get compiler warnings from within an eval()

Comment 2

[Michael Daumling]

Bug found in Mozilla 1.8 branch.

Comment 3

[Brendan Eich [:brendan]]

Easy fix, should get into 1.8 at this stage.  Anyone disagree?

/be

Comment 4

[Mike Shaver (:shaver -- probably not reading bugmail closely)]

Comment on attachment 194487 [details] [diff] [review]
Patch provided by Brendan Eich.

sr=shaver

Comment 5

[Brendan Eich [:brendan]]

Comment on attachment 194487 [details] [diff] [review]
Patch provided by Brendan Eich.

Getting this in for 1.8b4.

/be

Comment 6

[Aiko]

Somebody should mark bug 247077 as a duplicate.

Comment 7

[Brendan Eich [:brendan]]

*** Bug 247077 has been marked as a duplicate of this bug. ***

Comment 8

[Brendan Eich [:brendan]]

I fixed this the other day, trunk and branch.  Thanks for the report and review,
Michael!

/be

Comment 9

[Bob Clary [:bc] (inactive)]

Checking in regress-306633.js;
/cvsroot/mozilla/js/tests/js1_5/Regress/regress-306633.js,v  <--  regress-306633.js
initial revision: 1.1
done