Would not keeping a strong reference to a child be enough? Look at the list.remove() code.

Oh yes. I'll try using this one as a basis.

Would not keeping a strong reference to a child be enough? Look at the list.remove() code.

Ah I know why I couldn't make it work! it's because the children is not a list itself, but just a huge memory area that was dynamically allocated usingPyMem_Malloc. What do you suggest I do?

The list storage is also just a huge memory area that was dynamically allocated usingPyMem_Malloc. I do not see difference.

At worst, you can get different exceptions (or no exception) in C and Python implementations, but this is fine.

The list storage is also just a huge memory area that was dynamically allocated using PyMem_Malloc. I do not see difference.

The difference is that the list object usesPy_SIZE but it's not possible to use this on the current implementation of element trees (or is there?)

Py_SIZE orself->extra->length -- there is not much difference between them. Both just read a word from memory. You may need to check thatself->extra was not set to NULL during iteration, this is the only difference.

Py_SIZE or self->extra->length -- there is not much difference between them

I'm pretty sure I first tried using INCREF/DECREF as usual and encountered an issue with that but I can't remember how. I'll try reproducing this tomorrow.

@vstinner@serhiy-storchaka friendly ping in case you forgot about this one

As I wrote, I don't think that 200 lines of tests are needed. I would prefer way less tests.

I can remove the special test case which makes the test explode as it's only the Python implementation if you want. It would reduce the test by half at least (most of the line length is taken by comments that are needed to explain the discrepency between the Python and the C implementation).

@vstinner I've removed the pedantic check. However, we still need to have that much of tests becausedel root[:] androot.clear() behave differently although they should do the same. The reproducer I had did not make crashdel root[:], but it made crashroot.clear().

Thanks@picnixz for the PR 🌮🎉.. I'm working now to backport this PR to: 3.12, 3.13.
🐍🍒⛏🤖

GH-131929 is a backport of this pull request to the3.13 branch.

GH-131930 is a backport of this pull request to the3.12 branch.