elementary: in case of an object being destroyed, there is no point to warn.

ecore_wayland: remove logically dead code.

Summary:
remove logically dead code.
execution cannot reach this statement.

Reviewers: cedric

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

Reviewed-by: Cedric Bail <cedric@osg.samsung.com>

efl_part: remove duplicated text_aliases code

Summary: this widgets inherit from Layout. they can use same text_aliases of Layout.

Reviewers: Jaehyun_Cho, woohyun

Subscribers: herb, cedric

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

Reviewed-by: Cedric Bail <cedric@osg.samsung.com>

arc: update config to use current config value names

Summary: Depends on D5867

Reviewers: cedric

Reviewed By: cedric

Subscribers: cedric

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

Reviewed-by: Cedric Bail <cedric@osg.samsung.com>

ecore: reduce pipe wait time during shutdown

Summary:
this is only meant to listen to data which is currently available,
not wait for new data

@fix

Depends on D5866

Reviewers: cedric

Reviewed By: cedric

Subscribers: cedric

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

Reviewed-by: Cedric Bail <cedric@osg.samsung.com>

ecore: poll the thread pipe twice during shutdown

Summary:
this resolves a race condition when a thread join was pending during
shutdown but a pipe write was needed in order for the join to be
successfully executed before shutdown had occurred

@fix

Reviewers: cedric

Reviewed By: cedric

Subscribers: cedric

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

Reviewed-by: Cedric Bail <cedric@osg.samsung.com>

elm_inwin: use macro for content set/get/unset

Summary: also, content_aliases is not necessary. it can use Layout's content_aliases.

Test Plan:
elementary_test -to inwin
elementary_test -to inwin2
elementary_test -to inwin3

Reviewers: Jaehyun_Cho, woohyun

Subscribers: herb, cedric

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

Reviewed-by: Cedric Bail <cedric@osg.samsung.com>

efl_selection_manager: fix build on MacOS and WL2.

efl_wl: Add NULL check

Reviewers: zmike, ManMower, Jaehyun_Cho, woohyun

Reviewed By: ManMower

Subscribers: cedric

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

ecore audio - add comment that this is an intended leak...

tests: eldbus: mark unused parameter as such

tests: ecore: remove no longer needed jenkins check function

After changes to the ecore_timer tests this function is no longer used
or needed. It can safely go.

tests: elementary: remove unused variable

Looks like a copy and paste left over from another test.

../../../src/tests/elementary/elm_test_genlist.c:58:16: warning: unused variable ‘type’ [-Wunused-variable]

ecore audio - dont unload modules etc. in case audio obj around on exit

if audio objects are around after ecore_audio has been shut down then
they may access the sndfile or pulse lib ptrs/funcs and thus dont
unload these. should fix crashes on shutdown.

ecore: and rename the test for Eina_Future to be the only promise test.

ecore: remove tests of old Efl_Future based on Eo.

evas: by moving lifecycle management to be handle by the death of the parent, there should be no call to efl_del.

evas: the life cycle of an object inside an hash can not rely on the parent destroying, use ref/unref properly.

ecore: rely on object being destroyed on parent invalidate.

ecore: pipe fd can become invalid while being processed.

evas: we should always sync the canvas before destroying the object.

tests: ck_assert_ptr_null -> ck_assert_ptr_eq

tests: ck_assert_ptr_null -> ck_assert_ptr_ne

eina: mark thread by default as not cancellable as per Eina documentation.

By default all Pthread are cancellable. This leads to strange behavior that
happen during shutdown and hasn't been figured out. Most likely fix for T6622.

tests: move bindings tests to using checked fixtures too.

T6862
T6811

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

elementary: fix efl_ui_view_list examples

Reviewers: SanghyeonLee

Reviewed By: SanghyeonLee

Subscribers: cedric

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

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

tests: add test for check < 0.11 to enable timing info

tests: elementary: fix build break after rename of efl_ui_view examples

Commit ae8df9ba076e6086900353f0e10480bb2302b44d renamed the files but
did not rename the references in the Makefile.

elm - selection manager - fix string buffer handling and format types

this has 2 bugs really. 1 - the format masks for plain text included
markup which ... is wrong - right? that should have a mime type of its
own... either way it leads to plain text pasts being handled wrong in
wayland... so this fixes that

also the string buffers in wayland just were nothandled right. buffers
were raw binary buffers plus size,not strings, but they were strdup'd
etc. assuming nul termination. oh so wrong to do that. this also fixes
that as i fpound it while fixing the above format bug.

