diff options
author | Jonas Devlieghere <jonas@devlieghere.com> | 2019-02-13 06:25:41 +0000 |
---|---|---|
committer | Jonas Devlieghere <jonas@devlieghere.com> | 2019-02-13 06:25:41 +0000 |
commit | d5b440369dbb0d41e6ecd47d6ac7410201e27f17 (patch) | |
tree | 4dc2e3c44bcd3e14143715fa86584862b2290f9f /lldb/source/Symbol/SymbolFile.cpp | |
parent | 5cf777e41387c84518a9ff2ec1222058daf54e58 (diff) | |
download | bcm5719-llvm-d5b440369dbb0d41e6ecd47d6ac7410201e27f17.tar.gz bcm5719-llvm-d5b440369dbb0d41e6ecd47d6ac7410201e27f17.zip |
Replace 'ap' with 'up' suffix in variable names. (NFC)
The `ap` suffix is a remnant of lldb's former use of auto pointers,
before they got deprecated. Although all their uses were replaced by
unique pointers, some variables still carried the suffix.
In r353795 I removed another auto_ptr remnant, namely redundant calls to
::get for unique_pointers. Jim justly noted that this is a good
opportunity to clean up the variable names as well.
I went over all the changes to ensure my find-and-replace didn't have
any undesired side-effects. I hope I didn't miss any, but if you end up
at this commit doing a git blame on a weirdly named variable, please
know that the change was unintentional.
llvm-svn: 353912
Diffstat (limited to 'lldb/source/Symbol/SymbolFile.cpp')
-rw-r--r-- | lldb/source/Symbol/SymbolFile.cpp | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/lldb/source/Symbol/SymbolFile.cpp b/lldb/source/Symbol/SymbolFile.cpp index 4576bcfd10c..85242aeacb1 100644 --- a/lldb/source/Symbol/SymbolFile.cpp +++ b/lldb/source/Symbol/SymbolFile.cpp @@ -31,7 +31,7 @@ std::recursive_mutex &SymbolFile::GetModuleMutex() const { } SymbolFile *SymbolFile::FindPlugin(ObjectFile *obj_file) { - std::unique_ptr<SymbolFile> best_symfile_ap; + std::unique_ptr<SymbolFile> best_symfile_up; if (obj_file != nullptr) { // We need to test the abilities of this section list. So create what it @@ -57,13 +57,13 @@ SymbolFile *SymbolFile::FindPlugin(ObjectFile *obj_file) { (create_callback = PluginManager::GetSymbolFileCreateCallbackAtIndex( idx)) != nullptr; ++idx) { - std::unique_ptr<SymbolFile> curr_symfile_ap(create_callback(obj_file)); + std::unique_ptr<SymbolFile> curr_symfile_up(create_callback(obj_file)); - if (curr_symfile_ap) { - const uint32_t sym_file_abilities = curr_symfile_ap->GetAbilities(); + if (curr_symfile_up) { + const uint32_t sym_file_abilities = curr_symfile_up->GetAbilities(); if (sym_file_abilities > best_symfile_abilities) { best_symfile_abilities = sym_file_abilities; - best_symfile_ap.reset(curr_symfile_ap.release()); + best_symfile_up.reset(curr_symfile_up.release()); // If any symbol file parser has all of the abilities, then we should // just stop looking. if ((kAllAbilities & sym_file_abilities) == kAllAbilities) @@ -71,13 +71,13 @@ SymbolFile *SymbolFile::FindPlugin(ObjectFile *obj_file) { } } } - if (best_symfile_ap) { + if (best_symfile_up) { // Let the winning symbol file parser initialize itself more completely // now that it has been chosen - best_symfile_ap->InitializeObject(); + best_symfile_up->InitializeObject(); } } - return best_symfile_ap.release(); + return best_symfile_up.release(); } TypeList *SymbolFile::GetTypeList() { |