edje: get the color class value by iterating over all possible value provider.

edje: add edje_color_class_active_iterator_new()

This function make it possible to get a list of active Edje_Color_Class
in an application. Think about Enlightenment color class configuration,
but can now be done on any application dynamically.

@feature

edje: make Edje_Color_Class a public structure.

Evas 3D: Fix shadow map shader

Summary:
Silly mistake after this b9b5ced501f4642f03d8bcc8441fd3f7b55a7f52 commit.
Seems uninitialized components of vector gl_FragColor

Reviewers: cedric, Hermet, jpeg

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2258

edje: force NULL initialization of desc pointer.

edje: this desc pointer is actually never set.

ecore_drm: use get_vt instead of parsing the tty attribute

Summary: logind has a seperate function to get the vt number.

@fix

Reviewers: devilhorns

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2252

ecore/idler - simpler checking.

eina: include blank lines in file iterator T2237

any line that would not have been included before has length == 0

ecore_con: fix ftp upload function to follow documentation.

Summary:
While creating a Ecore_con_url object, the url is given in this format "ftp://ftp.example.com". While uploading a file, this function was prefixing "ftp://" to this url which resulted in DNS failure, and upload fail. So corrected the issue.

Signed-off-by: Srivardhan Hebbar <sri.hebbar@samsung.com>
Reviewers: cedric

Reviewed By: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2221

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

evas: fix gl_cocoa build after recent changes in evas_common_draw_context_font_ext_set.

Now evas_common_draw_context_font_ext_set takes 3 more parameters, fix the build
by adding NULL for the function pointers. We really need to switch to gl_generic
here, it would avoid this kind of problems.

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

eina: Change newline parsing to handle crlf better

Adjust test to demonstrate failure, existing code only worked for
windows newlines on blank line.
@fix T2236

edje: edje_edit - recalculate object after chenge aspect preference

ecore: trigger events on FD_READ, FD_CONNECT and FD_ACCEPT.

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

ecore_con: do not try to load /etc/resolv.conf when on windows.

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

[Evas: Evas_3D] Improve shaders math

Reviewers: cedric, jpeg

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2244

On many platforms the built-in mix(x, y, a) function faster
than its equivalent x * a + y * (1 - a)

Ecore: Use calloc() to initialize Ecore_Event_Key fully

Some fields (eg. data) may not be properly initialized.
Use calloc() instead of malloc() will ensure proper initialization
even if we add more fields.

These fields (data) would even be passed down to Evas as
Evas_Event_Key_Down for instance.

Build: Fix horribly slow compilation times

Cedric, our dear b0rker, introduced changes in the CFLAGS
generation when merging Emile. While the changes seem to make sense
at first sight (add the -I flags for the lib our new package depends on),
they were actually a terribly bad workaround.

The number of CFLAGS args would grow exponentially, slowing down libtool
a lot, which is known to be slow when it has a lot of arguments.

edje: add of textures to Edje 3D node.

Summary: Adding textures to edje_cc and some topblock keywords that I have forgotten to add in previous commits.

Reviewers: Hermet, raster, cedric

Reviewed By: cedric

Subscribers: cedric, artem.popov

Differential Revision: https://phab.enlightenment.org/D2129

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

eina: add eina_list_data_idx().

Summary:
Now the developer has to iterate the whole list to find the
index of the first occurence of the data.
I see that it is possible to get the index of the item for
the genlist widget, but for the eina_list not.

With these APIs it will be easier to implement *index_get
functions for the rest of widgets which contain items (Elm_List,
Elm_Ctxpopup etc.).

These functions returns the index of the given data or node in the eina_list.

Reviewers: Hermet, cedric

Subscribers: raster, cedric

Differential Revision: https://phab.enlightenment.org/D2189

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

ecore_con: add example for ftp upload.

Summary:
Added example for ftp upload. In the .gitignore only 2 files added which were missing. The differences it is showing is cos of reordering. I did ls and redirected the file to gitignore. So the files got reordered.

Signed-off-by: Srivardhan Hebbar <sri.hebbar@samsung.com>
Reviewers: cedric

Reviewed By: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2223

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

evas: fix logic of 180 degree image rotation in software_generic backend.

Summary: Signed-off-by: kabeer khan <kabeer.khan@samsung.com>

