Stefan Krah added the comment:

The approaches look good, but for clarity I want to replace all method calls 
that should never be overridden by the plain C functions of their corresponding 
static types.

I have no opinion about the Python version. The diff also "fixes" #26975 for 
the C version, so perhaps the Python version should be in sync. It is an 
academic question, since no one will write a class that triggers it.

Preemptively, I'm aware that long_bit_length() is defined with a single 
argument, then cast to a CFunction, then called with two arguments.

ceval.c does the same. -- We had a discussion about that with MvL a while ago, 
he preferred to define all CFunctions with two arguments. I'd also prefer that, 
but that is another issue.

----------
Added file: http://bugs.python.org/file43756/issue26974.diff

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue26974>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to