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/API/SBVariablesOptions.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/API/SBVariablesOptions.cpp')
-rw-r--r-- | lldb/source/API/SBVariablesOptions.cpp | 50 |
1 files changed, 25 insertions, 25 deletions
diff --git a/lldb/source/API/SBVariablesOptions.cpp b/lldb/source/API/SBVariablesOptions.cpp index 42eafaa8cd9..69c684d9962 100644 --- a/lldb/source/API/SBVariablesOptions.cpp +++ b/lldb/source/API/SBVariablesOptions.cpp @@ -80,98 +80,98 @@ private: }; SBVariablesOptions::SBVariablesOptions() - : m_opaque_ap(new VariablesOptionsImpl()) {} + : m_opaque_up(new VariablesOptionsImpl()) {} SBVariablesOptions::SBVariablesOptions(const SBVariablesOptions &options) - : m_opaque_ap(new VariablesOptionsImpl(options.ref())) {} + : m_opaque_up(new VariablesOptionsImpl(options.ref())) {} SBVariablesOptions &SBVariablesOptions:: operator=(const SBVariablesOptions &options) { - m_opaque_ap.reset(new VariablesOptionsImpl(options.ref())); + m_opaque_up.reset(new VariablesOptionsImpl(options.ref())); return *this; } SBVariablesOptions::~SBVariablesOptions() = default; -bool SBVariablesOptions::IsValid() const { return m_opaque_ap != nullptr; } +bool SBVariablesOptions::IsValid() const { return m_opaque_up != nullptr; } bool SBVariablesOptions::GetIncludeArguments() const { - return m_opaque_ap->GetIncludeArguments(); + return m_opaque_up->GetIncludeArguments(); } void SBVariablesOptions::SetIncludeArguments(bool arguments) { - m_opaque_ap->SetIncludeArguments(arguments); + m_opaque_up->SetIncludeArguments(arguments); } bool SBVariablesOptions::GetIncludeRecognizedArguments( const lldb::SBTarget &target) const { - return m_opaque_ap->GetIncludeRecognizedArguments(target.GetSP()); + return m_opaque_up->GetIncludeRecognizedArguments(target.GetSP()); } void SBVariablesOptions::SetIncludeRecognizedArguments(bool arguments) { - m_opaque_ap->SetIncludeRecognizedArguments(arguments); + m_opaque_up->SetIncludeRecognizedArguments(arguments); } bool SBVariablesOptions::GetIncludeLocals() const { - return m_opaque_ap->GetIncludeLocals(); + return m_opaque_up->GetIncludeLocals(); } void SBVariablesOptions::SetIncludeLocals(bool locals) { - m_opaque_ap->SetIncludeLocals(locals); + m_opaque_up->SetIncludeLocals(locals); } bool SBVariablesOptions::GetIncludeStatics() const { - return m_opaque_ap->GetIncludeStatics(); + return m_opaque_up->GetIncludeStatics(); } void SBVariablesOptions::SetIncludeStatics(bool statics) { - m_opaque_ap->SetIncludeStatics(statics); + m_opaque_up->SetIncludeStatics(statics); } bool SBVariablesOptions::GetInScopeOnly() const { - return m_opaque_ap->GetInScopeOnly(); + return m_opaque_up->GetInScopeOnly(); } void SBVariablesOptions::SetInScopeOnly(bool in_scope_only) { - m_opaque_ap->SetInScopeOnly(in_scope_only); + m_opaque_up->SetInScopeOnly(in_scope_only); } bool SBVariablesOptions::GetIncludeRuntimeSupportValues() const { - return m_opaque_ap->GetIncludeRuntimeSupportValues(); + return m_opaque_up->GetIncludeRuntimeSupportValues(); } void SBVariablesOptions::SetIncludeRuntimeSupportValues( bool runtime_support_values) { - m_opaque_ap->SetIncludeRuntimeSupportValues(runtime_support_values); + m_opaque_up->SetIncludeRuntimeSupportValues(runtime_support_values); } lldb::DynamicValueType SBVariablesOptions::GetUseDynamic() const { - return m_opaque_ap->GetUseDynamic(); + return m_opaque_up->GetUseDynamic(); } void SBVariablesOptions::SetUseDynamic(lldb::DynamicValueType dynamic) { - m_opaque_ap->SetUseDynamic(dynamic); + m_opaque_up->SetUseDynamic(dynamic); } VariablesOptionsImpl *SBVariablesOptions::operator->() { - return m_opaque_ap.operator->(); + return m_opaque_up.operator->(); } const VariablesOptionsImpl *SBVariablesOptions::operator->() const { - return m_opaque_ap.operator->(); + return m_opaque_up.operator->(); } -VariablesOptionsImpl *SBVariablesOptions::get() { return m_opaque_ap.get(); } +VariablesOptionsImpl *SBVariablesOptions::get() { return m_opaque_up.get(); } -VariablesOptionsImpl &SBVariablesOptions::ref() { return *m_opaque_ap; } +VariablesOptionsImpl &SBVariablesOptions::ref() { return *m_opaque_up; } const VariablesOptionsImpl &SBVariablesOptions::ref() const { - return *m_opaque_ap; + return *m_opaque_up; } SBVariablesOptions::SBVariablesOptions(VariablesOptionsImpl *lldb_object_ptr) - : m_opaque_ap(std::move(lldb_object_ptr)) {} + : m_opaque_up(std::move(lldb_object_ptr)) {} void SBVariablesOptions::SetOptions(VariablesOptionsImpl *lldb_object_ptr) { - m_opaque_ap.reset(std::move(lldb_object_ptr)); + m_opaque_up.reset(std::move(lldb_object_ptr)); } |