Skip to content

bpo-41287: Handle doc argument of property.__init__ in subclasses #23205

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
JelleZijlstra merged 19 commits into from
May 29, 2022
Merged

bpo-41287: Handle doc argument of property.__init__ in subclasses #23205

Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
b19523a
Handle `doc` argument of `property.__init__` in subclasses
sizmailov Nov 9, 2020
0745f70
Add NEWS entry
sizmailov Apr 3, 2022
ebec602
Update Misc/NEWS.d/next/Library/2022-04-03-11-25-02.bpo-41287.8CTdwf.rst
sizmailov Apr 5, 2022
f09e6ac
Move test to test_property.py
sizmailov Apr 5, 2022
dcab455
Rename local get_doc -> prop_doc
sizmailov Apr 5, 2022
de2c3c0
Add prop_doc != NULL check
sizmailov Apr 5, 2022
bd4cdca
Add more tests
sizmailov Apr 5, 2022
1a1247e
Fix test
sizmailov Apr 5, 2022
009a9f3
Fix introduced refleak
sizmailov Apr 7, 2022
d2aa925
Merge branch 'main' into fix-issue41287
JelleZijlstra May 2, 2022
8730abc
Test vanila property too
sizmailov May 22, 2022
0302ffb
Fix __doc__ assignment in property subclass
sizmailov May 22, 2022
e1ec1dc
Merge remote-tracking branch 'upstream/main' into fix-issue41287
sizmailov May 22, 2022
fb8c2d4
Fix Py_None ref leak
sizmailov May 22, 2022
f8fb254
Add a comment
sizmailov May 22, 2022
fc16ab9
Remove unnecessary check
sizmailov May 22, 2022
4943868
Add test for property copy doc
sizmailov May 26, 2022
36fc1b0
Assign getter_doc only for non-null prop_doc
sizmailov May 26, 2022
14bdd86
Apply suggestions from code review
sizmailov May 28, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Remove unnecessary check
  • Loading branch information
@sizmailov
sizmailov committed May 22, 2022
commit fc16ab974fb5d0e564478ed845ac18843bc70758
4 changes: 1 addition & 3 deletions Objects/descrobject.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1812,9 +1812,7 @@ property_init_impl(propertyobject *self, PyObject *fget, PyObject *fset,
a non-None object with incremented ref counter */

if (Py_IS_TYPE(self, &PyProperty_Type)) {
if (prop_doc != NULL && prop_doc != Py_None) {
Py_XSETREF(self->prop_doc, prop_doc);
}
Py_XSETREF(self->prop_doc, prop_doc);
} else {
/* If this is a property subclass, put __doc__
in dict of the subclass instance instead,
Expand Down