evas-gl common: clarify cryptic error message

ecore-evas-wayland: Fix issue of setting NULL cursor object

Summary: If we are calling ecore_evas_object_cursor_set with a NULL
object, then we need to inform the ecore_wayland window that we no
longer have a cursor surface.

@fix

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

evas-drm: Remove debug noise

Summary: No real function changes, just commented out some debug so we
have less noise for release.

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

ecore-drm: Remove debug noise

Summary: No real function changes, just commented out some debug lines
so we have less noise for release

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

eolian: fix lexer assertion failure

The function was missing an "else", so a wrong branch was always
executed with errors at EOF. This fixes that.
@fix

eolian: specialize tmp free for typedef (avoid double free issue)

@fix

ecore-drm: close fd when asked to close fd

Summary:
the logind device close function just lets logind revoke fd permissions
but doesn't actually close the (now useless) fd.

@fix

Reviewers: devilhorns

Reviewed By: devilhorns

Subscribers: cedric

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

ecore-drm: don't return EINA_FALSE as an invalid fd

Summary:
0 is a valid fd.  EINA_FALSE is 0.  EINA_FALSE is a valid fd.

@fix

Subscribers: cedric

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

ecore-drm: append O_CLOEXEC always

Summary:
Seems libinput wasn't bothering to set this so all child processes get
their hands on input devices

@fix

Subscribers: cedric

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

ecore_drm: close fd if flag set fails

@fix

Subscribers: cedric

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

ecore-evas-wayland: remove duplicated shutdown of ecore_evas_wl_common

Summary:
 After a creation for ecore_evas_wayland_shm or ecore_evas_wayland_egl is
failed, _ecore_evas_wl_common_shutdown() gets called 'twice'.

 First shutdown is from _ecore_evas_wl_common_free() which is set as fn_free()
of ecore_evas engine, and this fn_free() is called by ecore_evas_free().
 Second is from direct call inside ecore_evas_wayland_shm_new_internal()
(or ecore_evas_wayland_egl_new_internal()).
 This duplicated shutdown causes block of first next _ecore_evas_wl_common_init
because init count is not zero but -1.

 So second shutdown should be removed so that it makes enable fall back to
another wayland engine correctly when one is failed to create ecore_evas_wayland.

@fix

Reviewers: gwanglim, devilhorns

Reviewed By: devilhorns

Subscribers: cedric

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

evas-wayland-shm: Fix T2352 (focused window borders blink)

Summary: This fixes an issue where the wrong swapmode was being
returned to the evas render function. This was causing focused windows
to blink.

NB: Big thanks to Derek for assisting !! :)

@fix

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

ecore-x: use async pointer grab mode in ecore_x_window_key_grab()

using a sync grab here causes the device to be frozen in the xserver,
preventing any event processing on the device until the grab ends. it also
prevents new grabs, such as the ones which would be created by xkill when
launched from a keybinding; this means that as long as the grabbed key is
held, the pointer is effectively dead to the world.

I attempted to find the reason for using sync pointer grabs here,
but the original commit message was:

  aaaahmmm.. no.. this.. oops

see https://git.enlightenment.org/legacy/ecore.git/commit/?id=f62faeae288fc8211d5f97b43d3b81de87db0942

fix T2228

evas-wayland-shm: Update software_generic render engine size on output resize

Summary: As we override the software_generic function for
eng_output_resize, so we can handle resize edges, we should also be
updating the software_generic's render engine size here because the
function for software_generic->output_resize never gets called.

@fix

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

evas-wayland-shm: Minimize race condition when finding free leaf

Summary: This fixes a possible race condition that could happen when
using more than 2 buffers. Instead of searching for a free buffer in
various places, find a free buffer when evas requests the buffer data
and save that free buffer as the 'current buffer'

@fix

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

evas-wayland-shm: Use size of buffer when caching

Summary: evas_cache and evas_cache2 are expecting the width of the
buffer when caching so just use that directly rather than calculating
bpl.

@fix

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

ecore-drm: add touch_motion event on touch_down event handler.

Summary:
  ecore-drm touch down handler does not produce ECORE_EVENT_MOUSE_MOVE event.
  so, when user touches screen, application can't handle mouse down event correctly.
  this fix adds touch_motion event on touch_down event handler.

Reviewers: zmike, cedric, raster, gwanglim, devilhorns

Subscribers: cedric

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

ecore_drm: send a fake motion event to let other know the initial pos of mouse

