ecore_evas_x: make gl_sync_draw_done an explicit signed char type

Summary:
char type is interpreted as unsigned char in some implementations,
so setting -1 will be read as 255.
So we should explicitly make this a signed char type.
@fix

Test Plan: Local tests

Reviewers: cedric, jpeg

Reviewed By: jpeg

Subscribers: wonsik, cedric

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

ecore-evas-x: resize window when it is rotated by window manager.

Summary:
window manager can send arguments and its meaning as follows.

1) resize:0
it means client window doesn't need to resize its window by rotation.
this case is a ELM_WIN_BASIC window in mobile profile.

2) resize:1, ee->w != w, ee->h != h (deprecated)
it means client window should be resized by rotation, and wm already resize its window.
so, client don't need to resize its window.
it's just for backward compatibility.

3) resize:1, ee->w == w, ee->h == h (addition)
it means client window should be resized by rotation, and wm don't resize it.
so, client should resize its window.

Test Plan: N/A

Reviewers: gwanglim, raster, jypark, devilhorns

Subscribers: cedric

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

NEWS: Small manual tweaks

Most notably removed a merge commit and a commit stating to take EO out of
beta which is not happening for 1.15.

ecore_evas_input: add value to enable mouse button cancel feature

mike reported this feature break the behavior in E19, but I have trouble to reproduce it.
so I first add value to enable this feature and after that,

eina str: remove useless and wrong ifdef for FreeBSD

Evas filters: Fix make doc, check when EFL is not installed

Thanks @aerodynamik for even giving the proper environment
variable to set :)

Fixes T2614

evas: fix size of borders in case need generate texture without atlases

Summary:
Set size of texture unit without 2 pixels for borders in case use it without
atlses. Just one case if texture for 3D use repeat mode and non-normalized
tuxture coordinates

Reviewers: Hermet, cedric

Subscribers: cedric

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

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

ecore-evas-wayland-shm: Remove duplicated code on configure event

Summary: As we end up calling ecore_evas_wl_common_resize anyway,
there is no need for all these size calculations in the configure
callback. The resize function will deal with this anyway.

@fix

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

efl: Bump required wayland library versions

Summary: As a lot of functionality has changed in the relevant efl/e
wayland codebase, we should bump the required wayland library versions
as we are now dependant on updated wayland libraries.

@fix

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

evas-wayland-shm: Fix issue of surface_swap spamming invalid warning

Summary: Since the surface_swap functionality is changed now, we no
longer need to bother checking if the leaf is busy as we don't use the
leaf itself inside this function anymore.

@fix

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

wayland-shm: Remove "mapped"

Summary:
Looks like the recent changes to swap/redraw have fixed a problem with
damage tracking and we no longer need this.

Damage history seems ok without it now.

@fix

Reviewers: zmike, devilhorns

Subscribers: cedric

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

wayland-shm: re-organize some of the swap/redraw logic

Summary:
The way things were, we'd be committing a buffer we hadn't rendered to
yet.

Now redraw() contains all the attach/damage/commit logic, and swap() is
called afterwards to set up a new target buffer for the next render.

@fix

Reviewers: zmike, devilhorns

Subscribers: cedric

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

wayland_shm: Don't store frame_cb

Summary:
We don't need to - all the frame callback does for us is ensure a queue
push.

We should destroy any callback we receive when we receive it.

@fix

Reviewers: zmike, devilhorns

Subscribers: cedric

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

Ecore: Fix invalid lock count if DEBUG is enabled

With -DDEBUG, we can see an error message, like:
ERR: You are calling _ecore_lock() from outside of the main loop
     threads in lib/ecore/ecore_private at line 306

Looking at the code shows that ecore_lock fails immediately if
thread debugging is enabled, but ecore_unlock does not, so the
value _ecore_main_lock_count could go below 0.

This is not very important as the value is never used.

Evas filters: Fix order of header file includes

This fixes filters on Ubuntu 32 bits.

This was one hell of a weird bug to track down. Everything worked
like a charm on my 64 bit machines, but filters would simply fail
for no good reason: a safety check sees a NULL pointer when clearly
it was properly allocated.

Just after entering a function, the content of an RGBA_Image would
change, even though there was no memory write there. This made the
image data pointer NULL, and filters would fail miserably.

