[all-commits] [llvm/llvm-project] 5c1542: [lldb] Remove FormattersContainer's name member

Raphael Isemann via All-commits all-commits at lists.llvm.org
Tue Jul 21 04:55:11 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 5c15426d7c62aa8c38547144f28c5a1c6e50549a
      https://github.com/llvm/llvm-project/commit/5c15426d7c62aa8c38547144f28c5a1c6e50549a
  Author: Raphael Isemann <teemperor at gmail.com>
  Date:   2020-07-21 (Tue, 21 Jul 2020)

  Changed paths:
    M lldb/include/lldb/DataFormatters/FormattersContainer.h
    M lldb/include/lldb/DataFormatters/TypeCategory.h
    M lldb/source/DataFormatters/TypeCategory.cpp

  Log Message:
  -----------
  [lldb] Remove FormattersContainer's name member

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.

Reviewers: mib, davide

Reviewed By: mib

Subscribers: JDevlieghere

Differential Revision: https://reviews.llvm.org/D84154




More information about the All-commits mailing list