Closed
Bug 767660
Opened 13 years ago
Closed 13 years ago
"Assertion failure: pnu->isUsed(),"
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
VERIFIED
FIXED
mozilla16
People
(Reporter: gkw, Assigned: Benjamin)
References
Details
(Keywords: assertion, regression, testcase, Whiteboard: [js:t])
Attachments
(2 files, 1 obsolete file)
|
90 bytes,
text/plain
|
Details | |
|
4.48 KB,
patch
|
gkw
:
review+
|
Details | Diff | Splinter Review |
The attached testcase asserts js debug shell on m-i changeset cb74a377095a without any CLI flag arguments at Assertion failure: pnu->isUsed(), when the testcase passed in as a CLI argument.
autoBisect shows this is probably related to the following changeset:
The first bad revision is:
changeset: 97130:70acf458a32d
user: Benjamin Peterson
date: Sat Jun 02 20:16:24 2012 +0200
summary: Bug 759498: fix defaults when arguments are bound as functions, r=jorendorff
|
Assignee | |
Comment 1•13 years ago
|
||
That was quite a subtle bug! I have no idea why the defaults provoked it; it seems to be a very old bug.
Assignee: general → bpeterson
Attachment #636128 -
Flags: review?(jorendorff)
|
||
Updated•13 years ago
|
Whiteboard: js-triage-needed → [js:t]
|
||
Comment 2•13 years ago
|
||
Comment on attachment 636128 [details] [diff] [review]
fix and assurance that it won't happen again
Agreed. That's really strange.
Attachment #636128 -
Flags: review?(jorendorff) → review+
|
|
Assignee | |
Comment 3•13 years ago
|
||
Attachment #636128 -
Attachment is obsolete: true
|
|
Assignee | |
Updated•13 years ago
|
Keywords: checkin-needed
|
|
Assignee | |
Updated•13 years ago
|
Attachment #636468 -
Flags: review+
|
|
Assignee | |
Updated•13 years ago
|
Attachment #636468 -
Flags: review+
|
Reporter | |
Comment 4•13 years ago
|
||
Comment on attachment 636468 [details] [diff] [review]
a few cosmetic changes
Bringing over r+ to avoid potential confusion that this may be a lingering patch in the future.
Landing soon.
Attachment #636468 -
Flags: review+
|
|
Reporter | |
Comment 5•13 years ago
|
||
Target Milestone: --- → mozilla16
|
||
Updated•13 years ago
|
Keywords: checkin-needed
|
||
Comment 6•13 years ago
|
||
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
|
|
Reporter | |
Updated•13 years ago
|
Flags: in-testsuite+
|
|
Reporter | |
Comment 7•13 years ago
|
||
A type of test for this bug has already been landed because it is already marked in-testsuite+ -> VERIFIED.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•