[issue20182] Derby #13: Convert 50 sites to Argument Clinic across 5 files

Tal Einat report at bugs.python.org
Sat May 16 11:22:31 CEST 2015


Tal Einat added the comment:

Here's a patch just for Modules/sysmodule.c (and related files), based on Georg's patch, updated to apply against current default, including fixes thanks to Serhiy's review.

Despite Serhiy's suggestions, I've not used parameter groups, due to Larry's statement of his dislike of this (ab)use of the feature.

I've left an alias of the form "args as args_value:" since the latest clinic.py doesn't automatically alias "args" (causing a compile error).

At Serhiy's suggestion, I've also converted sys.callstats. I couldn't find any tests for it, so someone should definitely review this conversion!

All tests pass on my OSX 10.10.

----------
Added file: http://bugs.python.org/file39387/issue20182.sysmodule.patch

_______________________________________
Python tracker <report at bugs.python.org>
<http://bugs.python.org/issue20182>
_______________________________________


More information about the Python-bugs-list mailing list