Closed
Bug 767660
Opened 10 years ago
Closed 10 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•10 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•10 years ago
|
Whiteboard: js-triage-needed → [js:t]
Comment 2•10 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•10 years ago
|
||
Attachment #636128 -
Attachment is obsolete: true
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed
Assignee | ||
Updated•10 years ago
|
Attachment #636468 -
Flags: review+
Assignee | ||
Updated•10 years ago
|
Attachment #636468 -
Flags: review+
![]() |
Reporter | |
Comment 4•10 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•10 years ago
|
||
http://hg.mozilla.org/integration/mozilla-inbound/rev/2ae1e57c5851
Target Milestone: --- → mozilla16
Updated•10 years ago
|
Keywords: checkin-needed
Comment 6•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/2ae1e57c5851
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
![]() |
Reporter | |
Updated•10 years ago
|
Flags: in-testsuite+
![]() |
Reporter | |
Comment 7•10 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
•