From: Schollnick, B. <Ben...@xe...> - 2005-11-08 19:18:18
|
> > What about having this patch added to base Python? >=20 > Yes, I will probably do this (Hm, I'm still listed as the=20 > maintainer of modulefinder, so I have to do it anyway ;-). Ah.. That makes it much safer than... > But, it should probably go into some py2exe release as well=20 > because then it is available immediately, and even more=20 > tested when it becomes the 'master copy' in Python svn. Certainly and far easier to rectify if it fails... > So the questions: does it work for py2exe users? Is the=20 > speed gain worth the effort (optimizing for speed most of the=20 > time sacrifies readability - also in this case)? I have not yet examined the code change, but I have tried it. 1) It does seem to decrease "build" times... 2) It seems to speed up PY2EXE built applications roughly by 20-35% depending on system speed, etc... It's well worth it for me. I reduced the Auditor application loading time by roughly 26 seconds... That uses wxPython,=20 PythonCard, and a variety of other technologies, so it's no light weight... (And is roughly 5-6 Meg compressed in a ZIP) My suggestion to help, would be to increase the amount of comments to clarify the areas where you have optimized for speed... - Benjamin |