ecore_wl2: Take a library reference on tbmsurface create

We need to call ecore_wl2_init in tbmbuf_surface_create because of this commit
0d27985677897363f148c92042ef3d3917796981

reproduction:
1. Call elm_init(0, NULL);
2. Call elm_shutdown();
3. Check that there is no CRI error message
[CRI<14619>:eina_log lib/ecore_wl2/ecore_wl2.c:238 ecore_wl2_shutdown() unknown log domain -1, original message was: ERR: 'Ecore_Wl2 shutdown called without Ecore_Wl2 Init']

or

1. In efl-test-suite, execute utc_elm_bg_add
2. Check that there is no CRI error message

Change-Id: If8ec5795109e7a87bb893bd1633ec454873b3de7

ecore_wl2: Remove ecore_wl2_input_grab/ungrab

Summary:
These should never be called, so they shouldn't be API.

This kind of internal state fiddling should happen implicitly in the
API that need it.

@beta_break
Depends on D6126

Reviewers: zmike, cedric

Reviewed By: zmike

Tags: #efl

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

Change-Id: Ifd4de43a241cfb27bf85bce8bbb0a7199565ec4b

ee_wayland: clean up warnings

Summary:
Pretty sure these are my fault.
Depends on D6122

Reviewers: zmike, cedric

Reviewed By: zmike

Tags: #efl

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

Change-Id: Iaeff36cfc00f72bc0e7a169e8c77726551a635ab

ee_wayland: remove edges flag

Summary:
Nothing needs this anymore.
Depends on D6121

Reviewers: zmike, cedric

Reviewed By: zmike

Tags: #efl

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

Change-Id: I2db879c52a27a6ecd9ad1924c47ed91232447c14

ee_wayland_egl: Remove some amazing weirdness

Summary:
I don't even know what to put here, but I'll try.

wl_egl_window_resize()'s final two parameters indicate new attachment
points for a buffer relative to the previous top left corner.  When the
compositor is resizing a window it already handles the corner placement.
Fortunately, compositors seem to ignore the new attach co-ords during
resize, so this code hasn't broken anything.  It's just a complicated
NOP.

The new attachment points are intended for use in spontaneous resize,
not drag resize, but the only time these functions are called is for
drag resize.
Depends on D6119

Reviewers: zmike, cedric

Reviewed By: zmike

Tags: #efl

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

Change-Id: I00722662988871502b8ee41d98ec0951b3e72451

ee_wayland: Remove pointless ack_configure

If we don't ack_configure here, it'll just automatically happen in
ecore_wl2_commit() next time we render anyway.

If we do ack_configure here, we can totally screw up E's internal window
handling.

Removing it seems like a win.

Change-Id: I58b1e0a17919a7c5a3561b0db59ae2d9e4fe1384

ecore_wl2: Don't do anything but update alpha in alpha_set

This is now done in ecore_evas where it should be.  alpha_set now does
only what its name claims it does - sets whether a surface has an alpha
channel or not.

Change-Id: I8c6dc82404e417318d1c41e9f3d466e14cb951a7

ecore_evas_wayland: Handle region updates and alpha setting in engine

Instead of having elm handle all this stuff, handle it in ee.  This fixes
a loooong standing bug where we would always use an alpha enabled surface
because CSD needs shadows - even if we aren't drawing CSD.

Change-Id: Ifdbba2fe672f26ef84c54abd07fa5179c15267f0

elm_focus: fix comment range in focus_next_object_set

We can set next parameter to NULL.

reported by utc_elm_object_focus_next_object_get in efl-test-suite

@tizen_fix

Change-Id: I09c56cbb5ea1dac6eeaf572f7beb4d1671e25303

wayland_egl: Fix color depth when not using alpha

Since we don't actually set the color depth at all we can end up with
an RGB565 buffer.  We don't ask for depths because apparently the N900
had a problem with this under X.

I'm not aware of any efforts to bring wayland to the N900, so let's do
this normally.

