ecore_wl2: Silence a commit ERR on window hide

Cosmetic fix, should be functionally identical.  Hide can be called
asynchronously, so it's ok, and the ERR can be silenced.

eolian: add APIs to get unit from object and state from unit

eolian: intial API for a new error mechanism

Unlike panic, this will be used to handle regular errors
such as parse errors. There will be no jumps and you
will be able to pass in a pointer to get the error
data into some local memory. That way you will be
able to override printing error messages.

eolian: initial API for a new panic mechanism

This will be used to handle unrecoverable errors.
For robustness, you will be able to set a custom
panic callback, jump and try to recover manually.

csharp: Fix support for ptr(structs)

In general, ptr(struct) parameters behavior depends whether the
parameter has the @owned modifier or not.

If there is no @owned parameter (meaning no transfer of ownership
happens) and it is a "complex" struct, with reference type fields
(like strings), the struct is converted to the respective
<Struct>Internal struct and passed with "ref" to the DllImport'd
function. For @in parameters, after the function it returns, this
intermediate struct is converted to the public struct type and
assigned to the original parameter, updating it to the external
world.

When we have ownership transfers, the structure is copied to unmanaged
memory and given to the callee. We can't send managed memory directly as
the callee may try to free it. On the managed side, the original struct
is left to be garbage collected normally.

csharp: Change policy on ptr(struct) owned calls

When transferring the ownership of a ptr(struct) from Unamanaged to
managed, we should marshal the reference-typed fields or they can point
to Managed memory that would make no sense to access from C (for
example, strings would return garbage).

The downside is that it can cause potential leaks if the receiver of the
memory do not free it. In the current state of the EFL api this can
happen in two places:

- Efl.Ui.List.SegArray::remove
return -> ptr(Efl.Ui.List.LayoutItem) @owned
- Efl.Ui.Focus.Manager::fetch
return -> ptr(Efl.Ui.Focus.Relations) @owned

The resources copied by both structs may leak when those functions are
overriden in C# and the values returned to the C code.

Also hide some internal stuff instead of exporting it and generate
implicit conversion operators for struct external/internal.

eolian_cxx: Add 'unknown' to parameter direction

It'll be used in eolian_mono as the 'direction' of return types in some
helper functions.

efl-cxx: Fix installation of efl_part_impl header

It was being installed directly into efl-cxx instead of efl-cxx/cxx.

Change-Id: I769c1b526ae2da465a1f4108117b2c18e30db01b

eolian: simplify lexer init

eolian: event/function_get_by_name -> by_name_get

eo: Orphan an object that is being invalidated.

Summary:
This will avoid infinite loops and errors when the parent tries
to orphan an invalidated child.

Fixes T6780

Test Plan: Run `make check`

Reviewers: cedric

Maniphest Tasks: T6780

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

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

eolian: more stringent and fine-grained validation

eolian: fix unit child management

Also pass correct unit to validation for performance.

eolian: merge objects in units

eolian: reduce the number of database traversals during validation

This significantly improves performance by not iterating certain
paths multiple times.

ee_drm: prevent extra flips

Similarly to commit 8b962d24acf192bd6c22de779c57eac73dd2f4db prevent
extra flips in the drm engine.

These are potentially more harmful to DRM as they can cause frame
drops and missed render opportunities.

ee_wayland: Prevent extra commits

It's possible that we unregister then reregister an animator so quickly
that the ecore_evas idle enter/exiters haven't had a chance to run yet.
In this case a render will come shortly anyway and we shouldn't use a
'false' commit to kick off a frame callback.

Silences an ERR and unmeasurably improves protocol utilization.

eolian: fix minor memory leaks

eolian: make sure dtors are always run correctly

eolian: remove the temps system

eolian: manage exprs through node system

eolian: manage typedecls through node system

eolian: manage class/vars through new node system

eolian: store types in node hash

eolian: generic dtor management for lexer

eolian: unified node allocation api

eolian: in-lexer-state current class storage

efl_ui_focus_manager_root_focus: switch to focus adapter

efl_ui_focus_manager_calc: set redirect before setting focus

settings the redirect might result in unfocus calls, that might unset
evas_object_focus on a object that might be registered in two managers
due to border linking. This fixes mysterical disapearing focus.

efl_ui_focus_manager_calc: do not set focus on a deletion manager

otherwise objects might already be on the way out, without a elementary
parent.

elm_interface_scrollable: make sure to have the content always focusable

elm_widget_item_static_focus: move canvas_object_set to later

We dont delete the adapeter when we are still focused, to set anyway the
correct view to it, we need to set the view to the adapeter as often as
possible

elm_genlist: support immidiate focus once the item is realized

this supports the case of item.focus = true before the item is realized.

elm_genlist: move focus recovering to first_touch instead of
focus_update

this means that the function is called more reliable, and not only based
on focus state, but also based on the redirect situation.

