Status
()
People
(Reporter: gkw, Assigned: Benjamin)
Tracking
(Blocks: 1 bug, {assertion, regression, testcase})
Firefox Tracking Flags
(Not tracked)
Details
(Whiteboard: [js:t])
Attachments
(2 attachments, 1 obsolete attachment)
|
90 bytes,
text/plain
|
Details | |
|
4.48 KB,
patch
|
gkw
:
review+
|
Details | Diff | Splinter Review |
Created attachment 636029 [details] testcase 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•7 years ago
|
||
Created attachment 636128 [details] [diff] [review] fix and assurance that it won't happen again 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•7 years ago
|
Whiteboard: js-triage-needed → [js:t]
|
||
Comment 2•7 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•7 years ago
|
||
Created attachment 636468 [details] [diff] [review] a few cosmetic changes
Attachment #636128 -
Attachment is obsolete: true
|
|
(Assignee) | |
Updated•7 years ago
|
Keywords: checkin-needed
|
|
(Assignee) | |
Updated•7 years ago
|
Attachment #636468 -
Flags: review+
|
|
(Assignee) | |
Updated•7 years ago
|
Attachment #636468 -
Flags: review+
|
(Reporter) | |
Comment 4•7 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•7 years ago
|
||
http://hg.mozilla.org/integration/mozilla-inbound/rev/2ae1e57c5851
Target Milestone: --- → mozilla16
|
||
Updated•7 years ago
|
Keywords: checkin-needed
|
||
Comment 6•7 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/2ae1e57c5851
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
|
|
(Reporter) | |
Updated•7 years ago
|
Flags: in-testsuite+
|
|
(Reporter) | |
Comment 7•6 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
•