So I printed out the contents of the RGBA_Image, they changed. But
the memory itself had not changed. The size of the struct itself
had changed when jumping from one file to another! But its definition
had not! Non-sense!

Unless of course a system header file was included before config.h
and ino_t or off_t would switch between 32 and 64 bits...

@fix

Evas filters: Fix COW usage in filter mixin

Evas filters: Disable debug code path

Evas filters: Fix invalid rendering in case of async render

This simplifies the code and fixes async buffer management.

@fix

examples: Update .gitignore for binaries

ecore_con_eet: fix dereferencing of object during callback that crashed some tests case.

ecore_con: fix test to actually send a NULL terminated string.

evas canvas3d: bounding box does not change the position according object

Summary:
This fix need for synchronous rotation mesh and aabb of  "sonic" in example "evas-3d-aabb.c".
@fix

Reviewers: Hermet, cedric

Reviewed By: cedric

Subscribers: sri.hebbar, cedric

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

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

ecore_ipc: remove redundant code

Summary:
buf is always NULL (already freed and set to NULL).
We don't need to add NULL checking and free it.

Reviewers: raster, cedric, Hermet

Subscribers: seoz, cedric

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

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

eina_quaternion: add forgotten implementation of converting eina_matrix3 to eina_quaternion

Summary:
Implemenation of eina_matrix3_quaternion_get function

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

Subscribers: cedric

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

This was a function I forgot to finish implement. Thanks Vivek to take care of it.
As it comes with a test case, is self contained and fix a missing bit of code I will
push it at this point in time of our release process. Sorry everyone for that late push.

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

eina_quaternion: add test cases for various eina quaternion operations

Summary:
Added test cases for scale, multiply, normalize,dot operations for quaternion
Signed-off-by: Vivek Ellur <vivek.ellur@samsung.com>
Reviewers: cedric

Reviewed By: cedric

Subscribers: cedric

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

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

eina_quaternion: add test cases for eina f16p16 quaternion functions

Summary:
Added test cases for various operations on eina f16p16 quaternions

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

Subscribers: cedric

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

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

edje_cc: fix dereference before null check

Summary:
Fix Coverity issue which reports a dereference before null check.
The enc_info can not has NULL point
because _edje_multisense_encode api calls exit() in case of failure

Reviewers: Hermet, Jaehyun, cedric

Reviewed By: cedric

Subscribers: cedric

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

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

ecore-evas-x: fix typo

Test Plan: N/A

Reviewers: gwanglim, raster, jypark, cedric

Reviewed By: cedric

Subscribers: cedric

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

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

evas-wayland-shm: Don't call surface commit until we've setup the frame listener

Summary: This fixes an issue where we may been missing a frame
callback due to the wl_surface_commit being called before the frame
callback listener was setup. Now we will setup the frame callback
listener before doing any wl_surface_commit so that we don't miss
frames.

@fix

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

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

Evas masking: Fix crash in async rendering

There was a FIXME comment... >_<

Improper management of image resources in async render led to
a rare crash. This should fix that.

@fix

evas - image data unload - try plug another possible crash path

i am not sure if this is the odd crash i am seeing, but in theory it
could be. as these crashes are rare it's hard to find and gdb is "too
late" other than telling me the image is freed already by the time we
do an unload.

evas - render2 - ensure region files are in dist

evas/filter: Avoid leak in error path

If format is not found we go out of scope and would leak param. Free instead.

CID 1308610

evas - image unload - fix unload to only unload imgs needing it

need_unload seems to have been done wrong. using preload not
need_unload. no idea why it checked/cleared preload instead of
need_unload.

evas render2 work - no new feature - paralell bit of code

*this does not affect any "stable" code paths - it's render2 and some
added region code that i MAY have to use to fix some bugs/issues as i
am noticing tilebuf being pretty silly and just doing bounding boxes.
region code lifted from xserver (mit-x11 license thus compatible with
evas bsd) and it's been stripped down and cleaned up with some tilebuf
optimizations like remembering the last rect add/del to avoid work
when repetedly adding/delling the same rects (very common).

evas render - fix up debug macros so they don't generate any code

if render debug is not enabled make sure they end up totally empty to
avoid any possible overhead from the optimizer not stripping them out

evas grid: Eolian doc conversion of grid.

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

Reviewers: cedric, tasn, q66

Reviewed By: q66

Subscribers: cedric

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

