evas: follow API documentation more closely.

Ecore exe win32: Fix double-free errors.

As reported by vtorri, sometimes ecore_exe on win32 will encounter double
free issues. This was because the variable was freed, but not set to NULL
as expected by the cleanup function.

Fixes T2675

@fix

Revert "Revert "Revert "evas: fix borked rotation support in GL engine."""

This reverts commit f51168f4ec7a1bd3072428c7a271ad7c6715fde5.

if this is right or not... this RESULTS in enlightenment becoming
totally unusable. put this back WHEN other issues are fixed you think
that exist. last i knew we query yinvert start with glx per native
surface and use that info - so as best i know it is and has been right
for a long time.

:)

evas: fix compilation error

This is introduced in 5c02935a866a5c14

Revert "Revert "evas: fix borked rotation support in GL engine.""

This reverts commit 5bb1c52e3a075d3a69b5baee2f2ad8d2788655b0.

This patch is actually correct, but it highlight a bug in how we
retrieve GLX_Y_INVERTED_EXT value.

evas: correctly get GLX_Y_INVERTED_EXT.

ecore: mimic UNIX code even more

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

ecore_exe: avoid thread doing nothing by adding a small sleep

This fixes the CPU to be usedat 100% for each thread in ecore_exe. This
is obviously not an ideal fix and will be improved in the future.

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

efreet: use ecore_file function instead of stat() in the icon cache code

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

ecore_file: remove whitespaces

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

ecore_file: fix ecore_file management function on Windows

On windows, stat() returns -1 if a path is finished with a \ or /,
so replace all stat() calls with a function which removes the trailing
slash or backslash on Windows

At this stage the code duplicate many code path for avoiding potential
borkage on Unix system. During 1.17 release cycle, it would be nice to
refactor this piece.

@fix

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

ecore_exe: close handles only if they are valid

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

efreet: eina_file_mkstemp() needs a template with an extension

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

efreet: use eina_file_mkstemp() instead of mkstemp() for portability

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

edje: use eina_environment_tmp_get() instead of the TMPDIR env variable

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

ecore_exe: fix command line when a shell is used, and increase buffer to 32768 bytes

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

ecore_exe: remove infinite loop when killin process

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

ecore_exe: use macro to free resources

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

ecore_exe: better error message when child process can not be run

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

ecore_exe: make sure that ReadFile() will not hang

When child process exits, ReadFile() will hang if those pipes are maintained

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

ecore_exe: read remaining data when the child application has closed

Output and error threads could not read all the data sent by the child.

Based on a patch by Guillaume Friloux

@fix

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

ecore: lower log error level. Too much noise otherwise

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

ecore: mimic Win32 handlers deletion with Fd ones

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

ecore_con: remove debug

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

Ecore_Con: make sure to create named pipe with a unique name

Named pipes created with CreateNamedPipe() must have a unique name,
so append the process Id to the name

@fix

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

eina: fix potential security issue.

This also follow the behavior of eina_file_mkstemp.

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

Revert "evas: fix borked rotation support in GL engine."

This reverts commit 6153471ac3e209d86c21205ceedb97fdb8ce903b.

YES. YOU BROKE SOMETHING.... EVERY SINGLE COMPOSITED WINDOW IS NOW
UPSIDE-DOWN!

eina evlog - use CORRECT clock (monotonic) not gettimeofday if possible

evas: fix borked rotation support in GL engine.

This bug was introduced by b0d2643f933bca2d959b873701c85c6f237328e9. I am
not sure I am not breaking that one instead now. Hopefully not.

ecore_con: be more defensive while using environment variable provided by the system.

This fix a crash during make check for system where one of those string where not defined.

ecore_x: removing useless assignment pointer.

Summary:
The assignment to size has no effect on the caller and compiler
complain about it. Do shutdown compiler complain in a more reliable
way.

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

Reviewers: cedric

Subscribers: cedric

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

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

eina_tiler: add test cases for eina tiler area set/get functions

Summary:
Added test cases for eina_tiler_area_size_set and get functions

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

Reviewed By: cedric

Subscribers: cedric

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

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

eina_queue: add test cases in eina thread queue module

Summary:
Added test cases for eina_thread_queue_parent_get and eina_thread_queue_fd_get
functions

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

Reviewed By: cedric

Subscribers: cedric

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

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

edje_cc: fix segfault when a program attempts to play a non-registered sound

Summary:
When an edje program attempted to play a sound that was not registered
(e.g. in a sounds{} block), edje_cc would segfault instead of throwing
an error.

