Closed Bug 1180985 Opened 9 years ago Closed 9 years ago

JS GDB pretty-printers: need support for GCCellPtr

Categories

(Core :: JavaScript Engine, defect)

All
Linux
defect
Not set
minor

Tracking

()

RESOLVED FIXED
mozilla42
Tracking Status
firefox42 --- fixed

People

(Reporter: jimb, Assigned: jimb)

Details

Attachments

(1 file, 1 obsolete file)

Attached patch gdb-GCCellPtr.patch (obsolete) — Splinter Review
      No description provided.
Attachment #8630253 - Flags: review?(ttromey)
Assignee: nobody → jimb
Wrong patch, sorry.
Attachment #8630253 - Attachment is obsolete: true
Attachment #8630253 - Flags: review?(ttromey)
Attachment #8630486 - Flags: review?(ttromey)
Comment on attachment 8630486 [details] [diff] [review]
Implement a JS GDB pretty-printer for JS::GCCellPtr.

Review of attachment 8630486 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good.
Attachment #8630486 - Flags: review?(ttromey) → review+
Severity: normal → minor
Flags: in-testsuite+
Target Milestone: --- → mozilla42
sorry had to back this out in https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&revision=b65c048414f7 since one of this changes might have caused bug 1188799 at least started with this pushes
and re-checkedin since this turned out to be innocent. sorry Jim :(
https://hg.mozilla.org/mozilla-central/rev/6e93bd79e848
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: