diff options
| author | Andrew Burgess <aburgess@redhat.com> | 2022-05-24 11:54:40 +0100 |
|---|---|---|
| committer | Andrew Burgess <aburgess@redhat.com> | 2022-06-15 09:44:54 +0100 |
| commit | 8a0b60471a75ce81b8ea067f6e87457b3ed0c7a3 (patch) | |
| tree | 348a51966c354f6894e7c34f2dc9f6329ff956b1 /gdb/python/python-internal.h | |
| parent | 5fb28d2607a8325559b44a5dc0c8760236c81218 (diff) | |
| download | gdb-8a0b60471a75ce81b8ea067f6e87457b3ed0c7a3.zip gdb-8a0b60471a75ce81b8ea067f6e87457b3ed0c7a3.tar.gz gdb-8a0b60471a75ce81b8ea067f6e87457b3ed0c7a3.tar.bz2 | |
gdb/python: convert gdbpy_err_fetch to use gdbpy_ref
Convert the gdbpy_err_fetch class to make use of gdbpy_ref, this
removes the need for manual reference count management, and allows the
destructor to be removed.
There should be no functional change after this commit.
I think this cleanup is worth doing on its own, however, in a later
commit I will want to copy instances of gdbpy_err_fetch, and switching
to using gdbpy_ref means that I can rely on the default copy
constructor, without having to add one that handles the reference
counts, so this is good preparation for that upcoming change.
Diffstat (limited to 'gdb/python/python-internal.h')
| -rw-r--r-- | gdb/python/python-internal.h | 23 |
1 files changed, 10 insertions, 13 deletions
diff --git a/gdb/python/python-internal.h b/gdb/python/python-internal.h index 217bc15..da2e791 100644 --- a/gdb/python/python-internal.h +++ b/gdb/python/python-internal.h @@ -549,14 +549,12 @@ public: gdbpy_err_fetch () { - PyErr_Fetch (&m_error_type, &m_error_value, &m_error_traceback); - } + PyObject *error_type, *error_value, *error_traceback; - ~gdbpy_err_fetch () - { - Py_XDECREF (m_error_type); - Py_XDECREF (m_error_value); - Py_XDECREF (m_error_traceback); + PyErr_Fetch (&error_type, &error_value, &error_traceback); + m_error_type.reset (error_type); + m_error_value.reset (error_value); + m_error_traceback.reset (error_traceback); } /* Call PyErr_Restore using the values stashed in this object. @@ -565,10 +563,9 @@ public: void restore () { - PyErr_Restore (m_error_type, m_error_value, m_error_traceback); - m_error_type = nullptr; - m_error_value = nullptr; - m_error_traceback = nullptr; + PyErr_Restore (m_error_type.release (), + m_error_value.release (), + m_error_traceback.release ()); } /* Return the string representation of the exception represented by @@ -587,12 +584,12 @@ public: bool type_matches (PyObject *type) const { - return PyErr_GivenExceptionMatches (m_error_type, type); + return PyErr_GivenExceptionMatches (m_error_type.get (), type); } private: - PyObject *m_error_type, *m_error_value, *m_error_traceback; + gdbpy_ref<> m_error_type, m_error_value, m_error_traceback; }; /* Called before entering the Python interpreter to install the |