@fix

Reviewers: cedric

Reviewed By: cedric

Subscribers: cedric

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

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

ecore-evas-wayland: Trap for subsequent resize events

Summary: If we have already resized this ecore_evas to be what we
want, then there is no point in running the below resize code as we
should already be at the requested size. Add a test at the beginning
to see if we have already set these values

@fix

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

ecore_x: Optimize _ecore_x_modifiers_get()

Summary:
_ecore_x_modifiers_get() causes multiple XGetModifierMapping requests to the
server through _ecore_x_key_mask_get() function. _ecore_x_key_mask_get()
requests XModifierKeymap pointer every time it gets executed.
This can be optimized by a single XGetModifierMapping request in the
_ecore_x_modifiers_get() function itself and then passing the XModifierKeymap
pointer thus retrieved to the _ecore_x_key_mask_get() function.

Reviewers: raster, Hermet, tasn, zmike

Subscribers: yashu21985, alok25, sachin.dev, singh.amitesh, cedric

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

evas-drm: remove useless condition

Summary: 'ob->priv.last > ob->priv.last' is always false

@fix

Reviewers: devilhorns, stefan_schmidt

Subscribers: cedric

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

ecore-evas-wayland: Fix common window configure callback to not adjust
for framespace

Summary: As we have already adjusted for framespace in various code
leading up to a configure callback, don't adjust for it here. This
fixes an issue where xdg surface window geometry would get incorrect
values which were including framespace. The values of the
xdg_surface_set_window_geometry should be Just the geometry of the
visible window.

@fix

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

ecore-evas-wayland: Don't resize ecore_wl window to include frame size

Summary: ecore_wl_window_resize adjusts the internal representation of
the window size so this should not include the "framespace" also. This
fixes an issue where xdg surface would be getting an incorrect window
geometry which included framespace.

@fix

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

ecore-wl: Don't update window size during ecore_wl_window_resize
function

Summary: As we call ecore_wl_window_update_size from various places
when needed, don't call it inside the resize function. This addresses
an issue where sizes for the Ecore_Wl_Window may be out of sync

@fix

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

eolian generator: cleaner file write

eolian generator: merge legacy and eo header gen entry points

release: Update NEWS and bump version for 1.16.0-beta1 release

Evas filters: Fix some build warnings

Thanks Tom for spotting one of those

Evas filters: Fix some crash and reduce insanity

In a rare situation the filter would access an invalid buffer.

Solution: Stop messing with buffer references by properly
referencing and releasing them when not needed, rather
than stealing references and hoping for the best. (There were
flags tracking stolen references, but that was still madness)

Evas: Add engine function image_ref

This is to simplify and fix filters. This is an internal function.

Evas filter: Fix proxies of filtered images

@fix

Evas render: Don't render deleted / invisible snapshots

Don't render them. They are not visible

Evas render: Minor debug code improvements

Evas snapshot: Fix proxies of snapshot objects

This was broken because the wrong image size was used
in the proxy's fast path (proxy of another image).
Why did snapshot use a specific surface_w,h instead of
reusing the usual cur->image.w,h? (@cedric)

Simplify code.

eeze - fix if formatting to be consistent within file

some if's were single line:
  if (x) blah();

others were
  if (x)
    blah();

make formatting consistent at any rate.

same for EINA_LIST_FREE() blah();

fix tom's fix by nmot making e segfault EVERY EXIT

when fixing something like a leak .. it might be good to fix it to not
CRASH 100% of the time. a leak (minor) is better than a crash by far.

the strings in the list are stringshared.

fixes 896c18045aafb9c677d8d41b4876c47db2b4d5ce

Ecore_xcb_keymap: Fix memory leak in _ecore_xcb_keymap_finilize

Summary:
xcb_get_modifier_mapping_reply_t *reply is obtained from
xcb_get_modifier_mapping_reply and should be freed after use.

Reviewers: raster, Hermet, tasn, zmike

Subscribers: singh.amitesh, yashu21985, alok25, sachin.dev, cedric

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

@fix

eolian-cxx: Fixed possible exception escaping main

Added try/catch around main to show the reason for the generation failure.

@fix CID 1265600

eolian-cxx: Fixed assert which was a tautology

Fixed the assert to the correct check

@fix CID 1316018

Evas engine wayland_shm: pass the correct enum value.

Following this patch, the correct enum type is now passed to this
function. It's also passing the same value (0), but now from the correct
enum. This doesn't change behaviour, and looks like what was intended.