Change-Id: Ic23534aa8d6022f2c2b7341f61c98b28a1b7de0c

efl_ui_focus_manager_calc: add onedirection to dirty

elm_genlist: performe elm_item_focus_set

this means all neccessary fields with last focus and current focused
item are updated correctly

efl_ui_focus_composition: redirect canvas move events to the adapter

otherwise a moving canvas object will not be recalculated, as the
manager does not know about the new focus geometry

elm_widget_item_static_focus: fix fetching of the item

elm_widget_item_static_focus: delete adapter when deleted or unrealized

somehow genlist leaks the view sometimes, thus the adapter is not
deleted when the item is deleted. This resulted in strange ghost focus
objects in the window.

elm_genlist: set focus to the item when a user calls the function

elm_genlist: set focus to the item when a user calls the function

efl_ui_focus_manager_sub: flush the border if we cannot find a candidate

after that someone probebly wants to check where the last focus was on,
if the set of boder elements has changed since the last border flush, then the
element might not be registered in the parent manager

ecore_wl2_window: Silence some spurious warnings

ecore_wl2_window_commit() must be called during window size negotiation,
but this currently trips a warning when no frame callback has been
received for the first commit.  We can't even have frame callbacks at
that point because no buffer is attached.

Don't set up the commit_pending logic until after we have a buffer.

ee_wayland: Set window visible before trying to render it

if we try to render before setting ee->visible the results aren't
likely to be... visible.

eolian: turn some state APIs into inlines over unit APIs

pyolian: remove unneeded bindings in eolian_state

elua: inherit state from unit in eolian bindings

eolian: refactor doc ref resolution API

eo: fix invalidate to always happen.

genlist: fix "insane" order [BUG COMPATIBILITY]

This patch implements bug compatibility.

genlist internally uses both a tree structure with Eina_List and a flat
Eina_Inlist to track its items. ALL of the items are in the inlist while
subitems appear in their parent's list. As a consequence both lists must
be kept in sync pretty tightly. Obviously this is not done at all and
has led to countless bugs, as soon as tree or groups are used:
 - Invalid order of items (visually)
 - Invalid order of items with sorted_insert
 - Glitches in the matrix
 - Crashes with sorted_insert
 - Odd/even styles not properly set
 - Promote/demote functions broken by design
 - Developers send to psychiatric hospitals
 - Etc...

Legacy genlist (1.19 and before) used an inlist order that basically
didn't make sense, as it didn't follow the logical order of elements (as
they appear visually). Unfortunately this has "worked" (really, that's a
huge stretch to use this word here) for a long time this way. As a
consequence, some applications (*cough* empc *cough*) have relied on
this order to implement "next album" or "previous album" where the
album title is a group node.

By changing the order of items in the inlist, this has broken the
assumptions made above, and ends up in cases that return NULL, leading
to SEGV. Sure, the app should have checked NULL, but that's not really
the point here. The behavior has been changed.

This patch implements "fixes" for the following functions:
 - elm_genlist_first_item_get(): Don't return a parent
 - elm_genlist_last_item_get(): Return a parent
 - elm_genlist_item_next_get(): return a parent upon reaching the last child
 - elm_genlist_item_prev_get(): return a child when a parent is passed

Important notes:
 - This does not cover 100% behavior compatibility here. The only way to
   have it would be to simply revert the entire genlist code to its
   original version and never touch it again, ever.
 - An explicit API is required for an application to specify which API
   level it targets, so that we can cherry-pick which bug compatibility
   features we want to enable. We are already doing this for EDC,
   unfortunately.

@fix

fix T5938

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

tests: add genlist test to verify group item api behaviour

ref T5938

Signed-off-by: Mike Blumenkrantz <zmike@osg.samsung.com>

efl_ui_clock: do not locally store clock module api

there is no corresponding code to unset this global pointer when the module
is unloaded, so this will crash if elm_shutdown is ever called

Change-Id: Id296a265f899394d74812f8d79c14778127a5638
Signed-off-by: Mike Blumenkrantz <zmike@osg.samsung.com>

elm_gengrid: do not zero item cache if object is not finalized

this is a no-op case which causes unnecessary errors

Signed-off-by: Mike Blumenkrantz <zmike@osg.samsung.com>

efl_promise: avoid invalid calls when unsetting parent

if promise has no parent then the promise's loop is null

Signed-off-by: Mike Blumenkrantz <zmike@osg.samsung.com>

tests: remove invalid api usage from elm layout test

the comment here indicates that the author knew it was an invalid api use
so it should not have been here in the first place

Signed-off-by: Mike Blumenkrantz <zmike@osg.samsung.com>

tests: disable elm glview tests when running in buffer engine

Signed-off-by: Mike Blumenkrantz <zmike@osg.samsung.com>

elm_map: remove unnecessary error message in destructor

