ecore-wayland: only do deferred shell surface creation when a surface exists

ecore-wayland: defer shell surface creation for windows if shell is not bound

in the case where a surface is created before the compositor binds its shell(s),
a shell surface would never be created

fixes case where internal windows would not create frames in enlightenment

@fix

evas textblock: fix memory leak

Summary:
Fix memory leak

Delimiter string is being saved using
eina_stringshare_replace without any del or free
when object is deleted.

@fix

Test Plan: N/A

Reviewers: tasn, herdsman

Subscribers: cedric

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

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

ecore_x: removing useless assignment.

Summary:
The assigned value has no effect outside the function. So removing it.

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

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

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

ecore_x: handling realloc failure.

Summary:
If realloc fails, lst would be NULL. So handling it.
Signed-off-by: Srivardhan Hebbar <sri.hebbar@samsung.com>
Reviewers: cedric

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

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

ector: removing useless assignement.

Summary:
Null assignment has no effect in the caller function. So removed it.

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

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

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

edje: comparision with non-scaled min values issue fix.

Summary:
In a different scaling environment, wrong comparison of min values causes resize issue as original size is compared instead of scaled size.

Signed-off-by: Shilpa Singh <shilpa.singh@samsung.com>
Signed-off-by: Subodh Kumar <s7158.kumar@samsung.com>
@fix

Test Plan:
Create a layout with some min size and swallow a resizable layout inside the layout.
the parent layout will not expand even when the height has crossed its min size.

Reviewers: cedric, tasn, raster

Subscribers: subodh6129

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

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

edje: fix memory leak on realloc.

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

Reviewers: cedric

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

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

evas table: fix miscalcuation in cells with span and padding

Summary:
In evas table that homogeneous mode is turned off,
the size of items in cells, whose rowspan or colspan is larger than 1
and horizontal or vertical padding exists, are miscalculatd.

T2655

@fix

Test Plan: elementary_test "Table Padding"

Reviewers: Hermet, cedric

Subscribers: cedric, DaveMDS, Hermet

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

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

evas: fix memory leak

Summary:
Fix of memory leak in evas_3d_utils.c

@fix

Reviewers: raster, Hermet, cedric

Reviewed By: cedric

Subscribers: artem.popov

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

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

Evas GL shaders: Simplify a bit the shaders code

Following raster's change, the Program_Source struct is useless.
This is a big commit, again, but really all is auto-generated.

elua: remove non-portable lconv fields (breaks windows)

Evas GL: Try to fix automated tests

I forgot the #ifdef in the shaders. This might be it. @stefan,
try again please.

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

evas gl common - remove unused struct fields from shader src to save mem

we use a bit more memory than we need by having unused fields for
shaders. the old binary data ptrs and size fields we just have not
used for years now as we dont compile in shader binaries anymore as no
living drivers need this anymore. so this removes that with no actual
side-effects.

evas gl - fix big endian fully by adding a host of new img shaders

no shortcuts. we used the same img sharder for pixles as for native
surf. so need new shaders to do the swiszzling. add them all,
generation scripts metadata and enums etc.

@fix (along with previous 3 commits)

evas bigendian fix - only fix img shaders as only these src pixels

evas gl fix color on bigendian with gl engine

this fixes rendering on ppc (bigendian) where we have thnigs swizzled
oddly. not bgra -> argb but rgba -> grab ...

so generate a bigendian shader file and use if on bigendian.

this should fix T2721

it fixes it in the visual screenshots i can get remotely.

eo - another 1.5 percent speedup in eo_bench eo_do by removing err handl

so we do a bit of error handling like does a stack fail to allocate,
does setting the tls var fail, have the stack frames been nulled or
not allocated, etc. - these acutally cost every call because they mean
some extra compare and branches, but ore because they cause a lot fo
extra code to be generated, thus polluting instruction cache with code
and cacheline fetches of code that we rarely take - if ever.

every if () and DBG, ERR etc. does cost something. in really hotpath
code like this, i think it's best we realize that these checks will
basically never be triggered, because if a stack fails to grow... we
likely alreayd blew our REAL stack for the C/C++ side and that can't
allocate anymore and has already just crashed (no magic message there -
just segv). so in this case i think this checking is pointless and
just costs us rather than gets us anything.

evas - clean if check to be a single return point for clarity

if (a) return;
if (b) return;

->

if ((a) || (b)) return;

ecore-evas-wayland: add frame callback listener during render

when a render occurs, frame callbacks must be managed in order to ensure
successful rendering for future frames. the best place to do this is in the
engine here, since this is the lowest-level place which has access to both
the wl_surface as well as the evas rendering state

ref T2784

Revert "ecore-wayland: Redo window animators to not use Custom source animators"

