[all-commits] [llvm/llvm-project] 03819d: eliminate one form of PythonObject::Reset()

llvm-git-migration via All-commits all-commits at lists.llvm.org
Thu Oct 17 15:21:05 PDT 2019


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 03819d1c80ad5d7b34f8fe0d375fcd2ef6aa5e7f
      https://github.com/llvm/llvm-project/commit/03819d1c80ad5d7b34f8fe0d375fcd2ef6aa5e7f
  Author: Lawrence D'Anna <lawrence_danna at apple.com>
  Date:   2019-10-17 (Thu, 17 Oct 2019)

  Changed paths:
    M lldb/source/Plugins/ScriptInterpreter/Python/PythonDataObjects.cpp
    M lldb/source/Plugins/ScriptInterpreter/Python/PythonDataObjects.h
    M lldb/unittests/ScriptInterpreter/Python/PythonDataObjectsTests.cpp

  Log Message:
  -----------
  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




More information about the All-commits mailing list