Reviewers: cedric

Reviewed By: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2238

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

eet: fix variable naming

Summary: this file name is used for input, not for output.

Reviewers: cedric, seoz, Hermet, raster

Subscribers: cedric, reutskiy.v.v

Differential Revision: https://phab.enlightenment.org/D2243

Wayland shm: Fix shadow variable warning

edje: add text_class_get() APIs

Summary: No APIs to get text_class for global hash and object hash

Reviewers: woohyun, cedric, raster

Reviewed By: cedric, raster

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2213

[Evas: Evas_3D] Refactor shader system: refactored shadow map shade mode.

Reviewers: cedric, jpeg

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2240

[Evas: Evas_3D] Refactor shader system: refactored parallax shade mode.

Reviewers: cedric, jpeg

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2241

Edje: Edje_Edit - code generation for TABLE part (description.table section)

Edje: Edje_Edit - fix typo in code generation of BOX part

@fix

ecore_idler: + null check.

like timer, animator we do check the idler validation
and prevert crash there just in user misusage case.

@fix

evas: fix use after free.

Thanks to Thanatermesis for tracking this in T2135.

ecore-drm: Use better output names

Summary: As there are various DVI and HDMI output names, update the
output connector types area to be more accurate so we get better
output names

@fix

Signed-off-by: Chris Michael <cp.michael@samsung.com>

Evas 3D: Print SHADERS instead of SHADERS_3D at compile time

This is just a cosmetic change.

Evas 3D: Add forgotten changes after generation of shaders

Evas GL: Add missed GLES 3.0 functions.

Summary:
Initialise glVertexAttribDivisor & glVertexAttribIPointer
EvasGL function pointers.

Reviewers: wonsik, spacegrapher, jpeg

Reviewed By: jpeg

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2236

[Evas: Evas_3D] Refactor shader system: refactored normal map shade mode.

Reviewers: cedric, jpeg

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2232

[Evas: Evas_3D] Refactor shader system: refactored  color pick shade mode.

Reviewers: cedric, jpeg

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2230

[Evas: Evas_3D] Refactor shader system: refactored diffuse shade mode.

Reviewers: cedric, jpeg

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2229

[Evas: Evas_3D] Refactor shader system: refactored vertex color shade mode.

Reviewers: cedric, jpeg

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2228

[Evas: Evas_3D] Refactor shader system: refactored flat shade mode.

Reviewers: cedric, jpeg

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2224

[Evas: Evas_3D] Fix shader code for using with GL_ES

Summary: GL_ES variables precision added.

Reviewers: cedric, jpeg

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2234

ecore_drm: Add event of ECORE_DRM_EVENT_SEAT_ADD.

Summary: To support multi-seat, ecore_drm need to issue the event to notify compositor of seat is added.

Test Plan: N/A

Reviewers: zmike, raster, gwanglim, devilhorns

Reviewed By: devilhorns

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2212

ecore-drm: Fixed null pointer free

Summary:
Fixed null pointer free
         when sd_session_get_seat() fails, it does not allocate string.

Reviewers: zmike, cedric, raster, gwanglim, devilhorns

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2219

ecore_imf: Add NULL check before accessing evas device API

evas: Add safety check for NULL pointer of clipper.

Summary: It caused crash when clipper is NULL and it makes evas test-suite fail.

Test Plan: Run evas test-suite. (make check)

Reviewers: woohyun, tasn, herdsman, Hermet

Reviewed By: Hermet

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2215

ecore-imf: Added the device information in the Ecore_IMF_Event structs

Summary:
Added the device information in the Ecore_IMF_Event structs.
Applications may require to know actual source of key event to handle it differently depending on device.
Even if key events are currently generated from different devices, application couldn't distinguish it.
This patch will support device information for the Ecore-imf key events.

Reviewers: jihoon

Reviewed By: jihoon

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2214

evas/gl_x11: add env variables for disable partial rendering

Summary: add 'EVAS_GL_PARTIAL_DISABLE' environment variables for disable partial rendering

Reviewers: cedric, jpeg

Reviewed By: jpeg

Subscribers: mer.kim, spacegrapher, cedric, wonsik

Differential Revision: https://phab.enlightenment.org/D2183

