[Patches] [ python-Patches-768187 ] replace apply() with f(*args, **kwds)

SourceForge.net noreply@sourceforge.net
Wed, 09 Jul 2003 06:20:44 -0700


Patches item #768187, was opened at 2003-07-08 21:12
Message generated for change (Comment added) made by rhettinger
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: None
Priority: 5
Submitted By: Raymond Hettinger (rhettinger)
Assigned to: Kurt B. Kaiser (kbk)
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: 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