this fixes T6284

@fix

eolian: _ is allowed in doc references

Fix static analysis result

[Dereference after null check]

(1) src/lib/ecore/ecore_main.c
 - _efl_loop_handler_efl_object_finalize checks if pd->loop_data is NULL.
   After that, _handler_reset > _handler_clear > _ecore_main_fd_handler_del >
   _ecore_main_fdh_pool_del is directly dereferencing pd->pool_data.
 - _efl_loop_handler_efl_object_parent_set checks if pd->loop_data as well.
   Then it calls _handler_reset as well.

(2) src/lib/ecore_wayland/ecore_wl_dnd.c
  - ecore_wl_dnd_selection_set checks if t - result of wl_array_add - is NULL.
    And it is dereferecing t directly for wl_data_source_offer.

(3) src/lib/elementary/efl_ui_dnd.c
 - Third parameter const char *data could be NULL.
   In this case strlen dereferences NULL. The data should be non NULL value.
   I have checked this with Mr. Thiep Ha.

(4) src/lib/evas/canvas/evas_object_inform.c
 - _efl_canvas_object_efl_gfx_stack_stack_below checks if obj->layer is NULL.
   So it could call evas_object_inform_call_call_restack which is dereferencing
   obj->layer directly.

efl selection manager - build again if no wl or no x is enabled

so make check and my builds passed because i have wl and x enabled. if
one of these isnt then this build will fail. my fix fails. this fixes
that.

elm - fix selection debug print to not print unterminated buffers

so it's printfing a non-0 byte terminated string ... it's a
buffer/slice which has explicit lenmgth and no nul, so fwrite it
approplriately.

evas - plug in dirty + pixels get hold in "is inside" for images

this should fix T6791

@fix

efl selection manager - handle multi window selections properly

tell other windows they lost their selections correctly if you have
multiple windows within the same process. this fixes multi-window mode
in terminology.

@fix

efl selection - fix paste with multiple windows and filter by window

so changes in elm cnp (efl selection) broke cnp with terminology
multi-window mode. this fixes it.

@fix

efl threadio - make call and call sync func proto same as efl events

well call sync returns a void * too.... instead of just void return
(nothing) but this makes it easier to write and maintain code due to
consistency of function callback signatures.

efl exe - map some "well known" exit codes to specific errnos

efl task - support flag to ignore exit code errors for future resolve

ecore - thread - document thread max limits more sensibly

leave the exact numbers less defined because core count can change on
the fly and could be virtually limited by policy in future etc. so
keep things less exactly defined so people dont go depending on exact
results which was never really intended.

elm config tool - get web backend when entry is deleted not on terminate

terminate may be called after widgets/windows are deleted already...
so donyt get this stuff in terminate.

elm config - dont leak config if re-loading it again later on

delete config object and so on before the relaod

elm win - change parent to only use win parents in efl win struct as parent

tests: increase step sizes for eina_fp_div and eina_fp_mul tests

fix T6841

Reviewed-by: Stefan Schmidt <stefan@osg.samsung.com>

tests: reduce looping in slstr_thread thread callbacks

this works out to the same number of total tests as slstr_many but
now split across all the threads

fix T6846

Reviewed-by: Stefan Schmidt <stefan@osg.samsung.com>

tests: reduce looping in eina_test_spinlock

no changes to coverage

fix T6845

Reviewed-by: Stefan Schmidt <stefan@osg.samsung.com>

tests: add eina_test_hash_int64_fuzze

ref T6844

Reviewed-by: Stefan Schmidt <stefan@osg.samsung.com>

tests: reduce looping in eina_ustringshare_putstuff

fix T6843

Reviewed-by: Stefan Schmidt <stefan@osg.samsung.com>

tests: reduce looping in eina_binshare_putstuff test

also slightly increase eina_hash coverage

fix T6843

Reviewed-by: Stefan Schmidt <stefan@osg.samsung.com>

tests: improve eina_binshare_collision test

remove rand() usage and loop less

fix T6843

Reviewed-by: Stefan Schmidt <stefan@osg.samsung.com>

tests: change eina cmdline test names to match api namespaces

Reviewed-by: Stefan Schmidt <stefan@osg.samsung.com>

tests: remove 9900 loop iterations from eina_test_file_thread

no changes in coverage

fix T6842

Reviewed-by: Stefan Schmidt <stefan@osg.samsung.com>

