[dev] [commits] Horde branch master updated. 789ee9e7146f043974ff9e676a4b0edd737ba052

Michael M Slusarz slusarz at horde.org
Wed Jan 7 07:57:34 UTC 2015


Quoting Michael M Slusarz <slusarz at horde.org>:

> commit 8e28721b86544127b272855dad9855bf453b51c0
> Author: Michael M Slusarz <slusarz at horde.org>
> 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 [slusarz at horde.org]



More information about the dev mailing list