This reverts commit bd83d4c03ab9f6f6ae225976b9595a1dfde61237.

adding an animator (and then not managing its ticks) causes the animator
to fire constantly. in this case, it was causing 100% cpu usage and forcing a
compositor re-render for every frame regardless of damages

Eo do: Reuse stack fetching across eo functions.

This causes a significant speed up (around 10% here) and is definitely
worth it. The way it's done lets the compiler cache the value across
different eo_do calls, and across the parts of eo_do. Start and end.

This breaks ABI.

Eo do: optimise getting the thread call stack for the main loop thread.

This may look like an insignificant change, but it doubles the speed of
this function, and since this function is called so often, it actually
improves my benchmarks by around 8%.

Eo: Move mainloop checks inside Eo.

This breaks ABI in a harmless way, and it will give us the ability to
drastically improve Eo in the future without breaking ABI again, thus
allowing us to declare Eo stable for this release if we choose to.

Eo events: Add a struct member marking if it's a legacy event or not.

My previous patch to this piece of code
(37f84b7e966372384e2dfe5d191a6f907a17962e), caused a significant
performance regression. This is such a hot path, that even accessing the
strings when we don't have to slows things down drastically. It makes
more sense to just store it in the structure.

This commit breaks ABI (though most people probably won't even need to
recompile anything else because of the memory layout).
It was discussed on IRC and was decided this is a big enough issue to
warrant a fix during the freeze.

@fix

Evas GL: SW engine CID fixes

Following previous changes, there was unreachable code.
- CID 1327421
- CID 1327420

Evas: Fix image orientation with GL engine

Fixes T2738

Not sure what to do if yinvert is true. How to handle image
orientation then? Flip vertically on top of the current orientation?

evas - add another layer->evas check in addition to layer

fixes T7215

evas - was missing evlogs for the threaded rendering handling - fix

edje: load - use evas_object_clipees_has instead of evas_object_clipees_get

evas_object_clipees_has is far cheaper than evas_object_clipees_get in case of checking if
clipees exist or not. This should improve the performance in case of large set of clipees.

@fix

evas: render - use evas_object_clipees_has instead of evas_object_clipees_get

evas_object_clipees_has is far cheaper than evas_object_clipees_get in case of checking if
clipees exist or not. This should improve the performance in case of large set of clipees.

@fix

Tests: use eina_environment_tmp_get()

Evas GL: Add more tests

- glReadPixels
- FBO test
- PBuffer test (only GL engine)
- surface formats test (up to depth=24, stencil=8)

Evas GL: Add test case for surfaceless contexts

Evas GL: Add EGL_KHR_surfaceless_context since it is supported

See 3e39bf36389fdbc8a3f845214.
Surfaceless contexts are supported even if the driver does not
support them.

Evas GL: Error out on surfaceless make_current with OSMesa

While OSMesa may support surfaceless contexts, we don't support
them yet in the SW engine. Instead of switching to NULL, NULL,
let's error out and do nothing instead.

Evas GL: Don't call eglMakeCurrent if already null

If the current context & surface are already null, avoid
calling eglMakeCurrent again, since it can return an error
(EGL_FALSE but with no error code, thanks Nvidia).

ecore_imf/wayland: Add NULL check for text_input_manager.

Summary: this patch fixes crash caused by accessing the text_input_manager in wayland_im_context_add().

Test Plan: N/A

Reviewers: jihoon

Reviewed By: jihoon

Subscribers: cedric

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

evas: no need to compute clippees list if we just want to know if there is any.

This is a partial fix for a terrible performance scenario where application take
forever to close with a large number of edje object.

@fix

Eo: Fix confusing indentation and style.

ecore-evas-wayland: Don't use custom animators for frame callbacks

Summary: As we now have per-window timer-based animators in
ecore_wayland for frame callbacks, we no longer should be using Custom
animators to handle surface frame callbacks

@fix

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

ecore-wayland: Redo window animators to not use Custom source animators

Summary: This moves window animators (for frame callbacks) to not use
a custom animator source but rather use a timer-based source. This
also moves animators to be per-window based (in that an animator is
created per-window).

@fix

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

ecore: fix usage eina_condition_timedwait to use relative time instead of absolute time.

This could lead to some very long and unexpected pause as the timeout passed
to eina_condition_timedwait was passed as a absolute time instead of relative.
Hopefully we don't build rocket.

evas filter: silent coverity on self assignment

This is a false alarm. Just to make coverity happy, use eo_do
instead of eo_do_ret

CID: 1316017

edje_edit: Fix clang warning of unused variables

Summary: clang reports that these static variables are not being used
anywhere (and grep confirms this), so remove them.

@fix

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