Signed-off-by: Jean-Philippe ANDRE <jpeg@videolan.org>
jpeg added atoi(s) and copied the code for gl_drm and wayland_egl.

evas/gl_x11: Separate gl symbols check

Summary:
There is a restriction for some gpu drivers that
eglGetProcAddress must be called after eglMakeCurrent.
So separate egl/glx extensions check from gl_symbols
to be called inside eng_window_new.

Test Plan: egl and glx backend tests

Reviewers: cedric, jpeg

Subscribers: cedric, mer.kim, wonsik

Differential Revision: https://phab.enlightenment.org/D2193

Evas: Fix invalid test case

No idea how it worked before. It shouldn't. It was invalid.
But somehow evas_suite decided to start crashing on this test.

[Evas: Evas_3D] Refactor shaders: used m4 for removing duplication of code.

Reviewers: Hermet, raster, cedric, jpeg

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2076

Revert "evas/gl_x11: Optimize out calls to make current"

This reverts commit 0585540bb3558b2ff834a1bb730e897c6301a224.

This broke Evas 3d examples. I also suspected some weird things and
wasn't 100% confident with this patch.

Closes T2215.
Thanks for the report.

emile: meaningful initialization of some pointers.

ecore_con: clean up, refactor and unpdate documentation.

Summary:
1. Have refactored code in ecore_con_local.c
2. Changed env variable from "TMP" to "TMPDIR".
3. Added check for negetive port number when type is ECORE_CON_LOCAL_USER.
4. Added check to check TMPDIR before assuming /tmp as temp directory.
5. Updated documentation in Ecore_Con.h explaining about local socket and port number.

Signed-off-by: Srivardhan Hebbar <sri.hebbar@samsung.com>
Reviewers: cedric

Reviewed By: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2194

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

ecore_exe: fix ecore_exe_pid_get on windows

Summary:
ecore_exe_pid_get returns exe->pid value so it should be filled

@fix

Reviewers: raster, seoz, cedric, Hermet

Subscribers: NikaWhite, cedric, reutskiy.v.v

Differential Revision: https://phab.enlightenment.org/D2195

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

eina: add test cases for various eina xattr data type functions

Summary:
Added test cases for eina_xattr_string_set, eina_xattr_int_set and
eina_xattr_double_set functions

Signed-off-by: vivek <vivek.ellur@samsung.com>
Reviewers: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2218

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

ecore-drm: Fix issue of not getting updated for output hotplug events

Summary: When setting up the Eeze_Udev_Watch for output hotplug
events, we should also be listening for the "change" event. As eeze
will now check the HOTPLUG property for drm events, we don't need to
check that here.

@fix

Signed-off-by: Chris Michael <cp.michael@samsung.com>

eeze: Fix issue of eeze_udev_watch_add not getting events for drm

Summary: This fixes eeze_udev_watch_add for DRM events, Previously, we
were not getting a watch callback fired due to using the wrong
subsystem device type in
udev_monitor_filter_add_match_subsystem_devtype.

@fix

Signed-off-by: Chris Michael <cp.michael@samsung.com>

evas: prevent crash.

if user delete the interceptor in the interceptor callback,
the interceptors would be NULL.
So we need to check whether it's valid or not.

@fix

emile: fix build on NEON enable system.

T2214

evil: fix access to a file mapping object

Patch from Jorge Luis Zapatta (turran), a bit modified
by me

@fix

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

ecore-drm: Fix build error caused by missing function declaration

Summary: This is mainly just a build fix. This issue occurred due to
the function declaration being After a call to use it while not having
a function prototype. This just adds the function prototype

@fix

Signed-off-by: Chris Michael <cp.michael@samsung.com>

ecore-drm: Add more debug info when output is created or updated

Summary: This patch adds more information so we can debug output
hotplug events better. It fixes an issue where any new outputs
from _ecore_drm_update_outputs where not getting the eeze udev watch
event created, and also addresses the issue of disconnected outputs
not sending the ecore_drm_event (both minor fixes)

@fix

Signed-off-by: Chris Michael <cp.michael@samsung.com>

evas-drm: Enable more information during flip error

Summary: This adds a bit more output during crtc flip errors so we can
debug multi-monitor setups

Signed-off-by: Chris Michael <cp.michael@samsung.com>

eo: fix wrong check libaddr dependency.

