GeezerVison is showing github.com — bpo-41706: Fix dunder operator invocation docs by wchargin · Pull Request #22084 · python/cpython
Conversation
wchargin
commented
Sep 3, 2020
•
edited by bedevere-bot
Loading
Uh oh!
There was an error while loading. Please reload this page.
The expression x + y is evaluated using type(x).__add__, which may
differ from x.__add__ if the __add__ attribute has been directly
assigned on the instance.
Demonstration:
class C:
def __add__(self, other):
return "from class"
c = C()
print(c + c) # prints "from class"
c.__add__ = lambda other: "from instance"
print(c.__add__(c)) # prints "from instance"
print(type(c).__add__(c, c)) # prints "from class"
print(c + c) # prints "from class"!
wchargin-branch: dunder-op-invocation-docs
https://bugs.python.org/issue41706
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
The expression `x + y` is evaluated using `type(x).__add__`, which may
differ from `x.__add__` if the `__add__` attribute has been directly
assigned on the instance.
Demonstration:
```python
class C:
def __add__(self, other):
return "from class"
c = C()
print(c + c) # prints "from class"
c.__add__ = lambda other: "from instance"
print(c.__add__(c)) # prints "from instance"
print(type(c).__add__(c, c)) # prints "from class"
print(c + c) # prints "from class"!
```
wchargin-branch: dunder-op-invocation-docs
wchargin-source: 543786396ecba124a332357c771cfd6b20493abe
|
Verified that doc updates render correctly in the build output from the
GitHub Action.
|
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
|
@brettcannon Do you have an opinion on this tweak to the documentation since you wrote the desugarring blog posts?
|
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Assignees
No one assigned
Labels
docs
Documentation in the Doc dir
Milestone
No milestone
Development
Successfully merging this pull request may close these issues.
Uh oh!
There was an error while loading. Please reload this page.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.