[ZCM] [ZC] 2309/ 2 Comment "Make App.ImageFile use stream iterator"

"Collector: Zope Bugs, Features, and Patches ..." <[email protected]> Wed, 11 Apr 2007 14:15:19 -0400
Newsgroups gmane.comp.web.zope.devel.collector-monitor
Message-ID <[email protected]>
Issue #2309 Update (Comment) "Make App.ImageFile use stream iterator"
 Status Pending, Zope/bug medium
To followup, visit:
  http://www.zope.org/Collectors/Zope/2309

==============================================================
= Comment - Entry #2 by dreamcatcher on Apr 11, 2007 2:15 pm

Ops, forgot a print in there. :)
________________________________________
= Request - Entry #1 by dreamcatcher on Apr 11, 2007 2:07 pm


Uploaded:  "imagefile-iterator.diff"
 - http://www.zope.org/Collectors/Zope/2309/imagefile-iterator.diff/view
Here's a patch to make App.ImageFile to do two things:

 - Do *not* read the full file on startup, 64k is probably enough (too much?)
 - Return a filestream_iterator.

Should make Zope startup a little bit faster, and serving ZMI icons too.
==============================================================