What does this implement/fix? Explain your changes.

Adds a tp_dealloc function to ExceptionClassObject, so we clean the BaseException members the same way as other the derived exceptions in the cpython code does.

Does this close any currently open issues?

Fixes issue#1371.

Any other comments?

Fixes a bug where C# Exceptions set as the Inner of another C# Exception, then converted into a Python exception object (and thus having__cause__ set to a valid object) would leak theInner/__cause__ objects.

Simplified diagram :

AggregateException.Inner        -> ArgumentException(...)      ^                                    ^      |                                    |PyAggregateException.__cause__  -> PyArgumentException

In this case, where the AggregateException would be thrown from C# and caught in Python, the AggregateException and PyAggregateException would be freed, but not the ArgumentException set as the cause.

Checklist

Check all those that are applicable and complete.

• Make sure to include one or more tests for your change
• If an enhancement PR, please create docs and at best an example
• Add yourself toAUTHORS
• Updated theCHANGELOG