Summary:
At the first time after finishing booting, the position of mouse cursor is in the
center of screen. But e doesn't know it. ecore_drm should let e know the initial
position of cursor by sending a fake motino event.

Reviewers: devilhorns, ManMower, zmike, raster, gwanglim

Subscribers: cedric

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

evas-drm: Fix evas-gl_drm picking incorrect resolution/mode for outputs

Summary:
Fix evas-gl_drm picking incorrect resolution/mode for outputs
   reference,  commit: d3a207bf9ab1246b5a5299ade7b25ee46a8e3cf7

Reviewers: zmike, cedric, raster, gwanglim, devilhorns

Subscribers: cedric

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

ecore-drm: Fix using uninitialized output value

Summary:
This revision fixed using of uninitialized value 'output'.
The usage can cause segmentation fault when it try to refer current_mode

@fix

Reviewers: devilhorns

Subscribers: cedric

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

emile: only disable tests timeout on Windows.

eina: in fact this is gone and won't come back.

Eo and Efl.Model are here for that task.

Eina: fix eina_file_split on Windows and update unit test

@fix

Signed-off-by: Jean-Philippe Andre <jp.andre@samsung.com>

Edje embryo: Fix invalid check for get_state_val (text str)

@fix

eina: fix eina_stringshare format warning after commit 912924a

Summary:
Since the type of 'r' changed from unsigned short to unsigned int we
need to change its format as well.

Reviewers: stefan_schmidt, raster, cedric

Reviewed By: cedric

Subscribers: cedric

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

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

ecore-drm: shutup libinput log info

tests/ecore_con: Avoid warining about _parse_url being unused

It is only used inside two ifdef conditions. Better check them for
_parse_url as well.

eolian: fix tests after the EAPI switch

elocation: allow usage of it with C++.

eolian: actually these should be marked EAPI

eolian: mark _class_get func protos with EOAPI

this is necessary to properly export the funcs on Windows. @fix

autotools: fix non-verbose build output

D2431 added some mkdirs that were printed even when not in verbose mode.
This commit fixes that; those mkdirs will stay hidden (unless verbose).

Bad cedric! @fix

ecore_con: fix double free issue in ecore_con_eet.

Summary:
While creating ecore_con_eet_server, if client is not set to false, then while deleting, it is checking for client and it is by default not false. So set it.
ecore_con_reply object is being deleted twice. So removed code to delete the reply object as it would be deleted after the callback.

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

Subscribers: cedric

Maniphest Tasks: T2372

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

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

pkg-config: prefix eolian_flags with sysrootdir when crosscompiling

Otherwise, the elementary build is brocken while crosscompiling since
"pkg-config --variable=eolian_flags eo evas edje ecore efl" return paths
to the host's include directories (/usr/share/eolian/include).

Signed-off-by: Romain Naour <romain.naour@openwide.fr>
Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

autotools: add --disable-libeeze

When cross-compiling, we only want to build edje_cc, embryo_cc
and eet binaries for the host before starting the build for the
target.

This patch allows to disable libeeze in order to shorten the
build time but most of all remove the dependency on libudev.

In normal case it's not recommended hence a warning.

Signed-off-by: Romain Naour <romain.naour@openwide.fr>
Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

ecore_file: add more tests for ecore_file.

Summary:
This patch adds more tests for ecore_file.
It helps examine unchecked cases.

Reviewers: Hermet, cedric

Reviewed By: cedric

Subscribers: cedric

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

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

eolian: fix build of generated Eolian files.

Summary: Fix build with properly creation of output dir for eolian_gen on out of source build when passing the option --disable-dependency-tracking.

Reviewers: cedric

Subscribers: cedric

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

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

eio: fix compilation warnings in eio_test_xattr.c when xattr tests are disabled.

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

Reviewers: cedric

Subscribers: cedric

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

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

eio: add new tests for eio.

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

Reviewers: cedric

Reviewed By: cedric

Subscribers: cedric

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

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

emotion: trying to fix gstreamer backend build on Windows.

ephysics: trying to fix build on Windows.

Ecore audio: Fix (skip) build on Windows

Thanks vtorri for the report

eina stringshare - make short internal string refs ints not shorts

this should fix T2088 - at the expense of more memory but a short was
just too little here.

ecore-wayland: Fix issue of efl/elm apps not setting opaque & input regions

