[issue34287] bufferedio.c uses unused argument of METH_NOARGS functions

[INADA Naoki]

Change by INADA Naoki :


--
resolution:  -> fixed
stage: patch review -> resolved
status: open -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue34287] bufferedio.c uses unused argument of METH_NOARGS functions

[INADA Naoki]

INADA Naoki  added the comment:


New changeset fc512e3e0663f7f325862fcd42aef765fd34a453 by INADA Naoki 
(jdemeyer) in branch 'master':
bpo-34287: Do not use second argument of METH_NOARGS functions (GH-8582)
https://github.com/python/cpython/commit/fc512e3e0663f7f325862fcd42aef765fd34a453


--
nosy: +inada.naoki

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue34287] bufferedio.c uses unused argument of METH_NOARGS functions

[Jeroen Demeyer]

Change by Jeroen Demeyer :


--
keywords: +patch
pull_requests: +8091
stage:  -> patch review

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue34287] bufferedio.c uses unused argument of METH_NOARGS functions

[Jeroen Demeyer]

New submission from Jeroen Demeyer :

A METH_NOARGS function has a second unused argument which is always NULL (this 
is guaranteed by the documentation).

However, some functions in Modules/_io/bufferedio.c actually that second NULL 
argument. This is technically not a bug, but it looks more clear to explicitly 
mark that second argument as unused.

--
components: Extension Modules
messages: 322736
nosy: jdemeyer, rhettinger, scoder
priority: normal
severity: normal
status: open
title: bufferedio.c uses unused argument of METH_NOARGS functions
versions: Python 3.8

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com