Change-Id: I3c0651efe5623233f39c7d927cbce67a44fcd854

ee_wayland: Stop using manual_render_set

Now we use draw_block instead, so we don't accidentally stomp on other
code's usage of manual_render_set

fix T6834

Change-Id: I073dc71d56fa56e4ab2390503a66f820a6bcfa8c

efl_ui_focus_graph: remove unused functions

Reviewers: devilhorns, Hermet

Reviewed By: Hermet

Subscribers: cedric, #committers, zmike

Tags: #efl

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

eina_stringshare: cast in order to fix a warning

Summary:
casting here is safe, as before we check and return on size < 1.
Depends on D6379

Reviewers: devilhorns, Hermet

Reviewed By: Hermet

Subscribers: cedric, #committers, zmike

Tags: #efl

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

efl_gfx_path: remove unsued function.

Summary:
the function can be regained by reverting this.
Depends on D6381

Reviewers: devilhorns

Subscribers: cedric, #committers, zmike

Tags: #efl

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

efl_gfx_path: make counters unsigned

Summary:
we are comparing to unsigned number, and the number are moving strongly
from 0 up.
Depends on D6380

Reviewers: devilhorns

Subscribers: Hermet, cedric, #committers, zmike

Tags: #efl

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

evas_vg_node: do not propagate node changed callbacks for invalidated objects

Summary: this callback is triggered on the parent object. invalidated objects have no parent.

Reviewers: bu5hm4n, devilhorns, Hermet

Reviewed By: Hermet

Subscribers: Hermet, cedric, #committers

Tags: #efl

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

elm focus - fix focus composition prepare to not seg by unsafe list mods

efl_ui_focus_composition_prepare() methods in 2 widgets were modifying
the list (removing items) as they walked, using unsafe walking macros.
fix - use the safe variants.

elm win - fix resizing on show. rage was broken last week. this fixes it

rage window with no file passed ended up like 10x40 ... which is
wrong. it wasn't. rage code explicitly resizes the window before show
and this wsas being overriden. tested rage, terminology, em test in
both x and wl. all still work (for me) post commit.

ecore/main: fix build with --enable-g-main-loop

this resolves compiler errors with this option which resulted from refactoring

fix T7011

ecore/glib: add #ifdef guards for newly added includes

these headers are not available on all platforms (e.g., windows) and so
the corresponding #ifdef checks must be used in order to correctly include
them

ref 1adb73cef82c45f2cc8766f43ffb88288e7e8a65
ref T5725
fix T7063

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

evas/cpu: rework SIGILL code for non-x86 (and fix windows build)

Summary:
the previous patch which improved this code for x86 archs broke compiling
for non-x86 and, coincidentally, for windows builds on x86 due to some
unusual #ifdef blocks

this attempts to restore handling on non-x86 and adds additional #ifdefs for
functions which did not build on windows due to removed code

ref 6b1ab3cd9c7e19908690e9e913df7504cc8ae7c8

Reviewers: ManMower, devilhorns

Reviewed By: ManMower

Subscribers: cedric, #committers

Tags: #efl

Maniphest Tasks: T7062

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

tests: Use EFL_START/END_TEST in cxx tests

Summary: Will enable timing info in these tests.

Test Plan: run make check

Reviewers: zmike, felipealmeida, devilhorns

Subscribers: cedric, #committers

Tags: #efl

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

ecore_cocoa: explicity allocate for keyname and key

Summary:
Allocate memory for keyname and key. Also refactor
unnecessary calloc and free.

Reviewers: #committers, zmike, devilhorns

Reviewed By: zmike

Subscribers: cedric

Tags: #efl

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

Edje: recalc edje before fetching the real part

Summary:
There are many calls to `_edje_real_part_recursive_get`. Though, it is
not guaranteed that the Edje object had instantiated all of the real
parts.

This change makes edje to always recalc before retrieving the real part.

