[issue3852] kqueue.control requires 2 params while docs say max_events (the second) defaults to 0

[Georg Brandl]

Georg Brandl <[EMAIL PROTECTED]> added the comment:

Committed patch in r66523. At least for now, code and docs are
consistent again.

--
resolution:  -> fixed
status: open -> closed

___
Python tracker <[EMAIL PROTECTED]>

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

[issue3852] kqueue.control requires 2 params while docs say max_events (the second) defaults to 0

[Gabriel Genellina]

Gabriel Genellina <[EMAIL PROTECTED]> added the comment:

Attached a documentation patch, including the kqueue.control function 
docstring.

But I wonder if the code was incorrect instead - both the 
documentation and the function docstring specified a default value for 
max_events=0, and the corresponding variable was initialized to 0.
Perhaps the author meant to use PyArg_ParseTuple(args, "O|
iO:control",...) instead of the current "Oi|O:control"

--
keywords: +patch
nosy: +gagenellina
Added file: http://bugs.python.org/file11536/select.diff

___
Python tracker <[EMAIL PROTECTED]>

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

[issue3852] kqueue.control requires 2 params while docs say max_events (the second) defaults to 0

[Benjamin Peterson]

Changes by Benjamin Peterson <[EMAIL PROTECTED]>:


--
assignee: georg.brandl -> christian.heimes
nosy: +christian.heimes

___
Python tracker <[EMAIL PROTECTED]>

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

[issue3852] kqueue.control requires 2 params while docs say max_events (the second) defaults to 0

[Maries Ionel Cristian]

New submission from Maries Ionel Cristian <[EMAIL PROTECTED]>:

http://docs.python.org/dev/library/select.html#id1

Docs say: "select.control(changelist, max_events=0[, timeout=None])"
However, control requires 2 params ("TypeError: control() takes at least
2 arguments (1 given)").

Also, it should be "kqueue" not "select" (There are 2 more like this
"epoll.fromfd(fd)" in the kqueue section, "select.kqueue(ident,
filter=KQ_FILTER_READ, flags=KQ_ADD, fflags=0, data=0, udata=0)" instead
of "select.kevent( ... ")

--
assignee: georg.brandl
components: Documentation, Extension Modules
messages: 73144
nosy: georg.brandl, ionel.mc
severity: normal
status: open
title: kqueue.control requires 2 params while docs say max_events (the second) 
defaults to 0
type: behavior
versions: Python 2.6, Python 3.0

___
Python tracker <[EMAIL PROTECTED]>

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