[issue3042] Add PEP 8 compliant aliases to threading module

2008-06-11 Thread Benjamin Peterson
Benjamin Peterson <[EMAIL PROTECTED]> added the comment: Applied in r64125, and r64144. -- resolution: -> fixed status: open -> closed ___ Python tracker <[EMAIL PROTECTED]> ___ _

[issue3042] Add PEP 8 compliant aliases to threading module

2008-06-10 Thread Benjamin Peterson
Benjamin Peterson <[EMAIL PROTECTED]> added the comment: Ok. Pending your acceptance, I'll apply after docs+tests. Added file: http://bugs.python.org/file10576/new_threading_api3.patch ___ Python tracker <[EMAIL PROTECTED]>

[issue3042] Add PEP 8 compliant aliases to threading module

2008-06-08 Thread Nick Coghlan
Nick Coghlan <[EMAIL PROTECTED]> added the comment: I'd still be inclined to put a @wraps(meth) decorator on the definition of the wrapper function (and then override to the supplied name afterwards) - remember that functools.wraps is a decorator factory rather than a decorator itself. That wou

[issue3042] Add PEP 8 compliant aliases to threading module

2008-06-05 Thread Adam Olsen
Changes by Adam Olsen <[EMAIL PROTECTED]>: -- nosy: +Rhamphoryncus ___ Python tracker <[EMAIL PROTECTED]> ___ ___ Python-bugs-list maili

[issue3042] Add PEP 8 compliant aliases to threading module

2008-06-05 Thread Benjamin Peterson
Benjamin Peterson <[EMAIL PROTECTED]> added the comment: On Thu, Jun 5, 2008 at 8:56 PM, Nick Coghlan <[EMAIL PROTECTED]> wrote: > > Nick Coghlan <[EMAIL PROTECTED]> added the comment: > > The code changes in the patch look pretty good to me (although you > probably want to actually use @wraps in

[issue3042] Add PEP 8 compliant aliases to threading module

2008-06-05 Thread Nick Coghlan
Nick Coghlan <[EMAIL PROTECTED]> added the comment: The code changes in the patch look pretty good to me (although you probably want to actually use @wraps inside _old_api!) There are obviously also documentation and test suite changes that will be needed. __

[issue3042] Add PEP 8 compliant aliases to threading module

2008-06-05 Thread Benjamin Peterson
Benjamin Peterson <[EMAIL PROTECTED]> added the comment: Attaching a patch which fixes everything. Added file: http://bugs.python.org/file10523/new_threading_api2.patch ___ Python tracker <[EMAIL PROTECTED]>

[issue3042] Add PEP 8 compliant aliases to threading module

2008-06-05 Thread Benjamin Peterson
Benjamin Peterson <[EMAIL PROTECTED]> added the comment: Is this what you were looking for? (in the patch) What I did could also be done with metaclasses, but I thought that would be overkill. Warning: this patch causes test threading to crash when run with the -3 flag. -- keywords: +pat

[issue3042] Add PEP 8 compliant aliases to threading module

2008-06-05 Thread Nick Coghlan
New submission from Nick Coghlan <[EMAIL PROTECTED]>: PEP 371 highlighted the non-PEP 8 compliant nature of the threading API. Since part of that PEP involves updating the multiprocessing API to be PEP 8 compliant before addition the standard library, the threading API should also be updated to b