Page MenuHomePhabricator

[lldb] Remove FormattersContainer's name member
ClosedPublic

Authored by teemperor on Jul 20 2020, 5:29 AM.

Details

Summary

FormattersContainer currently has an unused m_name member. Usually LLDB allows giving objects names,
but for the FormattersContainer it seems excessive. There are only 4 FormattersContainer variables in LLDB
and they are not usually passed around, so one can always just go up a few frames when debugging to find
out which FormattersContainer you're dealing with.

Diff Detail

Event Timeline

teemperor created this revision.Jul 20 2020, 5:29 AM
mib accepted this revision.Jul 20 2020, 6:34 AM

LGTM!

This revision is now accepted and ready to land.Jul 20 2020, 6:34 AM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptJul 21 2020, 4:55 AM