move checking libaddr to eo step to valid it.

Wayland: Fix compilation

Bad merge?

ecore_wayland:  Check for nulls before wayland calls

Summary:
Wayland API routines by policy do not check input parameters for invalid
values.  In particular, many calls are wrappers through
wl_proxy_marshal, which derefs its first argument without a check.

So, for all wayland calls, always null check the first argument.

Reviewers: zmike, cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2211

ecore_wayland: Check for null inputs

Summary:
The other functions in this module check inputs; this one should as
well.

Reviewers: zmike

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2210

ecore_wayland: Add some comments

Summary: Signed-off-by: Bryce Harrington <bryce@osg.samsung.com>

Reviewers: zmike

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2209

Revert "evas-3d: materials in .mtl format are deleted"

This reverts commit 59271716ba5443bcba215f5dfbc190fc107ae4ab.

This patch submitted twice.

evas render2 - restructure it to be an explicit api call - cleaner to do

evas-3d: materials in .mtl format are deleted

Summary: Materials are not used in any loader and will not be used in nearest future.

Reviewers: raster, Hermet, cedric

Reviewed By: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2080

Conflicts:

src/examples/evas/Makefile.am

evas: fix description about Evas_Device_Subclass

Fix wrong drawing of shadows for some blending modes

Reviewers: cedric, Hermet

Reviewed By: Hermet

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2198

emile: Remove unnecessary check in _emile_jpeg_head function

Summary: Already checked it by "else if ((opts->w > 0) && (opts->h > 0))"

Reviewers: cedric, Hermet

Reviewed By: Hermet

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2199

evas: fix build failure with old version freetype.

Summary:
Fix build failure with old version freetype.
It is caused for supporting colored font.

Reviewers: raster, jpeg, woohyun, Hermet

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2200

evas - font - new color bitmap support - rename funcs to match file

evas-wayland-shm: Fix redrawing issues that were causing 'flashing' in latest engine code.

Summary: As reported by derek & zmike, the previous engine refactor
left some issues where the app would 'flash' during redraws due to
buffer handling. This commit fixes that issue along with flashing
during resize.

@fix

Signed-off-by: Chris Michael <cp.michael@samsung.com>

evas-software-generic: Allow future rendering calls if evas has to skip a frame

Summary: If we do not get a new_region_for_update from an engine, it
could mean that the engine has skipped a frame. In that case, we still
want to try and render any future frames. Before this commit, evas
would never try to re-render anything again once it hit a failed
frame. To fix this, we test for a valid returned region, and if we
don't get one, mark this frame as failed so we can try again.

NB: Thanks for the help Cedric !! :)

@fix

Signed-off-by: Chris Michael <cp.michael@samsung.com>

evas: use switch instead of a serie of if for cleaner example.

evas: remove unused variable.

evas: remove useless line.

elua: cast data, not cbs (data stores the cb id)

Revert "elua: more reliable eoid->lua number conversion"

This reverts commit 8bb13b626f3a5d284f50af493f3f7b5566e06586.

Seems like this method is not so reliable after all :) Results
in precision loss for Eo IDs, so revert to the old hacky way.

evas-gl-common: Fix building when --with-opengl=es

Summary: If we are building for only gles 2.0, then we cannot use
glAlphaFunc else we get compile errors. This adds an #ifndef around
that block to fix building

@fix

Signed-off-by: Chris Michael <cp.michael@samsung.com>

Edje: edje edit - ability to remove last item in BOX/TABLE

