Re: [commits] Horde branch master updated. 789ee9e7146f043974ff9e676a4b0edd737ba052

Quoting Michael M Slusarz <[email protected]>:

> commit 8e28721b86544127b272855dad9855bf453b51c0
> Author: Michael M Slusarz <[email protected]>
> Date:   Tue Jan 6 22:31:54 2015 -0700
>
>     [mms] Horde_Image_Exif_Bundled now supports reading data from a  
> PHP stream, rather than a file.
>
>  framework/Image/lib/Horde/Image/Exif/Bundled.php |   55  
> +++++++++++++++++----
>  framework/Image/package.xml                      |   12 ++--
>  2 files changed, 50 insertions(+), 17 deletions(-)
>
> http://github.com/horde/horde/commit/8e28721b86544127b272855dad9855bf453b51c0

Would be really great if we could get rid of the dual handle accesses  
in here (there's no reason this can't be accomplished with a single  
stream).  I just wanted to get this to work, so we are copying the  
data stream to a separate data stream to work with the current code.

For the record: Not using the other exif drivers because this needs to  
be done in-memory.  Saving to temp file to use exif binary or base64  
encoding to a data URL in order to use PHP's built-in exif handling is  
way too expensive.

michael

___________________________________
Michael Slusarz [[email protected]]

-- 
dev mailing list
Frequently Asked Questions: http://wiki.horde.org/FAQ
To unsubscribe, mail: [email protected]