[Patches] [ python-Patches-577031 ] Remove PyArg_Parse() and METH_OLDARGS
noreply@sourceforge.net
noreply@sourceforge.net
Sun, 28 Jul 2002 04:46:22 -0700
Patches item #577031, was opened at 2002-07-03 17:57
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=577031&group_id=5470
Category: Core (C code)
Group: Python 2.3
Status: Open
Resolution: None
Priority: 5
Submitted By: Neal Norwitz (nnorwitz)
Assigned to: Nobody/Anonymous (nobody)
Summary: Remove PyArg_Parse() and METH_OLDARGS
Initial Comment:
This patch removes more PyArg_Parse() and METH_OLDARGS
which are deprecated.
I've tested in select and string, but want to make sure
there's nothing else I'm missing.
I also have a huge change to glmodule, but I can't test
that. The diff is attached.
Let me know if I should check in glmodule or leave it
alone.
----------------------------------------------------------------------
>Comment By: Martin v. Löwis (loewis)
Date: 2002-07-28 13:46
Message:
Logged In: YES
user_id=21627
The other patches look all fine, please apply them. For
fmmodule, I'd recommend to convert those functions to
VARARGS/ParseTuple.
----------------------------------------------------------------------
Comment By: Neal Norwitz (nnorwitz)
Date: 2002-07-27 03:04
Message:
Logged In: YES
user_id=33168
All the "s" / PyString_Check() changes are in fmmodule. I
suggest to not patch fmmodule now. Are all the other
changes ok? Should I bother fixing glmodule at all?
----------------------------------------------------------------------
Comment By: Martin v. Löwis (loewis)
Date: 2002-07-05 07:45
Message:
Logged In: YES
user_id=21627
The changes look good, except for the ones that change
parsing of "s" to PyString_Check: that means to lose support
for Unicode.
For some of these methods, that may be acceptable, but that
would need documentation.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=577031&group_id=5470