efl_ui_focus_manager: fix infinit loop in enlightenment wizard

we unsetted and setted focus_manager once too often, so we ended up in a
infinite loop. This should be fixed now. Sorry Derek!

efl_ui_focus: merge efl_ui_focus_user and efl_ui_focus_object into one mixin

Summary:
focus_user and focus_object are similar classes. by merging them into
one mixin, we can maintain consistency.

Test Plan: make check

Reviewers: bu5hm4n

Subscribers: cedric, Jaehyun_Cho, woohyun, jpeg

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

Pyolian: fix and tests for recent changes

q66: please leave the pyolian updating work to me,
it's harder for me to fix wrong stuff instead of just
implement myself ;)

eina: make updating rectangle cache thread safe.

eolian: do not require unit when stringifying types

As it is no longer necessary to pass unit when evaluating exprs,
it is not necessary to pass it here either. Convert all the APIs
to the new style and update all instances in our tree.

eolian: do not require unit for expr eval APIs

The necessary information is now stored directly in the expr
during database validation.

Also enable expr validation for params.

eolian: cache enum expr in the database

eolian: expose _eval_type to internals

examples: Show off usage of "background" part

See bg_cxx_example_01

widget: Expose "background" part (for bindings)

cxx: Remove elm_widget.h from Elementary.hh

It's internal.

selection: Fix two remaining warnings

animation: Fix some warnings including a real one

Real issue here:

.../efl_animation_group.c:120:40: warning: for loop has empty body [-Wempty-body]
   EINA_LIST_FREE(pd->animations, anim);

efl_playable: split Efl.Player interface to Efl.Playable interface

Summary:
Efl.Player interface simply provides play functions,
but another interface which indicates Efl.Player will play is also
needed.

Test Plan: Run elementary_test->Efl.Animation tests

Reviewers: woohyun, conr2d, Jaehyun_Cho, jpeg, cedric

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

Efl.Ui.Progressbar: Add Efl.Ui.Translatable/Markable too.

it was missed in previous patch.

Efl.Ui.Progressbar: Add missing Efl.Text/Content

eo: Fix make check

