GeezerVison

Thank you Victor for your review!

Feel free to leave all the asserts to a follow-up PR.

Thank you!

Question: should we backport this to 3.12 and 3.13? it's both a feature and a bugfix in some sense but I feel it's more a bug fix even though it could change the behaviour of some code (hopefully, not too much...). The start getter was buggy (it could return -1) but the end getter was not.

WDYT? By the way, I spotted some typos here and there. I'll correct them as part of a follow-up PR.

If I understand correctly, it could only return -1 with an empty string, which isn't something one would typically get encoding errors for. Also, at this level you get a wrong value rather than a crash.
I don't think it's worth backporting.

it could only return -1 with an empty string

Yup. (to be clear, it's the size that is being stored in start, not the function return value which returns -1 on error and 0 on success).

In this case, subsequent calls usually lead to crashes because the C API assumes non-relative offsets in general.

I don't think it's worth backporting.

Thanks, it's essentially to know whether the patches for codecs handlers will be backported or not (without the backport of this PR, the handlers would still be broken I think).