Closed Bug 343675 Opened 18 years ago Closed 18 years ago

Allow |function delete()| and similar

Categories

(Core :: JavaScript Engine, defect)

defect
Not set
normal

Tracking

()

VERIFIED FIXED

People

(Reporter: mrbkap, Assigned: igor)

References

Details

(Keywords: verified1.8.1)

Attachments

(1 file, 1 obsolete file)

Blocks: 351515
Attached patch Implementation v1 (obsolete) (deleted) — Splinter Review
Assignee: general → igor.bukanov
Status: NEW → ASSIGNED
Attachment #236997 - Flags: review?(brendan)
Comment on attachment 236997 [details] [diff] [review] Implementation v1 Looks good, sorry I didn't move on this sooner. I will update the ES4 wiki. /be
Attachment #236997 - Flags: review?(brendan) → review+
Attached patch Implementation v1b (deleted) — Splinter Review
Patch to commit that accounts for line movements due to unrelated changes in jsparse.c in CVS head.
Attachment #236997 - Attachment is obsolete: true
I committed the fix from comment 3 to the trunk: Checking in jsparse.c; /cvsroot/mozilla/js/src/jsparse.c,v <-- jsparse.c new revision: 3.227; previous revision: 3.226 done
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Attachment #237107 - Flags: approval1.8.1?
Brendan, is this required for the 181 branch?
Yes, see the bug this blocks, which is blocking1.8.1+. /be
Flags: blocking1.8.1?
Comment on attachment 237107 [details] [diff] [review] Implementation v1b a=schrep for drivers.
Attachment #237107 - Flags: approval1.8.1? → approval1.8.1+
Flags: blocking1.8.1? → blocking1.8.1+
Checking in regress-343675.js; /cvsroot/mozilla/js/tests/js1_5/LexicalConventions/regress-343675.js,v <-- regress-343675.js initial revision: 1.1 done
Flags: in-testsuite+
verified fixed 1.9 20060909 windows/mac*/linux
Status: RESOLVED → VERIFIED
needs 1.8 checkin
Whiteboard: [at risk]
I committed the patch from comment 3 to MOZILLA_1_8_BRANCH: Checking in jsparse.c; /cvsroot/mozilla/js/src/jsparse.c,v <-- jsparse.c new revision: 3.142.2.51; previous revision: 3.142.2.50 done
Keywords: fixed1.8.1
Whiteboard: [at risk]
verified fixed 1.8 20060914 windows/linux 1.9 20060914 windows/mac*/linux
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: