cedric pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=d23aca3c8ec60b8db68511ff484b85edadd2f9aa

commit d23aca3c8ec60b8db68511ff484b85edadd2f9aa
Author: jiin.moon <jiin.m...@samsung.com>
Date:   Wed Sep 27 11:51:44 2017 -0700

    emile: unmap the memory for image file after decode.
    
    Summary:
    In case of emile, unmap the memory of the image file when the image file is 
closing.
    But closing of image file happen when the image is flushed from the cache, 
so the
    mapped memory of image file remain for long period of time with no benefit 
(If it is
    in cache, it won't get decoded again). So I think we don't need to keep the 
mapped
    memory of image file.
    
    Test Plan: self
    
    Reviewers: cedric, jpeg, jypark
    
    Reviewed By: cedric
    
    Differential Revision: https://phab.enlightenment.org/D5210
    
    Signed-off-by: Cedric BAIL <ced...@osg.samsung.com>
---
 src/lib/emile/emile_image.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/lib/emile/emile_image.c b/src/lib/emile/emile_image.c
index 858e79058a..9a0bdb04c4 100644
--- a/src/lib/emile/emile_image.c
+++ b/src/lib/emile/emile_image.c
@@ -680,6 +680,7 @@ _emile_tgv_data(Emile_Image *image,
 
 on_error:
    if (image->compress) eina_binbuf_free(buffer);
+   _emile_image_file_source_unmap(image);
    return r;
 }
 
@@ -2325,6 +2326,7 @@ done:
 
    jpeg_destroy_decompress(&cinfo);
    _emile_jpeg_membuf_src_term(&cinfo);
+   _emile_image_file_source_unmap(image);
    return r;
 
 }

-- 


Reply via email to