The D6364 patch raised a good point, but presented a local fix, where
it seems that a global fix such as this is needed, instead.

The local fix is removed in favor of this. Test suite still passes.

ref T7057

@fix

Test Plan: See T7057

Reviewers: devilhorns

Subscribers: cedric, zmike, #committers

Tags: #efl

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

genlist: Don't search for an item we know we won't find

Summary:
We track list presence already, so we can just do a boolean test instead
of an O(n) lookup.
Depends on D6349

Reviewers: devilhorns

Reviewed By: devilhorns

Subscribers: cedric, #committers, zmike

Tags: #efl

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

edje: fix text set/get issue without edje calculation

Summary:
Some changes broke really basical function behavior of text.
I couldn't get text from an edje object which I just set to the given edje object.
In the past code, edje called recalc function before trying to get text.
So, this patch bring that code to fix this issue.
@fix

Test Plan: Included. Run "make check"

Reviewers: herdsman, raster, cedric, woohyun, devilhorns

Subscribers: #committers, zmike

Tags: #efl

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

efl_ui_focus_manager: track list presence

By keeping track of whether a node is on the dirty list or not we can
save ourself list walks trying to remove items that aren't present.

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

Signed-off-by: Derek Foreman <derekf@osg.samsung.com>

efl_ui_widget: silence warnings with clang.

Clang doesn't like dangling else.

efreet: silent deprecated API build warnings.

Silent warnings on EFL build. Warn on API "mis-use".

Revert "Revert "eina debug - fix segv if an app is compiled with -pg for profiling""

This reverts commit 0d91088d0fe00ab64e1e852551e850d403e9db37.

It also includes a workaround for OSX where RT signals aren't
supported due to kernel age (too old a fork of freebsd). fixes D6310
and T7029

eldbux-cxx: Remove test accessing system bus.

Summary: Tests should use session bus only.

Reviewers: zmike, felipealmeida, devilhorns

Reviewed By: zmike

Subscribers: cedric, #committers

Tags: #efl

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

eina_debug_bt: add ifdef for windows build

Summary: ref D7028

Reviewers: vtorri, devilhorns

Reviewed By: vtorri

Subscribers: cedric, #committers

Tags: #efl

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

ecore_evas: Make draw_block block drawing

This was only used in the X custom render function, now it's meaningful
everywhere.  (Nothing that uses this code path currently sets it)

ref T6834

Change-Id: Icff5151235ea160931fe75e2ed7a75146c50dc81

ee_x: Rename draw_ok to draw_block and invert logic

I want to use this in other engines, but no other engine initializes this
properly, so draw_ok would be EINA_FALSE everywhere.  This way draw_block
is EINA_FALSE after calloc in all engines that don't know about it.

ref T6834

Change-Id: I0c499ebc940b05e436941cb2b7d7bce6d0d35ac5

efl_ui_win: Always show the wayland pointer

It's hidden by the compositor as required, so this just makes life much
more complicated in other code.

ref T6834

Change-Id: Ib40f9fd189edea084acdf99b18436adc5ce2f3e5

ee_wayland: Remove use of draw_ok

This appears vestigial, as the evas render path we use has no custom
draw_ok handling, so nothing ever reads this variable.

ref T6834

Change-Id: I8c8aef946e4cc6a7d097b1ad954777beae274e02

ee_cocoa: Remove draw_ok sets

draw_ok isn't currently used by anything but the X custom render function,
so it's doing absolutely nothing here.

ref T6834

Change-Id: Ibc68e65d147ce2b255683b5aac3342e4e583a7a3

gl_common: Make extension string checks more robust

strstr() can give false positives if the extension name is a subset of
a string in the extension list, for example EGL_EXT_image_dma_buf_import
would match EGL_EXT_image_dma_buf_import_modifiers.

I've opted for a mildly badgered copy of epoxy's test, which should be
robust in the face of subsets.

Change-Id: Ie2b0a4045795c9b11d5b526ce135bfb7ed7ee218

