[Patches] [ python-Patches-768187 ] replace apply() with f(*args, **kwds)
SourceForge.net
noreply@sourceforge.net
Wed, 09 Jul 2003 11:36:01 -0700
Patches item #768187, was opened at 2003-07-08 21:12
Message generated for change (Comment added) made by kbk
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=768187&group_id=5470
Category: IDLE
Group: Python 2.3
Status: Open
>Resolution: Accepted
Priority: 5
Submitted By: Raymond Hettinger (rhettinger)
>Assigned to: Raymond Hettinger (rhettinger)
Summary: replace apply() with f(*args, **kwds)
Initial Comment:
The warnings module causes apply() to run nearly two
orders of magnitude more slowly than f(*arg, **kwds).
With this patch IDLE appears to load and execute faster
(I say appears but I don't see an easy way of timing it).
----------------------------------------------------------------------
>Comment By: Kurt B. Kaiser (kbk)
Date: 2003-07-09 13:36
Message:
Logged In: YES
user_id=149084
This looks fine, please apply it.
----------------------------------------------------------------------
Comment By: Raymond Hettinger (rhettinger)
Date: 2003-07-09 08:20
Message:
Logged In: YES
user_id=80475
Thanks for the detailed review.
Here is the revised patch (only 1 line changed).
----------------------------------------------------------------------
Comment By: Walter Dörwald (doerwalter)
Date: 2003-07-09 03:49
Message:
Logged In: YES
user_id=89016
The first part of the patch to configDialog.py is wrong:
plane is a variable with the possible values "foreground"
and "background". The patch changes this to "plane".
Otherwise the patch looks OK.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=768187&group_id=5470