This fixes the clang warning.

ecore-wayland: Comment out unused functions

Summary: As we do not listen for surface enter/leave events, we do not
need these functions so comment them out.

Thanks to Tom for the report :)

@fix

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

ecore-wayland: Comment out unused functions for session listener

Summary: As session listener is currently disabled in Ecore_Wayland,
we don't need these functions defined so comment them out.

Thanks to Tom for the report :)

@fix

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

Evas engine software generic: Fix clang warning.

Double parenthesis comparison is reserved for evaluation for assignment, not just comparison.
So for example you should do ((x = 2)) and (x == 2), but not ((x == 2)) or (x = 2).

Ector renderer: fix return value to be of the right type.

This function expects a pointer as a return value, not a bool.

@fix.

Ethumb client: Fix shadow issues (warnings).

Eeze sensor udev: Fix memory leak and possible crash.

Due to a misplaced semicolon, EINA_LIST_FREE was doing nothing and only
the last part was freed if there was one, and probably a crash if there
wasn't.

This shows again that having statements without {} is dangerous. We need
to force {} and add a lint rule, or at the very least, add a rule to put
the ; in a new line (like clang warns about by default anyway).

(Same as issue found in previous commit)

@fix

Edje cc handlers: Fix memory leak and possible crash.

Due to a misplaced semicolon, EINA_LIST_FREE was doing nothing and only
the last part was freed if there was one, and probably a crash if there
wasn't.

This shows again that having statements without {} is dangerous. We need
to force {} and add a lint rule, or at the very least, add a rule to put
the ; in a new line (like clang warns about by default anyway).

@fix

Ector renderer: fix return value to be of the right type.

This function expects a pointer as a return value, not a bool.

@fix.

Eio model test: initialise pointer (clang warning).

This is probably not an issue because the function should always
return a value, but initialising this variable silence clang's
warning and is not harmful anyway.

Eo: Fix Eo on Windows.

We use function names instead of function pointers of Windows, because
of dll import/export issues (more in a comment in eo.c). Before this
commit we were comparing the pointers to the strings instead of the
content in some of the places, which caused op desc lookup not to work.
This fixes that.

Thanks to vtorri for his assistance.

@fix

evas_canvas3d: fix mesh being added twice added to scene data

Summary: @fix

Reviewers: cedric, Hermet

Subscribers: cedric

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

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

ecore_con: preventing possible NULL pointer dereference.

Summary:
Checking if client is NULL before freeing is not of much use. If it is NULL, it would have crashed before when it was dereferenced in printf. So checking NULL before dereferencing.
Signed-off-by: Srivardhan Hebbar <sri.hebbar@samsung.com>
Reviewers: cedric

Reviewed By: cedric

Subscribers: cedric

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

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

ecore_exe: correcting spelling mistake in comment.

Summary:
Corrected the spelling mistake of mechanism.

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

Reviewed By: cedric

Subscribers: cedric

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

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

ecor_xcb: removing useless assignment.

Summary:
The assignment of NULL will have no effect on the caller. So removed that statment.

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

Reviewed By: cedric

Subscribers: cedric

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

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

ecore_xcb: initializing the uninitialized variable.

Summary:
realwin is uninitialized and is being checked against 0 at 384. It is set at 377 only if win2 == win, else it remains uninitialized. So initializing it to NULL.
Signed-off-by: Srivardhan Hebbar <sri.hebbar@samsung.com>
Reviewers: cedric

Subscribers: cedric

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

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

ecore_xcb: handle realloc failure.

Summary:
If realloc fails, then lst would be NULL, so assigning lst[i] only if realloc is successful, else assigning lst to previous memory location.

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

Reviewed By: cedric

Subscribers: cedric

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

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

eio_monitor: fix memory leak in eio monitor module

Summary:
Free the allocated memory in eio monitor win32 module.

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

Subscribers: cedric

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

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

eio_monitor: fix memory leak issue

Summary:
@fix

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

Reviewed By: cedric

Subscribers: cedric

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

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

ecore_fd: add _ecore_main_fdh_poll_del() when fd handler is deleted.

Summary:
When fd handler is deleted by ECORE_CALLBACK_CANCEL, _ecore_main_fdh_poll_del() is not called.
So fd still exists in epoll's event pool.

Reviewers: raster, seoz, woohyun, Hermet, cedric

Reviewed By: cedric

Subscribers: cedric

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

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

ecore-evas x - solve internal window size flickrs in e

