René's URL Explorer Experiment

Title: Improve CPython compatibility related with PyBoundMethod by moreal · Pull Request #7476 · RustPython/RustPython · GitHub

Open Graph Title: Improve CPython compatibility related with PyBoundMethod by moreal · Pull Request #7476 · RustPython/RustPython

X Title: Improve CPython compatibility related with PyBoundMethod by moreal · Pull Request #7476 · RustPython/RustPython

Description: This pull request adjusts PyBoundMethod for compatibility with CPython. Behavior changes Correct PyBoundMethod initialization validation Implement missing __get__ Raise error when __name__ when calling PyBoundMethod __reduce__ class A: def f(): ... class B: def __call__(self): ... print(type(A().f)(B(), 1).__reduce__()) CPython (3.14.3) >>> class A: ... def f(): ... ... >>> A().f > >>> type(A().f) >>> type(A().f)(None) Traceback (most recent call last): File "", line 1, in type(A().f)(None) ~~~~~~~~~~~^^^^^^ TypeError: method expected 2 arguments, got 1 >>> type(A().f)(None, None) Traceback (most recent call last): File "", line 1, in type(A().f)(None, None) ~~~~~~~~~~~^^^^^^^^^^^^ TypeError: first argument must be callable >>> type(A().f)(lambda x: x, None) Traceback (most recent call last): File "", line 1, in type(A().f)(lambda x: x, None) ~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^ TypeError: instance must not be None >>> type(A().f)(lambda x: x, 1) of 1> >>> type(A().f).__get__ Traceback (most recent call last): File "/path/to/RustPython/hello.py", line 10, in print(type(A().f)(B(), 1).__reduce__()) ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^ AttributeError: 'B' object has no attribute '__name__'. Did you mean: '__ne__'? RustPython (before this pull request) >>>>> class A: ..... def f(): ... ..... >>>>> A().f > >>>>> type(A().f) >>>>> type(A().f)(None, None) >>>>> type(A().f)(lambda x: x, None) of None> >>>>> type(A().f)(lambda x: x, 1) of 1> >>>>> type(A().f).__get__ Traceback (most recent call last): File "", line 1, in AttributeError: type object 'method' has no attribute '__get__'. Did you mean: '__ge__'? (, (1, None)) RustPython (after this pull request) >>>>> class A: ..... def f(): ... ..... >>>>> A().f > >>>>> type(A().f) >>>>> type(A().f)(None) Traceback (most recent call last): File "", line 1, in TypeError: expected at least 2 arguments, got 1 >>>>> type(A().f)(None, None) Traceback (most recent call last): File "", line 1, in TypeError: first argument must be callable >>>>> type(A().f)(lambda x: x, None) Traceback (most recent call last): File "", line 1, in TypeError: instance must not be None >>>>> type(A().f)(lambda x: x, 1) of 1> >>>>> type(A().f).__get__ Traceback (most recent call last): File "hello.py", line 9, in print(type(A().f)(B(), 1).__reduce__()) ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^ AttributeError: 'B' object has no attribute '__name__'. Did you mean: '__ne__'? Summary by CodeRabbit Bug Fixes Enhanced validation when creating bound methods; passing a non-callable function or None as the object parameter now correctly raises a TypeError. Improved error handling in bound method serialization; errors during attribute access are now properly propagated instead of being silently ignored.

Open Graph Description: This pull request adjusts PyBoundMethod for compatibility with CPython. Behavior changes Correct PyBoundMethod initialization validation Implement missing __get__ Raise error when __name__ when ca...

X Description: This pull request adjusts PyBoundMethod for compatibility with CPython. Behavior changes Correct PyBoundMethod initialization validation Implement missing __get__ Raise error when __name__ when ca...

Opengraph URL: https://github.com/RustPython/RustPython/pull/7476

X: @github

direct link

Domain: github.com

Links:

Viewport: width=device-width