eio: fix free eina_promises in error_callbacks

eina: eina_promise_all callback cast removed

eina: Avoid promise early deletion when cancelled.

Cancelling a promise will fulfill it but won't actually free the memory. This
memory is under custody of the owner, who must either call value_set or
error_set to finish it.

eolian-cxx: Rewrite to accomodate new features of the C++ binding

eina-cxx: Add placeholder for futures in C++ binding

eina-cxx: Fix wrongful operator++ for iterator and docs

eina-cxx: Add c_str() to string_view

eina-cxx: Restrict templated constructor

eina-cxx: Add optional specialization for pointer types

elm: Fix wrong use of char instead of string and void out parameter

edje: Fix wrongful uses of char instead of string in Eolian

eio: treat Eina_Accessor NULL as success instead of failure in model

We treated as errors when Eina_Accessor was NULL, which meant
that we could never get an empty range os children. Treat it
as success and pass an empty Eina_Accessor

eio: Add mime_type property in Eio_Model

eio: Eio_Model parent_get return parent model

eio: Fix Eio_Model promise accessor

ecore: Fix ecore thread promises

Make sure we free the promise memory, set MAGIC numbers and avoid
leaking the underlying eina promise

eina: Fix promise tests and avoid early deletion

- Progress notify refcount fixes, avoiding early deletion
- Avoid leaking memory on tests

eina: Fixes on promises benchmark

- Updated after pointer vs value change
- Updated callbacks after removal of promise argument

eo: Fix compilation of benchmark

- Missing Eo* self in method declarations.
- Gcc complaining about variable-sized object being initialized.

eolian: Fix generating pointer promises for class types

Eo: fix reference hack when calling composite objects.

We used to keep a reference to the parent object and have it in the call
structure although we were actually calling the function on the embedded
object. This was needed because we wanted to unref the parent correctly.
This was incorrect (and marked as a hack) and now I finally gotten
around to implementing the (amazingly simple) fix to remove this
workaround.

Essentially we just ref the comp object, unref the parent, and let the
normal eo call flow to unref the comp object correctly later on, like it
would have unreffed the extra ref we had for the parent.

FDO icons: and finally install the icons in fdo format

After this commit you will find a new icon theme in your system,
it is called Enlightenment-X and is build using the same icons used
in the elm theme. Using this theme you can have non-efl
and efl apps to use the same icons.

#StupidAutotools that require me to list each icon and each folder
to be able to install them

Eeze scanner: Fix setting of SUID.

I guess eeze scanner was not operational at all because the SUID bit was
never set. The reason for that was that where it was put in the makefile
made it not be a make rule (where @ would have worked) but used by a special
autofoo rule.

gitignore++

ecore_evas_cocoa: better handling of resize callback

evas-gl_cocoa: resize the gl context before resizing the window

This order seems more right. It does not have a visible
impact though...

ecore_cocoa: fix main loop handling

Pffff... another tricky one...

-windowDidResize is actually also called when the window is not
resize by the user, leading to multiple event posting, and of
course the infamous call of ecore_main_loop_iterate() when it
was not paused at all, leading to messy events handling... the
most visible being the initial resizing of the window going rogue.

We now ensure with -windowWillStartLiveResize and -windowDidEndLiveResize
that we only send an event when the user requested it. Since the main
loop is paused at this point, calling ecore_main_loop_iterate()
becomes safe.

Fixes T3648

ecore_audio: remove useless eo data in core audio

Eina promise: Fix broken return value.

I broke it in dc73f6644e0ed8fdbd2c6c8187591fb7eb8027c7.
Excuse of the day: I'm not working from my usual spot and there's screen glare
here that made me not see the *. Good excuse?

Generic modules: Fix out of source build.

Autotools really really really sucks.
Because of the above, we need to manually list both source dirs and
build dirs, because some of the include files are in the source dirs and
some are in the build dir (generated).

Eina promise: Simplify code and remove shadow variable.

There was no need for the extra variable. This also fixes a shadow
warning.