there is no check for the existence of this file so this will error most times
from it not existing

Signed-off-by: Mike Blumenkrantz <zmike@osg.samsung.com>

elm: call ecore_event_shutdown during elm_shutdown

ref 4e14ba3f4874b03185d5216829ecc1876d11c48e

Signed-off-by: Mike Blumenkrantz <zmike@osg.samsung.com>

tests: check whether ecore is properly shut down after elm_shutdown

Signed-off-by: Mike Blumenkrantz <zmike@osg.samsung.com>

efl_ui_win: do not trigger elm quit policy twice for same win

only check each policy for the case which should trigger it to avoid borking
main loop

Signed-off-by: Mike Blumenkrantz <zmike@osg.samsung.com>

tests: avoid test failures for elm_win when not running in X

Signed-off-by: Mike Blumenkrantz <zmike@osg.samsung.com>

tests: use buffer engine for elm_suite if engine is not specified

this should add some consistency to tests

Signed-off-by: Mike Blumenkrantz <zmike@osg.samsung.com>

tests: properly destroy elm_win timers to prevent crashes

Signed-off-by: Mike Blumenkrantz <zmike@osg.samsung.com>

efl_ui_win: track the finalize future for del to avoid crash

Signed-off-by: Mike Blumenkrantz <zmike@osg.samsung.com>

ecore_evas: implement the move method for basic buffer canvas

Signed-off-by: Mike Blumenkrantz <zmike@osg.samsung.com>

efl_threadio: always return a value to silence warning

efl_loop: fix exit code of the loop

For numeric types, eina_value_set() accepts values instead of references
on the value to be set. Hence, we were affecting as the exit code of the
loop a garbage value, yielding to invalid results.

eo: cleanup future on invalidate.

eo: add an helper to get an object from an Eina_Value.

eo: allow setting value eo Eina_Value from an Eo pointer directly.

eina: fix promise scheduler lookup to find it when given a resolved chain of future.

T6738

eina: avoid warning when accessing first item and just return an error.

eolian: update type/var name APIs

eolian: make more name APIs into inline helpers

eolian: update class name APIs

pyolian: update name APIs

elua: update eolian name APIs

eolian: expose short_name/namespaces via object

eolian: use short name for function pointer name

eolian: only merge units on success when parsing all files

doc: check - correct the ingroup name for legacy apis

Pyolian generator: adjust to follow eolian changes

eolian: simplify namespace handling

This is mostly a preparation for rework of name(space) APIs.

Pyolian: remove the useless Object.type binding

This is useless in py as Objects are never returned
directly, you always get the correct subtype (Class,
Variable, etc..). It was also clashing with Class.type

Pyolian: add tests for Eolian_Object

Two of the new tests are failing, the problem is that now
we have name clashes between Eolian_Object and Eolian_Class (at least)
For the moment I spotted:
 - Object.name clash with Class.name
 - Object.type clash with Class.type

Also fixed a typo in eolian_lib.py spotted by the new tests,
and removed the old tests for Declaration.

Pyolian: build the correct type from generic Object

This way all the generic Eolian_Object types are automatically
converted to the correct types (Class, Variable, etc...)

elm test - adapt to argc/v being regular argv 0 being binary name

ecore - efl thread - remove unused local vars

ecore - efl exe/thread - only allow run once ever

eolian: resolve to Eolian_Objects instead of voids

eolian: return Eolian_Object_Type from ref resolver

This needs only one enum and allows for better interop.

ecore: do not remove a timer that has already been removed.

csharp: update examples

Summary:
They were still using the old SetSize(w, h) API instead of receiving
eina.Size2D structs.

Test Plan: Build examples and run them.

Reviewers: felipealmeida, cedric

Reviewed By: cedric

Subscribers: cedric

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

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

csharp: generate helper constructors for structs.

Summary:
C# does not have a literal form for structs (like C++'s {} aggregate
initialization). Before this commit the user would need to explicitly
instantiate a struct and assign the required values to it, like:

   eina.Size2D size;
   size.W = width;
   size.H = height;
   widget.SetSize(size);

As a workaround, this commit generates helper constructor with
parameters corresponding to the struct fields in the order they are
declared. These parameters have default values if one does not want to
explicitly initialize all fields directly. With these constructs, the
above code could be translated to:

   widget.SetSize(new eina.Size2D(width, height));

It should be noted that the constructed struct will live on the managed
memory (GC) instead of the stack.

Test Plan: run "make check"

Reviewers: felipealmeida

Subscribers: cedric

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

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

eo: ERR some warning when illegally using invalidated object.

ecore: avoid calls to provider_find on invalidated objects in timer class

Signed-off-by: Mike Blumenkrantz <zmike@osg.samsung.com>

eolian cxx: fix typo

eo: when an object is reused, reset the invalidate flag.

eolian: remove old declaration APIs

eolian cxx: replace declaration api with obj api