Bug 21532

Summary: Remove duplicate const declaration in fast/js/const.html
Product: WebKit Reporter: Pam Greene (IRC:pamg) <pam>
Component: Tools / TestsAssignee: Adam Barth <abarth>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Fixes inadvertent const redeclaration and adds test for it darin: review+

Pam Greene (IRC:pamg)
Reported 2008-10-10 15:08:34 PDT
The test fast/js/const.js contains the lines const x = "RIGHT"; ... const x = "RIGHT", y = "RIGHT"; Redeclaring 'const x', which produces an error. (http://www.mozilla.org/js/language/js20-2002-04/core/variables.html discusses disallowing redefining consts; other places may too.) The test should be fixed to use a different variable name there, since that portion is not trying to test redefining constants. A separate test can be added for the latter.
Attachments
Fixes inadvertent const redeclaration and adds test for it (1.56 KB, patch)
2008-10-10 15:37 PDT, Pam Greene (IRC:pamg) 		darin: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Pam Greene (IRC:pamg)
Comment 1 2008-10-10 15:37:03 PDT
Created attachment 24276 [details] Fixes inadvertent const redeclaration and adds test for it Note that the const-redeclaration part of this doesn't currently do the right thing. That's bug 14611.
Darin Adler
Comment 2 2008-10-10 16:19:08 PDT
Comment on attachment 24276 [details] Fixes inadvertent const redeclaration and adds test for it r=me
Adam Barth
Comment 3 2008-10-12 13:44:40 PDT
I'll land this.
Adam Barth
Comment 4 2008-10-12 18:57:07 PDT
Fixed in r37543.
Note You need to log in before you can comment on or make changes to this bug.