Further small simplification. With the GIL held, simply deleting the raw_ptr takes care of everything.

Author: rwgk

include/pybind11/pybind11.h

@@ -2629,7 +2629,6 @@ inline void
 error_already_set::m_fetched_error_deleter(detail::error_fetch_and_normalize *raw_ptr) {
     gil_scoped_acquire gil;
     error_scope scope;
-    raw_ptr->release_py_object_references();
     delete raw_ptr;
 }
 

include/pybind11/pytypes.h

@@ -541,12 +541,6 @@ struct error_fetch_and_normalize {
         return (PyErr_GivenExceptionMatches(m_type.ptr(), exc.ptr()) != 0);
     }
 
-    void release_py_object_references() {
-        m_type.release().dec_ref();
-        m_value.release().dec_ref();
-        m_trace.release().dec_ref();
-    }
-
     // Not protecting these for simplicity.
     object m_type, m_value, m_trace;