diff options
author | Lawrence D'Anna <lawrence_danna@apple.com> | 2019-10-17 22:22:09 +0000 |
---|---|---|
committer | Lawrence D'Anna <lawrence_danna@apple.com> | 2019-10-17 22:22:09 +0000 |
commit | 03819d1c80ad5d7b34f8fe0d375fcd2ef6aa5e7f (patch) | |
tree | 7d1f66147d89ee00239ace90f0b8280720979c7e /lldb/source/Plugins/ScriptInterpreter/Python/PythonDataObjects.cpp | |
parent | c86a6acaee55c98072ff06d372d049cb4a671fb5 (diff) | |
download | bcm5719-llvm-03819d1c80ad5d7b34f8fe0d375fcd2ef6aa5e7f.tar.gz bcm5719-llvm-03819d1c80ad5d7b34f8fe0d375fcd2ef6aa5e7f.zip |
eliminate one form of PythonObject::Reset()
Summary:
I'd like to eliminate all forms of Reset() and all public constructors
on these objects, so the only way to make them is with Take<> and Retain<>
and the only way to copy or move them is with actual c++ copy, move, or
assignment.
This is a simple place to start.
Reviewers: JDevlieghere, clayborg, labath, jingham
Reviewed By: labath
Subscribers: lldb-commits
Tags: #lldb
Differential Revision: https://reviews.llvm.org/D69080
llvm-svn: 375182
Diffstat (limited to 'lldb/source/Plugins/ScriptInterpreter/Python/PythonDataObjects.cpp')
-rw-r--r-- | lldb/source/Plugins/ScriptInterpreter/Python/PythonDataObjects.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lldb/source/Plugins/ScriptInterpreter/Python/PythonDataObjects.cpp b/lldb/source/Plugins/ScriptInterpreter/Python/PythonDataObjects.cpp index 3ac404b532d..1fb9355b9ee 100644 --- a/lldb/source/Plugins/ScriptInterpreter/Python/PythonDataObjects.cpp +++ b/lldb/source/Plugins/ScriptInterpreter/Python/PythonDataObjects.cpp @@ -410,7 +410,7 @@ void PythonString::SetString(llvm::StringRef string) { llvm::consumeError(s.takeError()); Reset(); } else { - PythonObject::Reset(std::move(s.get())); + *this = std::move(s.get()); } } |