gl_common: Replace strstr() for extension checks with a helper function

The helper incorporates NULL checks, and we love those, so it's better.

Change-Id: I9fb9b17f7f325fe58a563138acb4de14857500c6

elm_win: disable handling of auto_norender_withdrawn for fake wins

these wins are not to be trifled with.

fix T6377

Edje: recalc edje before fetching the real part

Summary:
There are many calls to `_edje_real_part_recursive_get`. Though, it is
not guaranteed that the Edje object had instantiated all of the real
parts.

This change makes edje to always recalc before retrieving the real part.

The D6364 patch raised a good point, but presented a local fix, where
it seems that a global fix such as this is needed, instead.

The local fix is removed in favor of this. Test suite still passes.

ref T7057

@fix

Test Plan: See T7057

Reviewers: devilhorns

Subscribers: cedric, zmike, #committers

Tags: #efl

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

Change-Id: I7defe4441a87a485070082903153d0f34767bcf5

edje: fix text set/get issue without edje calculation

Summary:
Some changes broke really basical function behavior of text.
I couldn't get text from an edje object which I just set to the given edje object.
In the past code, edje called recalc function before trying to get text.
So, this patch bring that code to fix this issue.
@fix

Test Plan: Included. Run "make check"

Reviewers: herdsman, raster, cedric, woohyun, devilhorns

Subscribers: #committers, zmike

Tags: #efl

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

Change-Id: I51af5389755d5f2f6dfb66ddf55e097c2a239e38

Revert "efl_ui_win: add missing wayland interface for setting role of winodw"

This reverts commit a8ebaee4149c91b7d8f5473d5a5f463b1ac39c90.

Change-Id: Iebe9c4078d8e71d88f9649067098cbcafdcde2c1
Signed-off-by: Wonki Kim <wonki_.kim@samsung.com>

eo_debug: Fix segfault when parsing EO_LIFECYCLE_DEBUG envvar

Summary:
The 'if' block should only be executed when the string contains the colon and
something else behind, but sscanf cannot be used in this case.
If the string contained no colon, the following line with strchr(s, ':')
returns NULL and everything explodes.

Test Plan: eo_debug -l now works for me without segfaulting.

Reviewers: bu5hm4n, zmike, devilhorns, q66

Subscribers: cedric, #committers

Tags: #efl

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

efl: Bump required libinput version for Elput

Since commit 7a7abb, LIBINPUT_EVENT_SWITCH_TOGGLE was introduced to
Elput, however this enum did not get released into libinput until 1.7
so we need to require updated libinput now in our configure.ac

@fix

ecore_cocoa_cnp: Check for nil object before adding.

This is fixing a segfault. More work needs to be done in
efl_selection_manager.c. Current status is broken.

ecore_cocoa: fix ECORE_EVENT_KEY_DOWN/UP on MacOS.

Fixing issue with modifier keys on MacOS. These events were missing
window information. Tested with Fyne.io. Window ID and keyname
are valid. See: http://bogosys.org/images/fyne_OSX.png. Thanks
zmike for initial investigation.

@fix T6944

efl: include sys/types.h where ssize_t is used

This include is necessary according to POSIX. Without it, build
fails on certain platforms (QNX). It may already be included
implicitly in some of those files, but declare intent.

Fixes T6667.

Revert "edje: code refactoring for readibility."

This reverts commit 7fb47fc064b8f429242b4d5cdf57071475087821.

subsequential revert for previous one.

90a89d1ab2b5d23b29ad0f2680b77b5ba7f56b42.

Revert "edje: fix a regression error by 7fb47fc064b8f429242b4d5cdf57071475087821."

This reverts commit 475c93fd7599badad0c073e5eb025b92481b53ec.

long time after touch,
forgot that eet data has key-data pairing structure.
this change would break theme compatibility.

Thanks bu5hm4n for pointing out this.