Removing last item in BOX/TABLE part is actually successful
but then we shouldn't realloc an array of items for 0 items.
That's ridiculous
(and because of that function removes EINA_FALSE,
so user could think that it's unable to remove last item).

So simply array can be set into NULL.

@fix

Eolian: add API to determine if a function is implemented.

This is needed in Erigo to determine if a virtual pure function can be
invoked via eo_do, as we don't want error messages to be displayed in
case of a non implementation in the Eo object.
This function works with non virtual functions too.

Tests have been provided.

@feature

evas: Support bitmap embedded color font.

Summary:
Add the code for getting bitmap buffers from embedded color font,
And draw the bitmap buffers as images.
For drawing the bitmap buffers as images,
evas_common_draw_context_font_ext_set internal API is changed to
pass additional gl engine functions.
T2139
@feature

Test Plan:
1. Set a bitmap embedded color font to textblock.
2. Set a unicode emoticon text.
3. See the result.

Please check the sample unicode in the following link.
http://www.fileformat.info/info/unicode/char/1f3af/index.htm
http://www.fileformat.info/info/unicode/char/1f555/index.htm
http://www.fileformat.info/info/unicode/char/2600/index.htm
http://www.fileformat.info/info/unicode/char/263a/index.htm

Reviewers: tasn, woohyun, jpeg, raster

Reviewed By: raster

Subscribers: herdsman, cedric

Differential Revision: https://phab.enlightenment.org/D2084

evas/font: Add evas_font_reinit API.

Summary:
Reinitialize FontConfig. If FontConfig has to be reinitialized
according to changes of system enviroments(ex. Changing font config files), it will be useful.

Reviewers: woohyun, seoz, tasn, cedric, raster

Reviewed By: raster

Subscribers: raster, herdsman, cedric

Differential Revision: https://phab.enlightenment.org/D1962

evas/gl_x11: set EGL_PLATFORM environment variable

Summary:
Without EGL_PLATFORM environment variable, eglInitialize() can be
failed because egl tried to load DRM platform instead of X11 platform and it
tried to handle XDisplay pointer as a gbm_device pointer as well.
The failure seems to be occured especially if the egl was built
with DRM platform as native platform.

This revision can prevent the failure by indicating proper egl platform using
EGL_PLATFORM environment variable, and additionally it will unset
EGL_PLATFORM environment value when the engine free its info.

@fix

Reviewers: gwanglim, seoz, jaehwan, cedric, raster

Subscribers: raster, cedric

Differential Revision: https://phab.enlightenment.org/D1844

eina: safety check for NULL pointer

data should not be NULL. it is used as the source of memcpy.

@fix

Spelling fixes

Summary:
emmited  ==> emitted
resistence  ==> resistance
occured  ==> occurred

Reviewers: cedric, zmike, devilhorns

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2192

ecore_con: add tests cases to cover more scenarios in local sockets.

Summary:
In this patch, ecore_test_ecore_con_local_system_negport test case was failing.

Note that this patch does play with some environment variable to check more
code path in ecore_con.

Signed-off-by: Srivardhan Hebbar <sri.hebbar@samsung.com>
Reviewers: cedric

Reviewed By: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2176

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

ecore_con: add test case for download related functions.

Summary:
This has test cases for download related functions. Will make url
configurable and add post/upload related functions in the coming patch.

Signed-off-by: Srivardhan Hebbar <sri.hebbar@samsung.com>
Reviewers: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2175

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

ecore_con: fix ECORE_CON_LOCAL_SYSTEM use with negative port number.

Summary:
The socket can be created even with negative port number, but in that case the port
is ignored so that you can connect to non Ecore_Con based IPC. This patch remove
that test to make the client and server match.

@fix

Bug: While creating ECORE_CON_LOCAL_SYSTEM server and client pair, when the socket
name was "test_socket" and port number "-8" (Any negative number). Then while creating
listening socket, the code would go to line no 291 and socket is created in tmp by
".ecore_servicetest_socket|-8". When the same is passed to bind then the code would
go to line 118 and the socket it would try to bind would be ".ecore_servicetest_socket" !!
So the bind would fail.

Signed-off-by: Srivardhan Hebbar <sri.hebbar@samsung.com>
Reviewers: cedric

Reviewed By: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2186

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

evas: Evas_3d proxy example uses cube primitives.

Reviewers: Hermet, raster, cedric

Reviewed By: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2172

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

evas: pick example uses sphere primitives.

Reviewers: cedric, Hermet, raster

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2169

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

evas: frustum example uses sphere and cube primitives.

Reviewers: cedric, Hermet, raster

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2167

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

evas: Evas_3D - add alpha testing.

Reviewers: Hermet, cedric

Reviewed By: cedric

Subscribers: Ievgen, cedric

Differential Revision: https://phab.enlightenment.org/D2046

evas: cube-rotate example uses cube primitives.

Reviewers: Hermet, raster, cedric

Reviewed By: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2165

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

evas: cube2 example uses cube primitives.

Reviewers: Hermet, raster, cedric

Reviewed By: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2164

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>