Summary: Since evas handles surface damage & surface commit roles now,
we no longer are calling ecore_wl_commit inside ecore_evas. This was
causing the surface opaque & input regions to no longer be set. We fix
that here by creating and setting opaque/input regions whenever the
appropriate ecore_wl region functions are called.

@fix

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

evas: fix issue of using two times the same image with different orient in software.

Fixing T2338

ecore-wl: reject key events which do not correlate to a valid keysym

release: Update NEWS and bump version for 1.14.0-beta3 release

Evas masking: Free mask surfaces during evas dump

On minimizing, free all mask surfaces. This could save a lot
of memory.

Also, write "proxy" COW only when there is a surface (the "proxy"
pointer itself is always valid and non-NULL).

emile: correctly free buffer allocated for compressed image.

CID 1295688

ecore_evas: move docs from .c to .h and make it consistence

Summary:
There are some documents in .c file. This patch moves them to .h
file. This also uses primitive verbs in brief section to make document
more consistence.

Reviewers: Hermet, cedric

Subscribers: cedric

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

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

eet: fix interger overflow

CID 1256196

emile: fix memory leak.

CID 1288918 & 1288919

evas: get rid of clobbered warning.

ecore_con: fix typos and improve document consistence.

Summary:
This patch fixes typos and uses primitive verb in brief section
to improve document consistence for ecore_con.

Reviewers: Hermet, cedric

Reviewed By: cedric

Subscribers: cedric

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

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

eina: correct document for eina_hash_list_prepend/remove.

Summary:
The eina_hash_list_preprend/remove API have wrong description.
This patch fixes it.

@fix

Reviewers: Hermet, cedric

Reviewed By: cedric

Subscribers: cedric

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

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

emile: fix memory leak.

CID 1288924 & 1288923

emile: fix typo.

emile: opts can be NULL, so checking it before using it.

CID 1288927

emile: check that we actually do have option before using them.

CID 1288929

emile: fix support of AGRY88.

CID 1288931

evas: fix usage of the same ressource in the GL backend with orient.

When we set orient we actually need to return a new image or it will confuse
evas a lot.

ecore-drm: fix multiclick interval time

libinput returns timestamps in milliseconds as int, not double

ecore-drm: fix improper conditional syntax in _ecore_drm_tty_switch()

ecore-drm: add safety checks to all EAPI functions

ecore-con: try to load libcurl.so.5 before .so.4

Curl is using a strange soname bumping where soname version is changed by configure when it detects some differences with off_t.
Explanations here: http://curl.haxx.se/dev/readme-curl_off_t.html

As we are not using curl_off_t in our code, we can simply load libcurl.so.5 if present.

edje_util: fix memory leak

Summary:
list returned by evas_object_box_children_get should be freed

@fix

Reviewers: cedric, seoz, raster, reutskiy.v.v, Hermet

Subscribers: cedric

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

Evas GL common: Avoid excessive pipe flushes in image free

This should boost performance a little bit when an image is
just unref'ed but not deleted (thus doesn't need a pipe flush).

Evas GL masking: Improve performance with scaled images

The GL scaled images is a fast path for masking where
the shader scales masks on the fly.
This optimization actually fixes some issues where the current
texture binding was incorrect.

Fix ecore_con_eet_client_example example compilation error

Added eet to include flags for Makefile.am for ecore examples

Fix uninitialized member by coverity CID 1294558

Initialize the scope member in the default constructor.

ecore-drm: refactor _ecore_drm_output_backlight_init() to use fewer gotos

ecore-drm: use eeze_udev_find_by_filter() to only return relevant backlight devices

ecore-drm: restrict output backlight detection to drm device

previously this would pull in any available backlight device regardless
of whether it was actually attached to the device or output in any way

evas_text: fix default return value.

@fix

ecore-file: use hash list for inotify monitor tracking

inotify reuses watches for multiple instances on the same file, so
keeping a multimap here is necessary to avoid hash collision

fix T2360

ecore_timer: fix the default return value.

When this function fails to get the interval value, it should return -1.0.
Currently, the value can be integer(-1.0 has an Error).
Maybe it should be fixed.

@fix

eina thread_queue: fix typo in doc.

ecore-drm: fix uninitialized output mode pointers when creating outputs

ecore-drm: fix ecore_drm_devices_get() to return a const value

this is the internal device list, we're pretty hosed if someone decides
to free it because there's no const

efreet: fix leak in efreet_util_desktop_exec_find()

@fix

ecore_drm: Don't crash if a device is assigned a bad output