tests: rework eina hash fuzzing tests

these are supposed to be for testing the finding of values not in the
hash, not basic hash operations

fix T6839

Reviewed-by: Stefan Schmidt <stefan@osg.samsung.com>

tests: fix mem leak in eina_test_hash_add_del_by_hash

Reviewed-by: Stefan Schmidt <stefan@osg.samsung.com>

tests: reduce malloc usage in eina_test_hash_extended

ref T6839

Reviewed-by: Stefan Schmidt <stefan@osg.samsung.com>

tests: reformat eina_test_hash.c

no functional changes

Reviewed-by: Stefan Schmidt <stefan@osg.samsung.com>

tests: use namespacing for eina hash tests

no functional changes

Reviewed-by: Stefan Schmidt <stefan@osg.samsung.com>

tests: use namespacing for safepointer test names

Reviewed-by: Stefan Schmidt <stefan@osg.samsung.com>

tests: reduce array size in safepointer tests

this reduces test runtime by ~1.0s while having no effect on coverage

fix T6836

Reviewed-by: Stefan Schmidt <stefan@osg.samsung.com>

tests: simplify eina safepointer tests to be more readable

no functional changes

Reviewed-by: Stefan Schmidt <stefan@osg.samsung.com>

tests: use EINA_C_ARRAY_LENGTH macro in safepointer tests

no functional changes

Reviewed-by: Stefan Schmidt <stefan@osg.samsung.com>

tests: rework strbuf append/prepend tests

there's no point in iterating this many times; the base step size is 32
and so each loop will only increase the buffer...not at all since it takes
3 loops for the buffer to increase by the step size

verify that the buffer increases and then stop instead of spinning uselessly

fix T6835

Reviewed-by: Stefan Schmidt <stefan@osg.samsung.com>

tests: reformat eina_test_strbuf.c

Reviewed-by: Stefan Schmidt <stefan@osg.samsung.com>

tests: use namespacing for eina strbuf tests

Reviewed-by: Stefan Schmidt <stefan@osg.samsung.com>

tests: improve eina_test_list_shuffle

reduce iteratons by a hundredfold, remove verifying of rand() seed,
check whether shuffle actually shuffles

fix T6833

Reviewed-by: Stefan Schmidt <stefan@osg.samsung.com>

tests: rename eina_list tests to use list namespace for readability

Reviewed-by: Stefan Schmidt <stefan@osg.samsung.com>

tests: remove sleep calls from eina_barrier_test_simple

the timing does not matter for these tests, so there's no point in
randomly waiting longer and delaying build completion

fix T6832

Reviewed-by: Stefan Schmidt <stefan@osg.samsung.com>

tests: improve coverage in eina_rbtree_fuzzy test

Reviewed-by: Stefan Schmidt <stefan@osg.samsung.com>

tests: simplify eina_rbtree_fuzzy test and speedup by infinity%

this test is to try and match values in a tree which have not been
inserted, so remove irrelevant insert/remove operations as well as
insane looping

fix T6830

Reviewed-by: Stefan Schmidt <stefan@osg.samsung.com>

tests: greatly reduce iterations for eina_rbtree_remove test

this has no bearing on coverage, so there is not much point in uselessly
iterating here

ref T6830

Reviewed-by: Stefan Schmidt <stefan@osg.samsung.com>

tests: reduce malloc calls in eina rbtree tests

ref T6830

Reviewed-by: Stefan Schmidt <stefan@osg.samsung.com>

tests: rework eina inlist tests to be infinity% faster

coverage currently at ~95%

fix T6829

Reviewed-by: Stefan Schmidt <stefan@osg.samsung.com>

elm: remove hacky calls to emotion_init/shutdown

these are irrelevant since emotion manages init/shutdown internally

fixes elm test pass state

Reviewed-by: Stefan Schmidt <stefan@osg.samsung.com>

emotion: call shutdown when smart objects are destroyed

the constructor calls init for every object, so ensure a matching
shutdown call exists to balance the init count

ref 517471190f87e473e3bc38584c23ae28fb772c7c
ref T6816

Reviewed-by: Stefan Schmidt <stefan@osg.samsung.com>

tests: unbreak PACKAGE_BUILD_DIR usage in makefiles

for some reason, a few projects changed this from the expected value of
abs_top_builddir for their test suites, breaking the usage of this define
within those tests

fix T6812

Reviewed-by: Stefan Schmidt <stefan@osg.samsung.com>

tests: disable xim in ecore_imf test when DISPLAY is not set

