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.

Ector: Fix clang warning

Invalid type used for field and implicit conversions

ETC1/2: Fix clang warnings

Unused functions.
I keep them around for reference, for whoever wants to work on
improving the encoder.

There are still some warnings. Clang is just crazy.

Evas DDS: Fix clang warning

This one was almost a real catch by clang. Too bad we don't actually
support DXT3.

Evas filters: Fix clang warnings

Evas render2: Fix clang warnings

More unused functions. Kept around for later debugging.

Eolian: Fix clang warnings

Return proper type (NULL, not bool)

Eina: Fix clang warnings

Use fabs for double values

Eina: Fix clang warnings

Remove unused static function

Evas: Fix clang warning in generated shaders file

Use const char a[] instead of const char const a[].

Edje: Fix compilation warnings with clang

Fixes T2343.

Thanks for the report.

Edje pick: Fix use after free

Fixes CID 1267458

Edje: Remove non-sensical NULL check

Fixes CID 1261450

eet - fix emile changes to not leak in eet_cipher/deciper()

coverity spotted a missing binbuf free for the in binbuf created in
CID 1288921 and CID 1288920. this fixes both of these. leak. new.

ecore_x - axis events - if event struct alloc fails, free axis data

this frees the axis data coming in if struct alloc fails - should
basically never happen, but if it does we don't lose axis data as
caller asumes this data will be freed in the event

related to CID 1256115

ecore_evas extn module - fix coverity complaint

coverity is right. CID 1295139 fixed here. tmpstr_free on sometimes
uninitialized local var.

ecore-evas-extn: Use eina_mkstemp in place of mkstemp

Summary: This fixes Coverity CID1039610 'calling mkstemp without
securely setting umask first'. Since we have eina_mkstemp which
already handles creating temporary files in the proper directory,
let's use it.

@fix

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

emile: Fix resource leak

Summary: This fixes Coverity CID1288919 where buffer variable was
being leaked if emile failed to load the image due to corrupt file.

@fix

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

emile: Fix resource leak

Summary: This fixes Coverity CID1288918 where data_start variable was
being leaked if the rectangles did not intersect.

@fix

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

ecore-evas-x: Fix issue of wrong interface function pointer being set

Summary: This fixes Coverity CID1267461 where the pointer to the
interface shape_input_reset function was being assigned multiple
times. It looks like this is just a copy/paste error.

@fix

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

ecore-drm: Use #if 0 to comment out unused function

Summary: It was mentioned on the mailing list to use #if 0 instead of
commenting the block. This does not change any functionality

@fix

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

Revert "ecore-drm: Remove 'defined but not used' function"

This reverts commit d625167e0707cca3648e9933ef65936043372447.

Reverting this to use #if 0 as suggested by q66

ecore-drm: Remove 'defined but not used' function

Summary: As we no longer require the idler to repaint outputs, comment
out the function that was defined for the idler.

NB: I left it commented (and not totally removed) just in case it is
needed for a later (undiscovered) issue

@fix

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

Evas 3d mesh: remove non-implemented functions from "implements".

They are not implemented, so they should not be marked as implemented.
Eo already properly handles functions that are not implemented, this is
completely unnecessary.

Ector: remove use of #warning directive as TODO.

There's already a fixme in code, and an error output at runtime.
As discussed on the ML, using #warning this way is bad.

eio - fix crashes due to stale eio files in tracking list

so eio_eet didnt free fiels the same as the rest of eio. it thought it
was special. it thought it could just go free() it's objects (even
though they inherit the core Eio_File object type in their structs).
everyone else ended up calling eio_file_free() EXCEPT eio_eet. so the
eio_(long)_file_set() funcs ewnded up adding this eio file to the
tracking list via eio_file_register() but never unregistered because
they really liked to just do their own thing anyway...

BAD CEDRIC! SPANKING TIME

http://33.media.tumblr.com/3422c76c33c3b9b045f623ff73e0bf8d/tumblr_mhvu61N9br1rbavngo1_500.gi

so this unifies all allocation to now use a single allocator (that
also ensures all eio async io objects are zero'd out), and a SINGLE
free path, and then it all works. no more valgrind complaints on e
shutdown/restart whilst eio "things" were used before or ... maybe
still around.

this should fix T2129

so try again "git master" after this commit.

@fix

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

emotion gst1 module - remove printf

this debugging printf seems to have been left around - remove it so we
don't get unexpected noise in gst playback

emotion - gst1 module - handle long standing bad map/unmap of gst buf

this fixes a pending XXX where we didnt keep the buffer mapped as long
as the evas image object pointed to the video pixel data. this fixes
this along with less over-zealous refinf to make things more obvious
(now last_buffer actualyl really matters and if it was mapped and
refed).

@fix