ecore-evas-wayland-egl: Fix clang warning about extra parentheses

@fix

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

ecore-drm: Add missing field initializers for vt_mode

Summary: clang reports missing field initializers for usage of vt_mode
structure, so add the missing initializers

@fix

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

evas: Fix clang warning for missing field 'async' in default_state

Evas_Object_Filter_Data has a bool for 'async' at the end of the
struct. This field was missing from the initialization of
'default_state'

@fix

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

Evas GL: Fix support for the SW engines (OSMesa)

Since @raster changed the behaviour of the dirty flag on images,
damages must be added to redraw the GL surface. Evas_Image checks
if it is an Evas GL surface by looking at its native surface.
But in case of SW engine, there was no native surface information
for Evas GL surfaces. Also, the OPENGL surface type was awfully
abused for OSMesa support. Luckily EVASGL surface type lets us
pass arbitrary pointers :)

Evas GL: Improve ERR log (direct rendering failed)

Evas GL: Add a test case for Evas GL (make check)

For now this only covers SOME of Evas GL's functions.

It will try to run with opengl_x11 and buffer (OSMesa). It'll also
try to fail silently if the engine initialization failed, or if
OSMesa could not be found. If the engines work, then Evas GL must
work properly.

Evas GL: Implement glGetString wrapper for OSMesa

This is only one step into making the software engine actually
work the same as a proper GL engine from Evas GL APIs point of view.

This is necessary for the test suite (coming next).

Evas GL: Minor changes inside glGetString

Evas GL: Avoid make_current(0,0) during destroy

Show an error message and call make_current(NULL, NULL) only if
the object (context, surface) being destroyed is current.

Otherwise, avoid changing the current context & surface.

gl_x11: val can be everything expected 0 to mean true

For me on a intel driver val is -1 and it needs to be inverted.
So we need to checkout that val is not 0 and not equals to 1.

Thx to raster for helping debugging this thing :).

Eina: fix memory leak in eina_file_open()

@fix

Eina: fix eina_file_current_directory_get()

the length was not correctly computed and eina_file_path_sanitize() was
was writing beyond the limit of the string

@fix

Eina: fix spelling in eina_tmpstr documentation

Evil: set EAPI correctly in pwd.h

evas gl engine - typo remove from previous commit

some typo crept in that... compiled. a stya / on a line. how it
compiled, i don't know.

evas - obj destructrion - handle when obj layer is partly destroyed

this should fix T2715 and anything similar as during destruction
object internals are a bit weird.

evas - fix evlog to push and pop properly

eina: util - set tmp to NULL to fix compilation warning

lib/eina/eina_util.c: In function 'eina_environment_tmp_get':
lib/eina/eina_util.c:96:7: warning: 'tmp' may be used uninitialized in this function [-Wmaybe-uninitialized]
    if (!tmp) tmp = "/tmp";
       ^

Evas: Some coverity fixes

This are false alarms. But the explicit null check doesn't hurt.
Also, Coverity doesn't like eo_do_ret. Neither do I. :)

- CID 1327246
- CID 1327245

Evas render: Fix proxy source_clip with source images

In case the source is an Evas_Image, we skip proxy_subrender,
and so the clip was not set against the source clip.

Evas render: Fix proxy source_clip logic inversion

As spotted by @FurryMyad I inverted the logic for source_clip.
This should restore the proper behaviour while keeping my previous
fixes working. See D2940.

ecore_cocoa: fix a bug where event timestamp was not retrieved correctly.

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

evas: fix build of gl_cocoa engine.

__context_restore and __need_context_restore are undefined when
building gl_cocoa engine.

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

ecore_cocoa: use a timer instead of a poller to be more responsive when dealing with cocoa Event.

As all Mouse and Keyboard events comme from Cocoa, the poll period must
be reduced. Backwards of this method is that when no signal are
received for a long period, the timer fired anyway and consumes CPU
for nothig but it seems there is no easy method to integrate NSApplication
mainloop into an existing one.

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

ecore_evas_cocoa: use the same render function as ecore_evas_x

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

ecore_cocoa: fix warnings, add Debug outputs where needed and remove printfs

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

eina: Include <unistd.h> for getuid and geteuid functions

Summary: If we do not include unistd.h, we end up with implicit
declaration warnings when compiling

@fix

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

ecore-wl: Add missing EINA_UNUSED for unused function paramaters

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

evas: fix performance regression by reducing the unecessary memcpy we are doing.

Actually copying max is pretty useless and super slow. We usually have something
like 1024 slot in a context, but a very small amount of them are acutally active.
It would be better to actually do some kind of copy on write technique here, but
as Eina_Cow doesn't handle array and we are close to a release, let's be
conservative.

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>