tests should be able to run in console without failing

Reviewed-by: Stefan Schmidt <stefan@osg.samsung.com>

elput: include Eina.h in public header

Reviewed-by: Stefan Schmidt <stefan@osg.samsung.com>

tests: redo eina in-tree module loading

this performs an eina init/shutdown check and loads available modules
before running remaining tests

note that this still does not work and may never have worked because
PACKAGE_BUILD_DIR is broken

ref T6812

Reviewed-by: Stefan Schmidt <stefan@osg.samsung.com>

tests: move to using checked fixtures for all test suites

individual tests should not need to explicitly call init/shutdown functions
in most cases, and many did not properly do this anyway

see followup commit which resolves some issues with eina tests

ref T6813
ref T6811

Reviewed-by: Stefan Schmidt <stefan@osg.samsung.com>

tests: remove duplicated eina mempool test

Reviewed-by: Stefan Schmidt <stefan@osg.samsung.com>

eldbus: call ecore_init in eldbus_init

eldbus requires ecore and cannot run without it

@fix

Reviewed-by: Stefan Schmidt <stefan@osg.samsung.com>

tests: make test time output configurable with TIME_DIFF_THRESHOLD

setting this value too low will likely lead to unmanageable results

Reviewed-by: Stefan Schmidt <stefan@osg.samsung.com>

tests: add instrumentation to existing tests to find slow tests

efl_check.h must be included and the EFL_START/END_TEST macros must be
used in place of normal START/END_TEST macros

timing is enabled when TIMING_ENABLED is set
https://phab.enlightenment.org/w/improve_tests/

Reviewed-by: Stefan Schmidt <stefan@osg.samsung.com>

tests: match test names using strcasecmp when parsing cmdline

Reviewed-by: Stefan Schmidt <stefan@osg.samsung.com>

build: merge all coverage exclusions into a single command and exclude more files

files in the excluded directories either can only be tested with stochastic results
or testing them serves no purpose

pruning these in separate commands adds 10-20s to each coverage build

ref 28f9a5cdb36f01255a94ff8146ee915be506288f
ref T6825

Reviewed-by: Stefan Schmidt <stefan@osg.samsung.com>

ecore_con: resolve the possible read of uninitalized data.

Submitted-By-Off: JongMin Lee<jm105.lee@samsung.com>

ecore_con: resolve the possible read of uninitialized data.

Signed-Off-By: JongMin Lee<jm105.lee@samsung.com>

ecore: resolve the possible use of invalid handle.

Signed-Off-By: JongMin Lee<jm105.lee@samsung.com>

emil: resolve possible memory leak.

Submitted-By-Off: JongMin Lee <jm105.lee@samsung.com>

evas_gl_image: add null check

evas textblock: add null check to avoid segfault

evas: parent need to exist to create a vg object.

elm_win: Improve wayland cursor change race

When changing cursors under wayland sometimes we'll see the old cursor
moved to the new hotspot briefly before the cursor changes.  This makes
that suck less often.

A proper fix would involve creating a new wayland surface for every
cursor change (actual change, not just a new frame of an animated
cursor).  Given the current internals this is invasive.  Do the
easy thing for now.

efl_ui_focus_manager_root_focus: refactor _state_eval

Summary:
we have to check whether all of descendant are none_logical.

ref T6800

Test Plan:
1. elementary_test -to ctxpopup
2. Click 'Ctxpopup with user content'
3. Try focus ctxpopup content(button) using arrow key.
4. Check that the content is focused (you can move scroll bar)

Reviewers: bu5hm4n

Subscribers: cedric

Maniphest Tasks: T6800

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

elm_suite: add test case for history_pop

elm_suite: add test case for no child

make sure 39739ba333b63c59ece4c5622adedb106ebab9b1 does not happen again

elm_win: Remove unused variable

hot_obj is unused

theme: Fix broken wayland cursors on non-legacy windows

since commit dd4467505ea29d6120e5e7d467d76836a6630ff4
wayland cursors haven't worked on non-legacy
windows because efl/pointer doesn't exist.  Add that.

efl_ui_view_list : change the name of Model based list

Summary:
model based list need to be under the namespace of 'Efl.Ui.View".
thus, I renamed 'Efl.Ui.List' to 'Efl.Ui.View.List' properly.

Test Plan: N/A

Reviewers: cedric, felipealmeida

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

efl_ui_focus_manager_calc: node depth is 0 for a NULL object