Make App.ImageFile use stream iterator

Bug #143902 reported by Sidnei da Silva
2
Affects Status Importance Assigned to Milestone
Zope 2
Fix Released
Medium
Hanno Schlichting

Bug Description

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.

Tags: bug zope
Revision history for this message
Sidnei da Silva (sidnei) wrote :
Revision history for this message
Sidnei da Silva (sidnei) wrote :

Ops, forgot a print in there. :)

Revision history for this message
Hanno Schlichting (hannosch) wrote :

I applied a slightly modified version of the patch to Zope trunk. My version doesn't load the file at all, unless the mimetype cannot be guessed from the filename alone.

Changed in zope2:
assignee: nobody → schlichting
status: New → Fix Committed
Changed in zope2:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.