Skip to content
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.

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

PyMem_Resize docs don't specify that it modifies an argument #44610

Closed
agthorr mannequin opened this issue Feb 24, 2007 · 2 comments
Closed

PyMem_Resize docs don't specify that it modifies an argument #44610

agthorr mannequin opened this issue Feb 24, 2007 · 2 comments
Labels
docs Documentation in the Doc dir

Comments

@agthorr
Copy link
Mannequin

agthorr mannequin commented Feb 24, 2007

BPO 1668036
Nosy @birkenfeld

Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.

Show more details

GitHub fields:

assignee = None
closed_at = <Date 2007-03-02.20:30:58.000>
created_at = <Date 2007-02-24.21:09:20.000>
labels = ['docs']
title = "PyMem_Resize docs don't specify that it modifies an argument"
updated_at = <Date 2007-03-02.20:30:58.000>
user = 'https://bugs.python.org/agthorr'

bugs.python.org fields:

activity = <Date 2007-03-02.20:30:58.000>
actor = 'georg.brandl'
assignee = 'none'
closed = True
closed_date = None
closer = None
components = ['Documentation']
creation = <Date 2007-02-24.21:09:20.000>
creator = 'agthorr'
dependencies = []
files = []
hgrepos = []
issue_num = 1668036
keywords = []
message_count = 2.0
messages = ['31331', '31332']
nosy_count = 2.0
nosy_names = ['georg.brandl', 'agthorr']
pr_nums = []
priority = 'normal'
resolution = 'fixed'
stage = None
status = 'closed'
superseder = None
type = None
url = 'https://bugs.python.org/issue1668036'
versions = []

@agthorr
Copy link
Mannequin Author

agthorr mannequin commented Feb 24, 2007

PyMem_Resize is defined as follows:

#define PyMem_Resize(p, type, n) \
	( (p) = (type *) PyMem_Realloc((p), (n) * sizeof(type)) )

The docs for PyMem_Resize don't specify that it modifies p. I suggest adding the following sentence:

"On return, p will point to the new memory area, or NULL in the event of failure."

@agthorr agthorr mannequin closed this as completed Feb 24, 2007
@agthorr agthorr mannequin closed this as completed Feb 24, 2007
@agthorr agthorr mannequin added docs Documentation in the Doc dir labels Feb 24, 2007
@birkenfeld
Copy link
Member

Fixed in rev. 54088, 54089 (2.5).

@ezio-melotti ezio-melotti transferred this issue from another repository Apr 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Documentation in the Doc dir
Projects
None yet
Development

No branches or pull requests

1 participant