ecore-drm: Ignore key events that are not seat wide state changes

Summary: This patch adds some checks to make sure that we can ignore
key events which are not seat wide state changes.

@fix

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

Ecore_Win32: do not allocate data as it is useless. Fix memory leak.

@fix

examples/emotion: Remove unused callback functions from generic example

examples/ecore_con: Mark some unused parameters as such

examples/evas/shooter: Move image path into file that uses them.

The evas-3d-shooter-header.h is used by two binaries. One is using these image
path and the other not. Which means that we get a bunch of warnings thrown at
us during the compile of the binary which does not use them as unused variables.
Solve this by having them only in the place where they are actually used.

examples/evas/shooter: Add double include guards to header file.

examples/evas/shooter: Remove more unused variables.

examples/evas/shooter: Remove unused variable

The column.ply does not even exist. Might have been removed and this part
forgotten.

Ecore_Evas GDI engine: free module data

Evas GDI engine: save the original bitmap into the memory DC

This allows the memory DC to be killed and fix a memory leak

@fix

Evas filters: Set state.next to nil when not in transition

This is what the doc says and makes more sense.
That was some kind of inverted logic.

eina: Add since 1.15 tag for environment*get API additions

 eina: Add since 1.15 tag for quaternion API additions

eina_evlog: Remove trailing whitespaces while looking at this file.

eina: Add since 1.15 tag for evlog API additions

eina: Add since 1.15 tag for crc API additions

edje: Add since 1.15 tag for language_set API additions

ecore_x: Add since 1.15 tag for keygrab API additions

ecore_x: Add since 1.15 tag for keyrouter API additions

evas vg: Eolian doc conversion of vg.

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

Reviewers: cedric, tasn, q66

Reviewed By: q66

Subscribers: cedric

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

eina tests, ecore_con tests: fix compiler warnings

eo tests, evas filter tests: fix compiler warnings

eina, ecore_imf: fix uninitialized field warnings

evas out: Eolian doc conversion of out.

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

Reviewers: cedric, tasn, q66

Reviewed By: q66

Subscribers: cedric

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

Evas: Fix wayland-only compilation without Xlib headers

If the Xlib headers are not installed, a special macro must
be defined before including EGL headers.

Evas/psd: Remove commented-out debug code

Removing because it matches a grep of <<<< used for merge conflicts :)

Evas GL: Add missing call in case of direct rendering

I spotted this by looking at the code in a different branch (that
was tested on real hardware).

Revert "Eolian: fix eolian helper for windows."

This reverts commit 60b473d949c2582695a0ed7e67dc2e2c32eeee1b.

It breaks our mingw cross build setup we are using on jenkins. In summary it
silently fails to use eolian_gen.exe while it should use the linux native
eolian_gen and thus does not generate the c and h files from the eo ones.

See the ml for details or look at the error here:
https://build.enlightenment.org/job/changely_efl_mingw_x86_64/2115/console

evas polygon: Eolian doc conversion of polygon.

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

Reviewers: cedric, tasn, q66

Reviewed By: q66

Subscribers: cedric

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

evas object: Eolion doc conversion of object_smart.

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

Reviewers: cedric, tasn, q66

Reviewed By: q66

Subscribers: cedric

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

Use eina_environment_tmp_get()

Ecore: lower log level

On Windows, this insanely fills a temporary file, reaching
gigabytes

Eina: use Eina API in documentation

Efreet: use eina_file_mkstemp() instead of mkstemp in efreetd.c

ecore-x: send last event time along with generic Ecore_X_Event_Client_Message

this won't necessarily be the same time that the client sent the message,
but at least it will provide a better reference point than 0

evas_canvas3d_scene: fix missing color initializer

evas_object_vg: fix missing initializer warning

evas: fix missing render2_walk initializers

This caused annoying compiler warnings that are gone now.

@fix

evas textgrid: Eolian doc conversion of Textgrid.Pallelt.

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

Reviewers: cedric, tasn, q66

Reviewed By: q66

Subscribers: cedric

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

evas box: Eolian doc conversion of box.

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

Reviewers: cedric, tasn, q66

Reviewed By: tasn, q66

Subscribers: cedric

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

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

evas canvas3d: Eolian doc conversion of canvas3d_primitive.

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

Reviewers: cedric, tasn, q66

Reviewed By: q66

Subscribers: cedric

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