Eo: remove strict class checking for composite objects.

When compositing objects, we were checking that class_of(B) is in A's
inheritance tree before allowing attaching B as a composite object of A.
This is wrong and breaks a few cases, for example: B extends a class that
is in A's inheritance tree or B implements an interface that is in A's
inheritance tree.

Thanks to Marcel for reporting.

Enable evas loaders by default, if dependencies are found

I really cannot see why we should require to use --enable-feature on those, they should be safe.

Fix report for image loaders

In the final report we want to show if the loader is enabled, not if it's wanted

Fix building of librsvg if requested

This typo was preventing the svg loader to be built in every cases

ecore_cocoa: fix broken scrolling

Evas events recently changed, and involved the use
of the 'multi' field within a mouse mouve event.
It was used afterwards for mouse scrolling, but since
it was never set by ecore_cocoa, scrolling went
broken...

Fixes T3789

elementary: remove redundant code

The same checks are performed later on.
There is no use doing it twice.

Only affects Cocoa windows.

ecore_cocoa: fix window hiding logic

ecore_cocoa: remove extra whitespaces

edje: don't return a boolean when a pointer is expected

Fix a warning thrown by Apple-llvm.

elm_config: force icons in preview to keep aspect ratio

#SizeHintsMess... I really cannot understand why aspect_fixed is not respected :(

Configure: 2 small in-string quote changes

This should be harmless and just fix syntax hilighting is some editors (geany at least)

eina: Fix magic check for eina_promise_owner_buffer_get

eina: Fix EINA_MAGIC and eina_promise_then NULL check

ecore: add a test for detecting the double free problem.

ecore: properly handle double free case of timeout promise.

The bug came from the fact we need to handle the destruction of the
main loop which destroy the underlying timer. The event handler that
catch the destruction of the timer can not make the difference between
eo_del call from the timeout code and eo_del from the main loop
destruction. By removing the event handler, the double free is properly
avoided.

ecore: use EINA_ERROR_PROMISE_CANCEL instead of a custom Ecore one.

ecore: do not generate error while rescheduling the last timer.

eina: protect promise death during callbacks cancel.

ecore: Ecore_Cb is used by elm_box.eo.

ecore: Efl.Timer should be Efl.Loop.Timer for coherence.

As we add more object in the main loop, they can't live in the top
namespace as they make little sense there (Efl.Fd !). For coherence,
everyone should in the loop namespace, so move timer there.

ecore: remove Ecore_Animator Eo object.

This is only a legacy API.

eina: Remove Eina_Promise* parameter to callbacks to promises in examples

eina: Add NULL checks and EINA_MAGIC checks to promises

Add NULL checks and EINA_MAGIC checks that avoid crashing when it
shouldn't.

eina: Remove more Eina_Promise* parameter in promise callback

Remove not very useful parameter to eina_promise_then callbacks.

wayland_shm: Fix more dmabuf failure corner cases

With the exynos allocator a failure can happen during a resize, and the
resize path wasn't properly falling back to wl_shm buffering.

psd_loader: Fix bad assumption about signedness of chars

Whether "char" is signed or unsigned is architecture dependent, so if we
know we need a -1 in a char type we must declared it to be signed.

ecore_anim: Fix bad assumption about signedness of chars

Whether "char" is signed or unsigned is architecture dependent, so if we
know we need a -1 in a char type we must declared it to be signed.

eina: Remove Eina_Promise* parameter in promise callback

Remove not very useful parameter to eina_promise_then callbacks.

eolian-cxx: Fix C++ example compilation

eina: Fix examples using new Eina_Promise semantics

eina: Remove unnecessary indirection to promises

Now when dealing with pointer types, we will not get pointer to
pointer semantics in callbacks and eina_promise_owner_value_set
for Eina_Promise.

It will work as expected:

Eina_Promise_Owner* promise = eina_promise_add();

void* p = malloc(sizeof(T));
eina_promise_owner_value_set(promise, p, &free);