ecore_con: do proper version test for TLS_method.

efl_ui_navigation_bar: less warnings

Reviewers: zmike, ManMower, devilhorns, Hermet

Reviewed By: Hermet

Subscribers: cedric, #committers

Tags: #efl

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

efl_ui_focus_manager_calc: fix warnings

Summary: its a false positive...

Reviewers: zmike, ManMower, devilhorns, Hermet

Reviewed By: Hermet

Subscribers: Hermet, cedric, #committers

Tags: #efl

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

elm_gengrid : fix gengrid cached items disapearing issue by wrong reference counting

gengrid item view is managed by cache while scrolling,
but efl_wref_add/del and VIEW_ADD is not paired well.

this commit is fixing the issue of item dispearing when it scrolls.

Change-Id: I28c08f12bbb0d2a098bcceea107c6a0993c220d7
Signed-off-by: SangHyeon Lee <sh10233.lee@samsung.com>

efl_ui_widget_focus_manager: less warnings

Summary: fix T7050

Reviewers: zmike, ManMower, devilhorns, Hermet

Reviewed By: Hermet

Subscribers: Hermet, cedric, #committers

Tags: #efl

Maniphest Tasks: T7050

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

elm_gengrid : fix gengrid cached items disapearing issue by wrong reference counting

this patch is porting from upstream.

Change-Id: Iee1af82ca1a121433015f2b84abdd43ce9e25337
Signed-off-by: SangHyeon Lee <sh10233.lee@samsung.com>

efl_ui_win: add missing wayland interface for setting role of winodw

@tizen_fix

Change-Id: I359bc4589260ffaabfae5b75e18e499c1dd2d282

elm_colorselector: fix improperly named API theme parts

Summary:
API parts require namespacing, these parts have been namespaced with
compatibility code added to handle legacy naming

Reviewers: cedric, Hermet, devilhorns, stephenmhouston

Subscribers: segfaultxavi, Hermet, #committers

Tags: #efl

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

Edje load: add safecheck for TEXTBLOCK-specific code path

There is a specific TEXTBLOCK part code path that shouldn't be taken
for other parts.

@fixes T6279

elm: refactor do not call efl_data_scope_get on a object without data

Summary:
A call to efl_data_Scope_get is actually quite dangerous,
efl_data_scope_get will return a pointer to a 0 sized segment in memory,
this is happening based on how the class data is organized. So in theory
you could use this pointer and accidently write to it. This resolves
this issue.

Reviewers: devilhorns, zmike

Reviewed By: zmike

Subscribers: cedric, #committers, zmike

Tags: #efl

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

ecore_con: FreeBSD 11.2 not providing TLS generic methods.

On some systems the TLS_client/server_method() is not available.
Using fallback. In time as openssl changes propagate this really
shouldn't be necessary.

Patch for T6342

Summary: Deprecate SSLv3.

Reviewers: zmike, raster, devilhorns

Reviewed By: zmike

Subscribers: cedric, #committers

Tags: #efl

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

ecore_wl2_display: remove infinite loop possibility

The wl_display_dispatch_pending returns -1 on failure.
This could make infinite loop in _ecore_wl_cb_pre_handle_data.

Change-Id: I1730e6007fd08db308be4412f0a8c9f51d7f3726

elm_toolbar: fix icon sizing issue

icon size * scale - this causes the issue.
this is not wrong,
but Tizen UX did not consider scale to icon size.
so, this is removed for compatibility.

@tizen_fix

Change-Id: Ib71ec1998767988bc7ad39299c3282132f8a0fd5
Signed-off-by: Bowon Ryu <bowon.ryu@samsung.com>

tests: remove eldbus special casing from checks to enable parallel test runs

Summary:
now that eldbus tests can safely run in parallel there is no reason to
prevent them from doing so

fix T6848
Depends on D6204

Reviewers: stefan_schmidt, cedric, ManMower, devilhorns

Reviewed By: cedric, ManMower

