[Patches] [Patch #103252] Startup optimize: read *.pyc as string, not with getc()

noreply@sourceforge.net noreply@sourceforge.net
Tue, 16 Jan 2001 13:36:24 -0800


Patch #103252 has been updated. 

Project: python
Category: core (C code)
Status: Open
Submitted by: pj99
Assigned to : tim_one
Summary: Startup optimize: read *.pyc as string, not with getc()

Follow-Ups:

Date: 2001-Jan-16 13:36
By: pj99

Comment:
No problem, Tim (on the alledged abruptness).  In any case,
it was useful - in that it helped motivate /f into proposing
a fix, me into coding it, and you to consider applying
it.  All in all a useful result.  Most thanks, Tim.

-------------------------------------------------------

Date: 2001-Jan-15 20:26
By: tim_one

Comment:
Indeed, IDLE takes more than a full second to come up on my box, *much*
longer than Notepad.  Now that we have a clear goal, I assigned this patch
to me <wink>.  (OK, it's really that I was abrupt with Paul when this came
up on c.l.py, and left him hanging there -- I owe him on this one).
-------------------------------------------------------

Date: 2001-Jan-15 20:18
By: gvanrossum

Comment:
Oh, but I do care about the load time of Tkinter. :-)  Anything to keep
IDLE ahead of Komodo in the race. :-)
-------------------------------------------------------

Date: 2001-Jan-15 20:05
By: tim_one

Comment:
I like the idea, but gimmicks like this make porting to tiny platforms
harder when pushed "too far".

People concerned about startup time seem mostly to deal with apps that e.g.
fire up Python 1000s of times from shell scripts or cgi.  They're not going
to care about monster modules (like Tkinter and grail).  So I think Paul's
original 64K is a good compromise.

-------------------------------------------------------

Date: 2001-Jan-15 19:39
By: gvanrossum

Comment:
Not a bad idea.

I'd up the limit to 256K so Tkinter.pyc (comfortably) fits.
-------------------------------------------------------

-------------------------------------------------------
For more info, visit:

http://sourceforge.net/patch/?func=detailpatch&patch_id=103252&group_id=5470