autotools: use DISTCLEAN rules to properly destroy files.

ector: variable in _SOURCES do not work.

This fix the left over etc .deps during make distcheck.

wayland_shm: Add exynos allocator for dmabuf

Allows clients on exynos hardware to allocate GEM buffer objects
to back DMAbuf buffers.

wayland_shm: Abstract out dmabuf management

Split out the intel specific stuff into an abstraction in preparation for
adding exynos support.

Eina: Fix eina_promise_all refcount ordering problem

The call to eina_promise_then steals the first ref'count, so it is
possible that the promise is freed after the eina_promise_then,
so we need to eina_promise_ref before eina_promise_then.

evas: Get pixels from rendered 3D scene

Summary:
Implemented interface Efl.Gfx.Buffer functions bufer_map/unmap for Efl.Canvas3D.Scene.
Added function e3d_drawable_texture_rendered_pixels_get to module evas_gl_3d
to getting pixels from FBO. Added wrappers for functions
e3d_drawable_texture_rendered_pixels_get and e3d_drawable_texture_id_get
to have possibility call it through engine functions.

Reviewers: cedric, Hermet, raster, jpeg

Reviewed By: jpeg

Subscribers: jpeg

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

evas: fix huge memory leak for non-async rendering

So... I had issues with evas-fb engine which was massively leaking,
one image per frame.
After investigating a bit with @cedric on IRC, the reference count
of the cache entries was always 2 before the engine dropped.
So, for each frame with an animation, we could never drop a cache
entry, leading to a trumendous amount of memory leaking.

Now for non-async rendering, we copy the behaviour of
evas_render_pipe_wakeup() which is called in async-mode,
and actually drops a reference in the cache entry.

Fixes T3763

edje_edit: add API for set/get part item

ecore timeouts - dont crash on cancel

this fixes a nasty double deletion on cancel of timeouts. at least for
now as long as we have promises.

fix warning in build with uninit variable in generic loaders for xcf

elm_image: fix break related with elm image error enum

Efl.Pack: Replace pack_direction with Efl.Orientation

Since I first made the pack_direction before orientation
existed, the names didn't match. This should improve API
consistency.

Fix legacy name of elm_win_efl_screen_dpi_get()

...spank again goes to Ji-Youn Park, that insist to not even try his work

elm_win: remove main_menu, window_id, fake_canvas property from eo

elm_test: Add test case for event refeed

Not sure if this test case is really good, but it was
necessary to prove that input event refeed can work from
application side. This is a feature that should work
but shouldn't be used :)

elm_win: Forward events between window and evas

This does:
1. Forward keyboard events from evas to win
2. Allow feeding external input events

Input events can be faked by apps by simply forging
eo objects of the proper type (key or pointer evt) and
calling eo_event_callback_call().

Such events will be forwarded to the internal Evas, and
some bool flags prevent infinite refeeding loops.

efl_event_dup() returns fake events for this to work.

@feature

elm: Partially revert "elementary: avoid random memory access."

This reverts commit ac5a0b7e79d6b32285bd7b794f050826f38cc7c2.

I added assert(pd == sd) in the modified code and couldn't find
anything suspicious. The proper class is used in eo_data_scope_get().
I also can't find a crash or a bug currently in hover. I know there
was a problem earlier, but that was fixed in 3092e08a83f754ba8d49c.

Now, if there is still a problem, we should check it carefully.
Pinging @cedric

Efl.Ui.Box: Fix direction set

VERTICAL means UP but UP was ignored...

edje_edit: add API for get part item attributes by item index

Cedric revert patch 6f7608befd006f8076d237bee5d17c9e9e1e09ac same days
ago.

<quote Cedric>
Your patch 6f7608befd006f8076d237bee5d17c9e9e1e09ac did break elemines
which does relly on the proper index to be given to it. I think
Edje_Edit should be fine and use the index if no name is provided (See
edje_load.c around L1180). In all case this was clearly an ABI break and
needed to be reverted.
</quote Cedric>