Subscribers: cedric, #committers

Tags: #efl

Maniphest Tasks: T6848

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

tests: use unique dbus bus names for each eldbus test case fixture

Summary:
dbus bus names are unique and cannot be acquired by multiple connections
simultaneously; by making this per-file unique, this part of the test can
run in parallel without bus collisions

ref T6848
Depends on D6201

Reviewers: stefan_schmidt, cedric, bu5hm4n, ManMower, devilhorns

Reviewed By: ManMower

Subscribers: ManMower, cedric, #committers

Tags: #efl

Maniphest Tasks: T6848

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

eina_debug_bt: only enable SIGPROF handler when profiling is enabled

Summary:
if profiling is not enabled then reaching this signal handler will always
result in a crash

moreover, the signal handler has no functional value if profiling is not active,
so do not add it until that point

fix T7028

@fix
Depends on D6311

Reviewers: ManMower, devilhorns

Reviewed By: ManMower

Subscribers: cedric, #committers

Tags: #efl

Maniphest Tasks: T7028

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

tests: add test for eina_debug signal handling

Summary:
currently this crashes, so add a simple test to ensure that it works
eventually

ref T7028

Reviewers: ManMower, devilhorns

Reviewed By: ManMower

Subscribers: cedric, #committers

Tags: #efl

Maniphest Tasks: T7028

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

ecore: call eina_debug_fork_reset() in fork reset function

Summary:
ensure that this occurs as expected when forks happen

note that this is already being actively tested in the elm unit tests

Depends on D6307

Reviewers: ManMower, devilhorns

Reviewed By: ManMower

Subscribers: cedric, #committers

Tags: #efl

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

eina_debug: add function to reset debug subsystem after fork

fork() kills this entire subsystem and leaves its internals
in an inactive but allocated state, so it's necessary to explicitly restart
everything in order to guarantee behavior

@feature

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

eina_debug: make shutdown perform cleanups and kill the timer thread

this should not remain active after eina has been deactivated, so ensure
that everything stops as expected

ref T7019

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

eina_debug_cpu: make shutdown perform cleanups and kill the timer thread

this should not remain active after eina has been deactivated, so ensure
that everything stops as expected

ref T7019

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

eina_debug_timer: make shutdown perform cleanups and kill the timer thread

this should not remain active after eina has been deactivated, so ensure
that everything stops as expected

ref T7019

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

eina: prevent threads subcomponent from being shutdown while eina is active

this basically breaks all thread usage if it happens, so ensure that the
threads infrastructure stays active for as long as eina is active

@fix

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

eina: call eina_threads_init from eina_debug_init

eina is guaranteed to crash in any process without this call now that
eina_debug exists and requires threads

@fix

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

eina: don't init subcomponents inside eina_debug

this should be done in a single location to avoid dependency conflicts
and make the code easier to read

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

eo: CRI when class constructor or destructor is called from thread

Summary:
calling the constructor and deconstructor from different threads
causes issues with mempool deallocation, so ensure that the class is
always initialized in the main thread for safety, e.g., call the
SOME_NAMED_CLASS macro during init to instantiate the class

fix T7003

Reviewers: bu5hm4n, devilhorns

Reviewed By: bu5hm4n

Subscribers: cedric, #committers

Tags: #efl

Maniphest Tasks: T7003

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

elm_list: return from sizing_eval immediately if called during constructor

Summary:
this is both invalid and useless, so return immediately before spending cpu
time and generating error messages

fix T7035

Depends on D6324

Reviewers: bu5hm4n, Hermet, woohyun, devilhorns

Reviewed By: bu5hm4n

Subscribers: cedric, #committers

Tags: #efl

Maniphest Tasks: T7035

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

elm_colorselector: avoid calling functions with null objects

Summary:
a number of calls during construction resulted in null object errors here

fix T7036

Depends on D6323

Reviewers: bu5hm4n, Hermet, woohyun, devilhorns

