Is there an existing issue for this?

• I have searched the existing issues

Problem description

Unlike to PyTuple_GetItem or PyList_GetItem the function PySequence_GetItem returns a new reference instead of a borrowed reference. This means when using the function PySequence_GetItem then for the returned object Py_DECREF or Py_XDECREF must be called to decrease the reference counter.

Because this isn't done anywhere in the FreeCAD code where this function is used it causes memory leaks. It's recommended to use the class Py::Sequence instead.

See:
https://docs.python.org/3/c-api/tuple.html#c.PyTuple_GetItem
https://docs.python.org/3/c-api/list.html#c.PyList_GetItem
https://docs.python.org/3/c-api/sequence.html#c.PySequence_GetItem

Full version info

Development version

Subproject(s) affected?

None

Anything else?

No response

Code of Conduct

• I agree to follow this project's Code of Conduct