This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author serhiy.storchaka
Recipients Goplat, brett.cannon, catalin.iacob, eric.snow, serhiy.storchaka, ysj.ray
Date 2012-11-25.17:18:08
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1353863889.0.0.214163244795.issue8745@psf.upfronthosting.co.za>
In-reply-to
Content
> So this patch is a workaround for poor fseek performance in Microsoft's CRT, it doesn't cause performance issues on Linux but saves quite some milliseconds of startup time so I think it's worth the tradeoff.

I think some clarifying comments will be good in the code. In particular about the `dummy` variable.

> I'll also upload zipimport_speedup-v3.patch updated to apply to current default and with error handling for failing freads since the fseeks that the patch replaces have gained error handling on the default branch in the mean time.

Perhaps getc() requires error handling too.
History
Date User Action Args
2012-11-25 17:18:09serhiy.storchakasetrecipients: + serhiy.storchaka, brett.cannon, ysj.ray, Goplat, catalin.iacob, eric.snow
2012-11-25 17:18:09serhiy.storchakasetmessageid: <1353863889.0.0.214163244795.issue8745@psf.upfronthosting.co.za>
2012-11-25 17:18:08serhiy.storchakalinkissue8745 messages
2012-11-25 17:18:08serhiy.storchakacreate