so there is an issue that e brings out where configure events get
queued and deferred AND e ends up requesting a new size, but new size
is wrong as its read from an old event (requested size is updated) and
in the end ecore-evas doesnt request the actual new size because
current w/h is "the same" even though it isn't... bah - it's complex
and a self-feeding event issue. just doing the move/resize solves it.

@fix

ecore-evas-wayland: Set ee->draw_ok appropriately based on visibility

Summary: A previous code change to the common code which processes
updates is relying on having the ecore_evas->draw_ok flag set in order
to process updates. This change makes sure that ee->draw_ok is in sync
with canvas visibility.

@fix

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

evas-wayland-shm: Fix silly typo

Summary: OOpppss, this should be eina_rectangle_free

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

ecore-evas-wayland-shm: Check for fullscreen and override windows when
processing updates

Summary: When processing render updates, we should be checking if the
Ecore_Evas "should be visible" property is set.

@fix

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

evas-wayland-shm: Don't leak rectangles during flush

Summary: As we loop the regions of the output buffer and accumulate
changes to post, we should be freeing the rectangles when we are done
with them.

@fix

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

evas-wayland-shm: Fix to add support for MERGE_MODE Smart

Summary: Seems we were never setting any merge_mode for the
wayland-shm engine. This fix implements setting merge_mode

@fix

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

Evas filters: Fix crash with async sw rendering

If the filtered object (text or image object) was deleted, its
output image (cached inside the filter data) would be freed
immediately. This could cause crashes in case of async rendering.

@fix

eet - be robust about garbage at the end that looks like an idenity

if there is an identity signaure at the end, ONLY check it if it looks
like a real one (correct magic number, cert and sig size fields are
sane etc.). this means eet opens dont fail for files that may have
trailing garbage or padding that is not an eet identity signature.

ethumb - fix client free issue with pending dbus messages

this fixes T2769

@fix

eet - fix signature alignment and position

Evas textblock: Force relayout during proxy render

If the textblock object was not visible in the main canvas, but
still needs to be rendered in a proxy surface, then _relayout may
not have been called. This forces generation of paragraphs based on
the current geometry.

This patch is ugly. I know. This is evas render :)

Evas render: Fix another bad proxy clipping issue

This is an ugly hack to fix an issue reported in D3114. I don't
understand how the proposed patch could even fix anything given
the current situation.

Test case:
 - Create edje object with textblock inside
 - Clip out edje object (--> all children become not visible)
 - Take textblock from edje and set it as source of a proxy
 - Mark proxy as source_clip

Result: Nothing visible.

Expected: Proxy should contain the textblock object, since
  source_clip means we ignore the edje object's clipper, and
  only care about the textblock's clipper (entire canvas).

Here's what was happening:
 - During a first pass, textblock is not visible, cur->cache.clip
   is calculated, marked as clip_use=1 with geom 0,0 0x0
 - In a second pass, the proxy is rendered, which needs to draw
   the textblock in a surface. But cache.clip was used and it was
   wrong.

Solution:
 - Ignore cache.clip when rendering inside a proxy. I'm pretty
   sure there are other instances where cache.clip will still
   be a problem.

Problem: textblock never called relayout since it was not
  visible.

Conclusion: cache.clip needs to die. It's a legacy optimization
  that now causes more issues than it fixes.

EDC Doc: Some fixes

We need to move this doc to the wiki and complete it :)

ecore_x: Remove XPrint usage

Summary:
Xprint has been deprecated since 2008.

It's recently (August 2015) been removed from debian.

Reviewers: zmike, devilhorns

Subscribers: cedric

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

ecore-evas-gl-drm: Disable async rendering for EGL

Summary: Disable async rendering for EGL engine as EGL is sync only.
This fixes gl_drm engine to work (in my tests) using ecore_evas
example apps.

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

eina_quaternion: add since information for new APIs

eina_matrix: adjust since version for new APIs in 1.16

po: update po files

Evas image: Fix wrong usage of engine data macro

ENDT macro should be passed to engine functions instead of
some other function pointer!
@fix

evas/gl_x11: Fix indentation and invalid error check in native set/bind/unbind

evas sw x11 - xcb code has same changes as xlib with optimizations

evas engines - fix partial swap to not skip swaps if full mode used

@fix

evas sw generic - printf debug remove

eldbus - only do syntax checks if libdbus is new enough

ecore-wayland: Don't use custom animator when in server mode

Signed-off-by: Chris Michael <cp.michael@samsung.com>
Reviewed-by: @zmike
necessary to avoid breaking the compositor canvas animator