evas canvas3d: Eolian doc conversion of canvas3d_node.

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

Reviewers: cedric, tasn, q66

Reviewed By: q66

Subscribers: cedric

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

evas canvas: Eolian doc conversion of efl_vg_base.

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

Reviewers: cedric, tasn, q66

Reviewed By: q66

Subscribers: cedric

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

evas canvas3d: Eolian doc conversion of canvas3d_object.

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

Reviewers: cedric, tasn, q66

Reviewed By: q66

Subscribers: cedric

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

evas canvas3d: Eolian doc conversion of canvas3d_camera.

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

Reviewers: cedric, tasn, q66

Reviewed By: q66

Subscribers: cedric

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

Evas GL: Some documentation on sync objects

Evas GL: Deprecate API evasglCreateImage

Despite its ugly name, evasglCreateImageForContext should be used
instead, as it's a proper API with evasgl and ctx passed in as
arguments.

Evas GL: Fix sync, wlbuffer and image egl ext functions

Before this patch, those EGL/EvasGL functions can not work
without a current context. But EGL does not require any
current context for those to work, or at least, this should
be left to the driver to decide.

Evas GL was only able to get a pointer to the display
if a context was current.

The display pointer should be infered from Evas_GL unless
we can find a current display. EGL does not require a
context to be current in most of these function calls.

This should bring evasgl a little bit closer to EGL in terms
of behaviour (those are EGL-only extensions, btw).

Thanks @spacegrapher for the quick review

@fix

evas_image : fix unloads cleanup logic

Summary:
Now Evas gl preload feature is disabled.
But if it is turned on, memory crash occurs.
Because evas_gl_common_texture_upload is not excuted immediately.

Test Plan: EVAS_GL_PRELOAD=1 ELM_ENGINE=gl elementary_test -to "photocam"

Reviewers: raster, cedric, woohyun, seoz, Hermet, singh.amitesh, jpeg

Subscribers: jpeg, cedric

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

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

Ectore: And more build fixes for @mythri! :)

Evas filters: Fix color value for invisible

Since the filters default to alpha = 255 the actual value of
invisible / transparent was black.

Evas GL: Fix glClear(0,0,0,0) with DR and COPY

When glClear is called in direct rendering move (DR), we usually
have to skip the call altogether because clearing out transparency
would erase the pixels in the evas backbuffer. This means Evas
would not be able to blend an RGBA GLView on top of other objects.

But COPY mode should allow Evas GL to poke holes in a window
backbuffer.

Thanks @spacegrapher for the review :)

NOTE: Elm GLView also needs to pass the render op to its Evas.Image.

@fix

Evas GL: Fix internal function pointer

evas_gl_native_context_get is an internal function
passed around from an evas engine to evas_gl so that we can
implement evasglCreateImageForContext without exposing
any evas engine internal structure to evas_gl.

It's all a ittle bit ugly but the previous solution with
dlsym(DEFAULT) didn't work.

eina_module: Raise dlopen() error messages to WRN when file exists

Failing to load a module that does not exist is indeed not an error,
but failing to load a module that exists on disk happened probably
because of an error like "symbol not found".

Considering eina_module is most likely used by EFL itself, I believe
an internal linking failure is a warning worth reporting.

Evas GL: Fix evasglImageDestroy() from non-evasgl threads

If the TLS variable was not initialized, Evas GL can't get a pointer
to a valid EGLDisplay which is required by eglImageDestroy.

So, we keep track of the dpy used at creation time and use that
if we're in another thread.

Ector: Another build fix for @mythri

evas canvas3d: Eolian doc conversion of canvas3d_light.

Summary:
Apart from evas_canvas3d_node_look_at_set() all other things are referenced.
Tried to reference it to @Evas.Canvas3D.Node.look_at_set(). But getting error.

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

Reviewed By: q66

Subscribers: cedric

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

Ector: Fix potential build errors with double typedef

Depending on the compiler and its version, having twice a
typedef on the same name may lead to a build failure.

Thanks @mythri for the report.

image_savers/jpeg: actually properly fixed the jump

Ecore evas: Correct function name in magic failure.

Why this is done like this, and not just with __func__ in the macro is
beyond me.

Thanks to Vincent Torri for pointing this out.

@fix

Eolian: fix eolian helper for windows.

Based on an Elementary patch by Vincent Torri.

@fix