The cache had a generation count, removing this check breaks eo suite.
EO is designed to tolerate init/shutdown cycles (as long as
dlopen/dlclose isn't involved).

Note: the ugly goto are useless as GCC -O2 understands EINA_(UN)LIKELY
and deals with it as expected (just look at the asm produced).

See 34d9f2070696027199a56cb621c0526ea1430e8f

eo: Remove unused goto label

edje: Add beta event when an invalid part is used

This can be used for debugging, by listening to
  EFL_LAYOUT_EVENT_PART_INVALID

Or just use
  EINA_LOG_LEVELS=edje:2
if you just need the logs.

panes: Fix invalid part name in layout recalc

code widget: Fix order of operations

super functions, when creating stuff, should be called first.

edje: add invalid part handler

ecore/edje/elm: Fix a few WRN from calls to NULL

This fixes some of the warnings generated by calling functions on NULL
objects. One of the main remaining points is to avoid unwanted warnings
on non-existing parts.

Ref T6326

eo: Warn on calls to NULL object

This makes eo print a WRN message in case a function is called on NULL.
efl_del is an exception to this rule (implemented in a hackish way, I
admit). I don't know any language or object model where using a null
object doesn't result in an exception or crash, except EO. In any case,
calls to null are invalid.

The next commits will resolve most warnings for EFL.

Ref T6326

evas: Remove stupid warning

edje: Add property "part_exist"

This allows to safely verify if a part exists, without triggering any
potential call to NULL object, or even requiring the efl_part() handle
to be created.

This is perfectly equivalent to edje_object_part_exists(), but
implemented by both edje object and elm layout.

selection: Disable debug logs

selection's namespacing still requires some love

elm genlist test - don't overflow test time nanoseconds if system slow

with a long enough test or slow enough machine these do overflow so
force everything to unsigned long long's for genlist test performance
measuring.

eina: remove usless newline

Summary:
ecore_evas: remove debug

eina: unregister log level when done with

Fixes a constant memory leak.

eina: introduce EINA_HOT and EINA_COLD

These attributes respectivelly expand to __attribute__ ((hot)) and
__attribute__ ((cold)) when available. They allow to mark functions are
being hot/cold (frequently used or not) as well as to qualify labels
within a function (likely/unlikely branches).

eo: speed-up generated calls by removing call cache

The call cache needed to by thread-local, to avoid concurrency issues.
Problem with TLS is that is adds an extra overhead, which appears to be
greater than the optimization the cache provides.

Op is naturally atomic, because it is an unsigned integer. As such, it
cannot be tempered with while another thread is reading it. When
entering the generated function, the first operation done is reading
'op'. If we have concurrency, we will have access sequences returning
either EFL_NOOP or a VALID op, because 'op' is not set until the very
end of the function, when everything has been computed. As such, we now
use the 'op' atomic integer to instore a lock-free/wait-free mechanism,
which allows to drop the TLS nature of the cache, speeding up the access
to the cache, and therefore making functions execute faster.

We don't test anymore the generation count. This can be put as a
limitation. If means that if you call efl_object_shutdown() and
re-initialize it later with different data, opcodes will be invalid.
I am not sure there is any usecase for this to ever happen.
We could move all the caches in a dedicated section, that can be
overwritten after a call to efl_object_shutdown(), but I am not sure it
will be very portable.

Benchmark: mean over 3 executions of
   ELM_TEST_AUTOBOUNCE=100 time elementary_test -to genlist

```
                     BEFORE               AFTER
------------------------------------------------------------
time (ns)            11114111647.0        9147676220.0
frames               2872.3333333333335   2904.6666666666665
time per frame (ns)  3869364.6666666665   3149535.3333333335
user time (s)        11.096666666666666   9.22
cpu (%)              22.666666666666668   18.333333333333332
```

Ref T6580

Reviewers: raster, cedric

Subscribers: cedric, jpeg

Maniphest Tasks: T6580

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

selection_dnd: fix warnings in clang

Fix warnings about type convert and vars init in clang and remove comments.

elm_panel: add exception handling in _drawer_close()

this fixes a bug with scrollable panel not being blocked
when it is closed. scroll is blocked in _anim_stop_cb()
which is called after elm_interface_scrollable_region_bring_in().
but if panel content is already at the target position, _anim_stop_cb()
is not called. so there is a need to check content's position and
handle the exceptional case.

Revert "elm_panel: add exception handling in _drawer_close()"

This reverts commit 2acfc3e381dc4fc87e3249b0f63f1ba5e0d1c2de.

elm_panel: add exception handling in _drawer_close()

this fixes a bug with scrollable panel not being blocked
when it is closed. scroll is blocked in _anim_stop_cb(),
which is called after elm_interface_scrollable_region_bring_in().
but if panel content is already at the target position, _anim_stop_cb()
is not called. so there is a need to check content's position and
handle the exceptional case.

elm_panel: prevent redundant callback function addition or deletion

there is a need to check if callback functions already exist or not
before adding or deleting them, because they are added or deleted
at two points:
in _elm_panel_scrollable_set() and _elm_panel_elm_widget_disable().

ecore: forgotten EFL_MAIN_EX.

elementary dnd: fix tautology

elc_popup: reorder the sub objects free sequence

Summary:
when the popup is deleted, some EVAS_CALLBACK_DEL callback functions
try to use already freed objects.
reorder free sequence to prevent it.

Test Plan:
1. elementary_test -to popup
2. check 'Enable popup scroll'
3. open several popup test and click Close button.
4. check that there are no error message

Reviewers: Jaehyun_Cho, bu5hm4n

Reviewed By: Jaehyun_Cho

Subscribers: cedric, jpeg, herb

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

examples: efl_ui_list: remove unused variables

examples: calendar_cxx: comment out unused variables

I normally would simply remove them, but they belong to some more
commented out code below. We should either remove both or get this
fixed.

examples: cxx: avoid tick in warning message

The compiler is not to happy about having this tick in the warning
message. Switch to the more formal can not and be done with it.

menu_cxx_example_01.cc:3:26: warning: missing terminating ' character

ecore_file_monitor: replace EINA_LIST_FOREACH to EINA_LIST_FOREACH_SAFE

If ecore_file_monitor_del is called inside the file monitor callback function,
eina_list found from monitor_hash would be freed. (You can check this inside
eina_hash_list_remove.)
Then, EINA_LIST_FOREACH makes one more for loop with invalid eina_list pointer.

EINA_LIST_FOREACH_SAFE can prevent from this problem.

Ecore_Win32: move monitor window initialization to earlier in init func

Summary: This monitor window is just used to receive events when mutiple
monitors are available. it should not be managed by the ecore loop
(creation and destruction events), so initting it earlier means
ecore_win32 attaches less memory/overhead to it as it's just being
used for notifications for devices.

Test Plan: DrMemory to check used memory

Reviewers: cedric

Subscribers: jpeg

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

Pyolian: add tests for recently added funcs

Correct module name for the interface context

docs: Show when classes really are top level EFL namespace

ecore_audio wasapi - fix ecore win32 handler deletion and destruction

vtorri pointed this out in T6607. this should fix this.

fixed T6607

@fix

Evil: remove useless dirent code

Test Plan: compilation of the EFL

Reviewers: cedric

Subscribers: jpeg

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

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

elm_tooltip: immediately show tooltip if callback is set for obj under pointer

@fix

elm_tooltip: force immediate hint recalc during tooltip creation if 0x0 size

try to avoid initial 0x0 sized tooltips

@fix

eina: document quadtree

Reviewers: cedric, ajwillia.ms

Subscribers: segfaultxavi, jpeg

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

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

efl: remove inclusion of dirent.h where it is not used

Test Plan: compilation

Reviewers: cedric

Subscribers: jpeg

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

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

efl: make efl build again

After a function pointer validation branch got enabled, it turned
out that people have been writing obviously incorrect eo files
all along.

So while I have no idea if this is logically fully correct, at
least EFL builds again now...

cc @thiepha

eolian: correctly validate function pointers

eo: fix tests build.

Revert "promise: Add even simpler helper for main loop promise creation"

This reverts commit e931fd698d26b8bec0e34239d2f79c059b339a51.

Revert "efl-loop: Don't use 'main' as a variable name"

This reverts commit 214dbdbd590a8f3fd9a0f3d5ef975d6ff9ce0e43.

eolian: aliased_base_get funcs don't need unit

eolian: type_typedecl_get doesn't need a unit

eolian: do not require unit for type_class_get

eolian: store typedecl/class in type

elementary: Fix constant qualifer warning message

Gcc complains here due to _wl_default_seat_id_get not accepting a
'const' Evas_Object, so to avoid the warning just case it to a normal
Evas_Object when passing in.

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

potfiles.in - remove src files no longer in tree

edje - multisense - only complain once about being inable to play audio

no p[oint creating streams of noise, so only once.

ecore_audio - only complain once about not loading pulse/sndfile

otherwise we get a complaint for everty time some audio needs/wants to
play and that's just noisy and ugly, so only do it once - the first
time sndfile/pulse are being loaded and it fails.

ecore_audio - don't about destruction because of failed sndfile load

this means objetcs are only partly destroyed and that is wrong.

@fix

eolian_cxx: support multiple function pointers

eio: remove unecessary include.

ecore: remove use of efl_main_loop_get from efl_net_socket_ssl_server_example.

ecore: remove use of efl_main_loop_get from efl_net_socket_ssl_dialer_example.

ecore: remove use of efl_main_loop_get from efl_net_session_example.

mmap memory allocation - do not used when under valgrind

we can't sensibly use things like massif to track memory if we bypass
itr with mmaping -1 fd anonymous memory... so if built with valgrind
support and running under valgrind, use malloc/calloc and free so
these tools actually do something useful for these bits of memory.

evas sw generic - don't dlopen osmesa until first needed.

osmesa needs llvm. llvm apparently just by dlopening or linking to the
lib (libLLVM...) gets you 3.5mb of dirty pages just in this lib. that's
a whole lib entirely dirty pages. odd and horrible. in fact once i
stoppd dlopening OSMesa all the time on engine init (and only when gl
is needed)... the amount of dirty pages went from 17208 to 8860.
that's a whopping drop of 8mb! 8mb saved! in fact just dlopening
osmesa and doing the other gl init stuff led to more anonymuse
mappings with dirty pages. 2 of them (2072k and 2076k) which baffled
me as that didn't seem like heap or efl's own data. these disappeared
along with libLLVM-5.0.so (3520k + 60k dirty pages). we stopped
linking/loading libedit (12k dirty), libglapi (20k dirty),
libLLVM-5.0 (3580k dirty), libncursesw (72k dirty),
libOSMesa.so (260k dirty), libtinfo (20k dirty). ... or at least
stopped until absolutely needed. total 17208k of dirty pages went down
to 8860.

my test case was just launching terminology (and doing nothing with it).

@fix memory bloating

elementary: Remove use of implicit declaration

evas_canvas_default_device_get used here leads to an 'implicit
declaration of function warning'. Use evas_default_device_get instead
to remove warning.

@fix

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

ecore signal - since we arent going to use threads due to osx, remove

remove thread code since osx is not happy with threads trapping
signals (or at least a thread setting up the handler and trapping
there with signal blocks...). this should now work universally.

cnp_dnd: make legacy APIs use new interfaces

Legacy APIs can uses efl_selection, efl_ui_dnd, efl_ui_dnd_container interfaces
with helper functions.

dnd: add efl_ui_dnd_container

efl_ui_dnd_container is the interface of dnd for container objects.

dnd: add efl_ui_dnd

Efl_Ui_Dnd is the interface for drag and drop of elm_cnp.

selection: add efl_selection interface

Efl_Selection is the object interface for selection api of elm_cnp.
It allows get, set, clear, check selection.

cnp_dnd: add selection manager

Selection manager implements functions for selection (cnp)
and drag and drop.
All objects can uses it APIs. It also supports multi-seat.

efl_ui_focus_manager_calc: store node data before the node is freed

Summary:
we can consider that the node is freed during focus_manager routine.
for example, efl_ui_focus_manager_redirect_set call edje event callbacks,
and a application can delete a object in the edje callback. if the object is
the focusable object of a node, focus_manager make the node freed.
the focus_manager is able to use freed node. (a good example is test_popup.c)

this prevent reusing freed pointers.

Test Plan:
1. elementary_test -to popup
2. popup-center-text + 1 button
3. Click the Close button
4. check that there is no erroe message

Reviewers: bu5hm4n

Reviewed By: bu5hm4n

Subscribers: cedric, woohyun, jpeg, Jaehyun_Cho

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

ecore signal write - save and restore errno since may be on mainloop

now i disabled the signal thread watcher ... save+restore errno.

ecore signal - for now disable the sig watcher thread and do on mainloop

we used to do signals on main loop. keep doing. the pipes should work
in cleanly serializing the signals irrespective of when/where they are
caught (because we do into kernel and back out again). hoping this
makes osx work again. can't test as i have no osx box or vm. works on
linux and freebsd though.

Elm_test panel: use eina instead of dirent to list files

Summary:
Short/middle term: use UTF-16 on Windows.

So I plan to remove most of external API (like dirent in Evil) and use only EFL to have less work later

Test Plan: compile and run elm_test

Reviewers: jpeg

Subscribers: raster, cedric

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

ecore: remove use of efl_main_loop_get from ecore_promise2_example.

eina: add eina_promise_continue_new to create a promise using an existing dead future.

ecore: no write after del for efl_loop_timeout.

ecore: remove use of efl_main_loop_get from efl_net_server_simple_example.

ecore: remove use of efl_main_loop_get from efl_net_server_example.

ecore: remove use of efl_main_loop_get from efl_net_dialer_windows_example.

ecore_drm2: simplify device selection code

The platform check was added for systems (like ARM) that don't generally
have PCI graphics devices.  However, now we pick a fallback device that
doesn't have a PCI constraint, so the platform check should no longer be
necessary.

ecore_drm2: Allow non-pci DRM devices to be selected

Summary: This is a tweak to c264ef264f0936632a1603fef45c59a50369631d for D5712 . chosen_dev in the loop was only being set for DRM devices attached to PCI devices. While this is useful for determining if the device is the preferred boot_vga device, There is no apparent requirement (comparing to Weston) for all DRM devices to be attached to a PCI device. (This is considering USB DisplayLink devices. I am not sure how the parent device tree is with these...)

Reviewers: devilhorns, ManMower

Reviewed By: devilhorns, ManMower

Subscribers: cedric, jpeg, #efl

Tags: #efl

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

ci: keep dockerfiles in sync with repo used for building on docker hub

We have an extra repo for these files on our github project to allow
docker hub building these images automatically for us to be ready when
CI kicks in. We still want to have them here as well to aid developer
using them for testing.

travis: add misc and wayland build options to build matrix

travis: drop older osx build, latest should be enough

travis: test docker build with prebuild image for speedup

ci: install wayland and other build options dependencies in Fedora image

We will use this as base image for different build option testing.

ci: enable examples build in for OSX target

eo: Add "destruct" event

Triggered after (almost) complete destruction of the object.

Not called "deleted" because the other event is already "del".
I don't like "destruct" much but this follows the terminology of
"constructor" / "destructor".

@feature

elm_test: Fix invalid define for widget style api

ecore: remove use of efl_main_loop_get from efl_net_dialer_websocket_example.