Reviewed By: bu5hm4n

Subscribers: cedric, #committers

Tags: #efl

Maniphest Tasks: T7036

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

elm_colorselector: avoid passing null to efl functions

Summary:
this check already existed in the function but was not applied to the
entire function

ref T7030
Depends on D6322

Reviewers: bu5hm4n, Hermet, woohyun, devilhorns

Reviewed By: bu5hm4n

Subscribers: cedric, #committers

Tags: #efl

Maniphest Tasks: T7030

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

evas: switch to EFL_EVENT_INVALIDATE for event grabber sub-object deletion

Summary:
this requires the parent of the deleted object to be reachable, so use the
invalidate callback as this is the last point during which it's possible to
retrieve a parent object

ref T7030

Depends on D6320

Reviewers: bu5hm4n, Hermet, woohyun, devilhorns

Reviewed By: bu5hm4n

Subscribers: cedric, #committers

Tags: #efl

Maniphest Tasks: T7030

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

efl_ui_win: check frame_obj existence before operating on it in pre_render

Summary:
this object will not exist for fake win objects, e.g., in enlightenment

fix T7033
Depends on D6319

Reviewers: bu5hm4n, Hermet, woohyun, devilhorns

Reviewed By: bu5hm4n

Subscribers: cedric, #committers

Tags: #efl

Maniphest Tasks: T7033

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

efl_ui_win: do not attempt to focus a null object on focus-in

Summary:
ref T7033
Depends on D6318

Reviewers: bu5hm4n, Hermet, woohyun, devilhorns

Reviewed By: bu5hm4n

Subscribers: cedric, #committers

Tags: #efl

Maniphest Tasks: T7033

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

eo: make the debug build print leaked errors

Summary: Depends on D6328

Reviewers: devilhorns, zmike

Reviewed By: zmike

Subscribers: cedric, #committers, zmike

Tags: #efl

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

eo: get rid of eo_id

Summary:
build with this disabled is just not working as the thread model of eo
falls apart. The threadmodel is required to have eo_id, as it decoded
the thread information in the eo_id, which is not working without eo_id.

This also fixes the testcases that have never been executed due to the
fact of the missing HAVE_EO_ID

fix T6610

Depends on D6327

Reviewers: devilhorns, zmike

Reviewed By: zmike

Subscribers: cedric, #committers, zmike

Tags: #efl

Maniphest Tasks: T6610

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

eo: only call efl_parent_set when parent_id is available

Summary:
this is no functinal change, we safe the calling to efl_parent_set.
Nothing else, the base implementation is initializated with NULL as
parent, and efl_parent_set with NULL would have returned in the first
few lines.

fix T7032

Reviewers: devilhorns, zmike

Reviewed By: zmike

Subscribers: cedric, #committers, zmike

Tags: #efl

Maniphest Tasks: T7032

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

tests: set monitoring interval for eio tests to 0.05s

Summary:
cpu usage is not an issue during test runs, we want the tests to finish
as fast as possible in all cases regardless of the cost. this resolves
the last remaining issue with eio tests where sometimes the fallback
monitor would trigger and the test suite would hang for 60+ seconds

fix T6915
Depends on D6248

Reviewers: stefan_schmidt, bu5hm4n, ManMower, devilhorns

Reviewed By: bu5hm4n, ManMower

Subscribers: bu5hm4n, cedric, #committers

Tags: #efl

Maniphest Tasks: T6915

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

tests: use fixtures for eio tests

Summary:
remove a lot of duplicated code and make this test suite read more like
all the other suites

fix T6813
Depends on D6247

Reviewers: stefan_schmidt, bu5hm4n, ManMower, devilhorns

Reviewed By: bu5hm4n, ManMower

Subscribers: Hermet, bu5hm4n, cedric, #committers

Tags: #efl

Maniphest Tasks: T6813

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

eio: add method for setting the global file polling rate for fallback monitors