Summary:
If an input device is assigned via udev to an output that we can't find
we just assign it to the first output instead.

Reviewers: zmike, devilhorns

Subscribers: cedric

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

eina thread_queue: fix typo in doc.

efl: Move test for gl library to evas-gl-drm engine checks and Fix T2167

Summary: Since we won't need gl library unless we are building
evas-gl-drm engine, move the checks for with_opengl config option to
the gl_drm engine checking macro. Also, since gl_drm is ONLY supported
if we have --with-opengl=es configure option, let's error out with a
message if someone passes anything Other than with_opengl=es.

@fix

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

ecore: build newly added Ecore_Con examples.

ecore_con: example program for ecore_con_eet.

Summary:

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

Subscribers: cedric

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

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

eio: add testcases for various eio_xattr types set and get functions.

Summary:
Added test cases for eio_file_xattr_string_set, eio_file_xattr_string_get, eio_file_xattr_int_set, eio_file_xattr_int_get, eio_file_xattr_double_get, eio_file_xattr_double set functions

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

Reviewed By: cedric

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

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

ecore_con: add another test case to cover remaining functions in ecore_con_eet.

Summary:
When I checked the Jenkins report, I saw few of the functions were left off. So added another case and modified the existing case to cover all the functions or ecore_con_eet.

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

Reviewed By: cedric

Subscribers: cedric

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

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

ecore_con: add test cases to cover more ecore con functions.

Summary:
The dns lookup test would result in a crash unless https://phab.enlightenment.org/D2398 fix.
Also added more functions to existing case to cover ecore_con.c

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

Subscribers: cedric

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

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

eina: add test cases for eina file xattr functions.

Summary:
Added test cases for eina_file_xattr_get and eina_file_xattr_value_get functions

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

Subscribers: cedric

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

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

ecore_con: fix dns lookup crash issue.

Summary:
When executed ecore_con_lookup_example function, we were getting a crash. This is the fix for the crash. I analysed it in the valgrind and found the error. Valgrind reported invalid free at this place.
svr->name is getting freed in _ecore_con_server_eo_base_destructor function and svr is getting freed in ecore_con_shutdown function.
Signed-off-by: Srivardhan Hebbar <sri.hebbar@samsung.com>
Reviewers: cedric

Subscribers: cedric

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

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

ecore_con: fix ecore_con_lookup_example to actually exit when done.

Summary:
Without ecore_main_loop_quit the example won't end. So added that and as ecore_init is done in ecore_con_init removed that.

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

Subscribers: cedric

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

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

eina: add test case for eina_strbuf_string_free function.

Summary:
Added test case for eina_strbuf_string_free function in eina_strbuf module

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

Subscribers: cedric

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

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

evas: fix rotation and flipping of image in gl engine

Summary:
fix rotation(90, 180, 270) and flipping(vertical, horizontal, transpose,
transverse) of evas image in gl engine backend.

@fix

T2338

Signed-off-by: kabeer khan <kabeer.khan@samsung.com>
Reviewers: cedric, jpeg

Subscribers: cedric

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

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

Eldbus: fix potential unsigned int underflow.

Subtracting two unsigned values results in an unsigned value, which means
abs() will have no effect.
This also means that if base is smaller than size, we'll have an
unsigned int underflow.

I did it this way (with the if) because it looked like base might be
smaller than size. However, please remove the if (and don't reinstate
the abs) if this is not the case.

Ecore_evas wayland: remove double parenthesis around comparison.

Double praenthesis should be used around assignments, not comparisons.
Doing this correctly all aronud lets the compiler warn us about potential
mistakes.

Evas text utils: move the non-ot function into the current ifdef.

This function should only be build when OT is off. Move
it to the #else part of the ifdef.

Eina safety: remove unused error string.

At the moment there are no errors possible for safety checks.
This may change in the future (unlikely), so we have the eina_error
already exposed for that. However, there's no need for the string.

One more down towards a warningless build.

Eolian generator: remove unusued variable.

Edje smart: Add NULL safety

This case actually happens with E. It's invalid but still happening,
and because Eo doesn't actually check the parent (or its type) fully,
an invalid, non-NULL object was returned.

See T2336 (this is not fixing the root cause!).

ecore-drm: Change ecore_drm_fb_set warning level

Summary: This is not actually "critical", it just means that we tried
to set a dumb buffer of the wrong size. Ecore_Evas will call this
function again after the canvas has been resized to the proper size,
so just issue a WRN here.

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