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>

ecore-evas-drm: Fix issue of fullscreen surfaces not getting focus on show

Summary: This fixed T2349 where elm apps would not get initial focus
on show when running via drm. This also addresses the issue of always
sending the state_change event even when not an override.

NB: Thanks to bu5hm4n for reporting ... and even providing a potential fix ;)

@fix

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

evas-blend-ops: fix off-by-1 in the C MUL3_SYM "simd" multiplier

Summary: op_blend functions give off-by-1 using NEON intrinsics in C
MUL3_SYM, this commit fixes that issue with no ill effects on other
platforms.

@fix

Reviewers: raster, cedric

Subscribers: cedric

Projects: #efl

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

ecore_imf: init wayland before using wayland calls.

Summary:
ecore_imf: init wayland if wayland env var exists

Otherwise ecore-wl functions could return some invalid values. The env var check to avoid error messages.

@fix

Reviewers: Sergeant_Whitespace, devilhorns

Reviewed By: Sergeant_Whitespace

Subscribers: Sergeant_Whitespace, cedric

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

Evas 3d: Fix clang warning

Unused function

Edje: Silence more clang stuff

And make it explicit that the enum is unsigned

Evas tiler: Remove unused functions (clang)

rect_list_add_split_strict is still kept for reference in
eina_tiler.c

Ector: More clang silencing

Efreet: Silence clang warning

[unsigned - unsigned] is unsigned and might wrap around, the implicit
conversion to int (by calling abs()) saved the game here.

Eeze: Silence clang warning

Edje edit: Fix clang warning

Remove useless variable

Evas: Fix clang warning (actually use function)

Those helper functions are barely ever used...

Evas GL common: Fix clang warnings

Mooooarrr!

Ecore_Con: Fix clang warnings

Evas 3d: Fix clang warnings

ecore_evas_convert: Fix useless clang warning

ETC1: Fix more clang warnings

Some of these are a bit pointless (eg. init with {0})
but at least this silences clang and helps reveal more
potentially useful warnings.

Ethumb: Fix clang warning

Unused function.......

Evas/cserve2: Fix clang warning

Ector: Fix (probably valid) clang warning

This time it looks like this is a valid warning.
if (abs(double) <= 0.00001f) might too often be true since abs() returns
an int.