Summary:
the default value for the fallback poll monitor timer interval is 60.0 seconds,
which is not useful for all cases, such as CI, where we don't care about cpu
usage and just want things to process as fast as possible at all times

this enables setting the interval to any value, ensuring that any existing
timers are modified to use that value immediately

@feature

Reviewers: stefan_schmidt, bu5hm4n, raster, devilhorns, ManMower

Reviewed By: bu5hm4n, ManMower

Subscribers: ManMower, raster, bu5hm4n, cedric, #committers

Tags: #efl

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

eina_threadqueue: add locking for variable used between threads

Summary:
the 'first' member of this struct is used simultaneously across
threads and can have conflicting read/write operations occurring at this
time

as int operations are not guaranteed to be atomic, ensure that we are
using atomic operations or locking as necessary

@fix

Depends on D6299

Reviewers: ManMower, devilhorns

Reviewed By: ManMower

Subscribers: cedric, #committers

Tags: #efl

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

eina_threadqueue: simplify/unify some locking code

Summary:
this moves lock calls outside #ifdef blocks to make the code more
readable

no functional changes
Depends on D6298

Reviewers: ManMower, devilhorns

Reviewed By: ManMower

Subscribers: cedric, #committers

Tags: #efl

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

eina_threadqueue: fix spinlock destruction by using spinlock api

Summary:
this would deadlock in some cases

@fix
Depends on D6297

Reviewers: ManMower, devilhorns

Reviewed By: ManMower

Subscribers: cedric, #committers

Tags: #efl

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

eina: replace memsets in thread debugging lock create/free with manual zeroing

Summary:
memset overwrites the thread value, triggering errors when running tools like
helgrind

attempting an operation on an invalid thread will cause errors naturally,
so zeroing the rest of the struct and ignoring the thread member is fine

Reviewers: ManMower, devilhorns

Reviewed By: ManMower

Subscribers: cedric, #committers

Tags: #efl

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

emotion_generic: clean up event handlers on fork failure

Summary:
this case would guarantee a crash if the module was ever unloaded
and the event handlers were called, and it leaks otherwise

@fix

Reviewers: ManMower, devilhorns, bu5hm4n

Reviewed By: ManMower, bu5hm4n

Subscribers: cedric, #committers

Tags: #efl

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

efl_mono: Fix call to System.Threading.Thread.

Summary: It was resolving to efl.Thread instead.

Reviewers: devilhorns, felipealmeida, zmike

Reviewed By: zmike

Subscribers: cedric, #committers, zmike

Tags: #efl

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

theme: rename non-API actionslider part to avoid confusion when reading

Summary:
no functional changes
Depends on D6211

Reviewers: cedric

Reviewed By: cedric

Subscribers: #committers

Tags: #efl

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

elm_dayselector: fix improperly named API theme parts

Summary:
API parts require namespacing, these parts have been namespaced with
compatibility code added to handle legacy naming
Depends on D6210

Reviewers: cedric

Reviewed By: cedric

Subscribers: #committers

Tags: #efl

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

theme: fix part namespacing in multibuttonentry theme

Summary:
this was released with improperly namespaced parts which must be maintained
for future releases, but the namespacing can be corrected for future
releases while adding aliasing to preserve compatibility
Depends on D6208

Reviewers: cedric

Reviewed By: cedric

Subscribers: #committers

Tags: #efl

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

theme: change part in elm photo group from swallow to spacer

Summary:
this appears to be a remnant of the time before edje had spacer parts
and other part types were randomly used instead

there is no library reference to this part and it is not namespaced so
there is no reason to leave it as a (confusing) swallow
Depends on D6040

Reviewers: cedric, Hermet

Reviewed By: Hermet

Subscribers: Hermet, #committers

Tags: #efl

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

theme: use elm namespace for spinner button signal source

Summary:
ref D6036

Depends on D6037

Reviewers: cedric

Reviewed By: cedric

Subscribers: #committers

Tags: #efl

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