So now edje_cc not guarantee name for part items. This is bad, because
edje_edit use item name for access to item attributes. If item have not
name we cann't get it attributes, because edje_edit compare the given
name with part items name from edj-file, strcmp dislake NULL args.
For now I see one way mark all edje_edit_part_item* APIs as deprecated
and add new APIs for manage part items by index.

For clearly and history:
The eason for revert - https://phab.enlightenment.org/D3820
The proposed solution - https://phab.enlightenment.org/D3907

examples: elementary: fix the wrong spell in comment.

elementary: avoid random memory access.

It seems that this content_get function are not getting the data pointer
they expect at all.

elementary: make eo_provider_find lookup in both eo and elementary parent hierarchy.

We can not assume that the elementary parent will be also somewhere in the eo parent
hierarchy. We need to explore both when doing an eo_provider_find in elementary for it
to be really useful.

evas: attempt to fix massive memory leak in the fb engine.

T3763

emotion: properly fix build of vlc generic player.

evas: correctly handle dependencies of evas generic loaders.

autotools: we can't destroy this files as it break a make clean all cycle.

Revert "Revert "elput: add missing dependency on libudev""

Apparently, libudev is not thread-safe which means we cannot use Eeze
for getting the udev but rather need to actually use libudev, so those
revert needs to be reverted.

This reverts commit 354bc66d0d58a74b0464275e865a9cc6c920a538.

build: use the normal efl tree macros for dependency handling

This should hopefully be the last fix for the merged emotion generic player.

Revert "elput: Use Eeze function to get the udev to create context with"

This reverts commit 7b3d20371e8095207c4bafb3684b8a30d4ca6895.

using a separate context was intentional here since libudev is not threadsafe
and this will crash

Revert "elput: add missing dependency on libudev"

A better fix to not use libudev functions has been pushed previously,
so there is no need to add an extra dependency on libudev as we use
Eeze already.

This reverts commit 9d429a7c1ff059e2e4da961af73a6aecec3b5b98.

elput: Use Eeze function to get the udev to create context with

As we already link to Eeze and use it throughout this code, there is
no point in adding an extra dependency on libudev itself when we can
just use Eeze functions to get what we need. This is a better fix for
T3712

@fix

Signed-off-by: Chris Michael <cpmichael@osg.samsung.com>

build: use internal dependency for eina and ecore for generic players

On a system without efl installed these two packages would not be found as.
Need to be internal now that we merged them inside efl.

elementary: add missing dependency on libwayland-client in elm_cnp

lib/elementary/.libs/lib_elementary_libelementary_la-elm_cnp.o: In function `wl_data_offer_accept':
/usr/include/wayland-client-protocol.h:893: undefined reference to `wl_proxy_marshal'
/usr/include/wayland-client-protocol.h:893: undefined reference to `wl_proxy_marshal'
collect2: error: ld returned 1 exit status

Based on a patch by aerodynamik. Thanks!

Fixes T3398

elput: add missing dependency on libudev

Elput is using udev directly so we need to amke sure we link against it.

lib/elput/.libs/lib_elput_libelput_la-elput_input.o: In function `_elput_input_init_thread':
/<<PKGBUILDDIR>>/src/lib/elput/elput_input.c:317: undefined reference to `udev_new'
/<<PKGBUILDDIR>>/src/lib/elput/elput_input.c:326: undefined reference to `udev_unref'

Patch by aerodynamik. Thanks!

Fixes T3712

evas generic svg loader - dotn continue if rsvg handle is null

should avoid complaints of null rsvg handles

efl - build again after adding generic players

eolian: utilize the new void_ptr builtin across eo files

This lets me narrow down the remaining cases of pointers across the EFL.
The void pointers will later need to be reevaluated on per-case basis and
replaced appropriately where possible/feasible.

edje_cc: improve code readability when parsing external params

Reviewers: Jaehyun_Cho

Reviewed By: Jaehyun_Cho

Subscribers: cedric, jpeg

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