Closed Bug 1481555 Opened 2 years ago Closed 2 years ago
Update lldbinit to summarise ns
46 bytes, text/x-phabricator-request
|Details | Review|
The current summary string doesn't work since nsAtom changed. https://searchfox.org/mozilla-central/source/.lldbinit#25
Tom, do you know if we can just change that line to call a method on the atom instead, or does calling methods imply that we have to move to a more complicated printer for nsAtom?
Not quite sure this is right, but it seems to give the right results. Who could review this?
FWIW that patch looks reasonable to me. I don't know the answer to Nathan's question. However the patch seems to make it moot.
Comment on attachment 8998369 [details] Bug 1481555: Use lldb's python API to summarise nsAtoms. Nathan Froyd [:froydnj] has approved the revision.
Attachment #8998369 - Flags: review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/ff3a0784588f Use lldb's python API to summarise nsAtoms. r=froydnj
You need to log in before you can comment on or make changes to this bug.