diff options
author | Kate Stone <katherine.stone@apple.com> | 2016-09-06 20:57:50 +0000 |
---|---|---|
committer | Kate Stone <katherine.stone@apple.com> | 2016-09-06 20:57:50 +0000 |
commit | b9c1b51e45b845debb76d8658edabca70ca56079 (patch) | |
tree | dfcb5a13ef2b014202340f47036da383eaee74aa /lldb/source/Plugins/SymbolFile/DWARF/DWARFDeclContext.cpp | |
parent | d5aa73376966339caad04013510626ec2e42c760 (diff) | |
download | llvm-b9c1b51e45b845debb76d8658edabca70ca56079.zip llvm-b9c1b51e45b845debb76d8658edabca70ca56079.tar.gz llvm-b9c1b51e45b845debb76d8658edabca70ca56079.tar.bz2 |
*** This commit represents a complete reformatting of the LLDB source code
*** to conform to clang-format’s LLVM style. This kind of mass change has
*** two obvious implications:
Firstly, merging this particular commit into a downstream fork may be a huge
effort. Alternatively, it may be worth merging all changes up to this commit,
performing the same reformatting operation locally, and then discarding the
merge for this particular commit. The commands used to accomplish this
reformatting were as follows (with current working directory as the root of
the repository):
find . \( -iname "*.c" -or -iname "*.cpp" -or -iname "*.h" -or -iname "*.mm" \) -exec clang-format -i {} +
find . -iname "*.py" -exec autopep8 --in-place --aggressive --aggressive {} + ;
The version of clang-format used was 3.9.0, and autopep8 was 1.2.4.
Secondly, “blame” style tools will generally point to this commit instead of
a meaningful prior commit. There are alternatives available that will attempt
to look through this change and find the appropriate prior commit. YMMV.
llvm-svn: 280751
Diffstat (limited to 'lldb/source/Plugins/SymbolFile/DWARF/DWARFDeclContext.cpp')
-rw-r--r-- | lldb/source/Plugins/SymbolFile/DWARF/DWARFDeclContext.cpp | 152 |
1 files changed, 68 insertions, 84 deletions
diff --git a/lldb/source/Plugins/SymbolFile/DWARF/DWARFDeclContext.cpp b/lldb/source/Plugins/SymbolFile/DWARF/DWARFDeclContext.cpp index 5512072..79f2f22 100644 --- a/lldb/source/Plugins/SymbolFile/DWARF/DWARFDeclContext.cpp +++ b/lldb/source/Plugins/SymbolFile/DWARF/DWARFDeclContext.cpp @@ -9,96 +9,80 @@ #include "DWARFDeclContext.h" -const char * -DWARFDeclContext::GetQualifiedName () const -{ - if (m_qualified_name.empty()) - { - // The declaration context array for a class named "foo" in namespace - // "a::b::c" will be something like: - // [0] DW_TAG_class_type "foo" - // [1] DW_TAG_namespace "c" - // [2] DW_TAG_namespace "b" - // [3] DW_TAG_namespace "a" - if (!m_entries.empty()) - { - if (m_entries.size() == 1) - { - if (m_entries[0].name) - { - m_qualified_name.append("::"); - m_qualified_name.append(m_entries[0].name); - } - } +const char *DWARFDeclContext::GetQualifiedName() const { + if (m_qualified_name.empty()) { + // The declaration context array for a class named "foo" in namespace + // "a::b::c" will be something like: + // [0] DW_TAG_class_type "foo" + // [1] DW_TAG_namespace "c" + // [2] DW_TAG_namespace "b" + // [3] DW_TAG_namespace "a" + if (!m_entries.empty()) { + if (m_entries.size() == 1) { + if (m_entries[0].name) { + m_qualified_name.append("::"); + m_qualified_name.append(m_entries[0].name); + } + } else { + collection::const_reverse_iterator pos; + collection::const_reverse_iterator begin = m_entries.rbegin(); + collection::const_reverse_iterator end = m_entries.rend(); + for (pos = begin; pos != end; ++pos) { + if (pos != begin) + m_qualified_name.append("::"); + if (pos->name == NULL) { + if (pos->tag == DW_TAG_namespace) + m_qualified_name.append("(anonymous namespace)"); + else if (pos->tag == DW_TAG_class_type) + m_qualified_name.append("(anonymous class)"); + else if (pos->tag == DW_TAG_structure_type) + m_qualified_name.append("(anonymous struct)"); + else if (pos->tag == DW_TAG_union_type) + m_qualified_name.append("(anonymous union)"); else - { - collection::const_reverse_iterator pos; - collection::const_reverse_iterator begin = m_entries.rbegin(); - collection::const_reverse_iterator end = m_entries.rend(); - for (pos = begin; pos != end; ++pos) - { - if (pos != begin) - m_qualified_name.append("::"); - if (pos->name == NULL) - { - if (pos->tag == DW_TAG_namespace) - m_qualified_name.append ("(anonymous namespace)"); - else if (pos->tag == DW_TAG_class_type) - m_qualified_name.append ("(anonymous class)"); - else if (pos->tag == DW_TAG_structure_type) - m_qualified_name.append ("(anonymous struct)"); - else if (pos->tag == DW_TAG_union_type) - m_qualified_name.append ("(anonymous union)"); - else - m_qualified_name.append ("(anonymous)"); - } - else - m_qualified_name.append(pos->name); - } - } + m_qualified_name.append("(anonymous)"); + } else + m_qualified_name.append(pos->name); } + } } - if (m_qualified_name.empty()) - return NULL; - return m_qualified_name.c_str(); + } + if (m_qualified_name.empty()) + return NULL; + return m_qualified_name.c_str(); } +bool DWARFDeclContext::operator==(const DWARFDeclContext &rhs) const { + if (m_entries.size() != rhs.m_entries.size()) + return false; -bool -DWARFDeclContext::operator==(const DWARFDeclContext& rhs) const -{ - if (m_entries.size() != rhs.m_entries.size()) - return false; - - collection::const_iterator pos; - collection::const_iterator begin = m_entries.begin(); - collection::const_iterator end = m_entries.end(); + collection::const_iterator pos; + collection::const_iterator begin = m_entries.begin(); + collection::const_iterator end = m_entries.end(); - collection::const_iterator rhs_pos; - collection::const_iterator rhs_begin = rhs.m_entries.begin(); - // The two entry arrays have the same size - - // First compare the tags before we do expensive name compares - for (pos = begin, rhs_pos = rhs_begin; pos != end; ++pos, ++rhs_pos) - { - if (pos->tag != rhs_pos->tag) - { - // Check for DW_TAG_structure_type and DW_TAG_class_type as they are often - // used interchangeably in GCC - if (pos->tag == DW_TAG_structure_type && rhs_pos->tag == DW_TAG_class_type) - continue; - if (pos->tag == DW_TAG_class_type && rhs_pos->tag == DW_TAG_structure_type) - continue; - return false; - } - } - // The tags all match, now compare the names - for (pos = begin, rhs_pos = rhs_begin; pos != end; ++pos, ++rhs_pos) - { - if (!pos->NameMatches (*rhs_pos)) - return false; + collection::const_iterator rhs_pos; + collection::const_iterator rhs_begin = rhs.m_entries.begin(); + // The two entry arrays have the same size + + // First compare the tags before we do expensive name compares + for (pos = begin, rhs_pos = rhs_begin; pos != end; ++pos, ++rhs_pos) { + if (pos->tag != rhs_pos->tag) { + // Check for DW_TAG_structure_type and DW_TAG_class_type as they are often + // used interchangeably in GCC + if (pos->tag == DW_TAG_structure_type && + rhs_pos->tag == DW_TAG_class_type) + continue; + if (pos->tag == DW_TAG_class_type && + rhs_pos->tag == DW_TAG_structure_type) + continue; + return false; } - // All tags and names match - return true; + } + // The tags all match, now compare the names + for (pos = begin, rhs_pos = rhs_begin; pos != end; ++pos, ++rhs_pos) { + if (!pos->NameMatches(*rhs_pos)) + return false; + } + // All tags and names match + return true; } - |