Efl.Ui.Flip: Fix compilation outside EFL

After a previous patch, compilation of apps would fail
as the include order was invalid. Efl_Ui_Flip_Mode was
defined after the typedef using it unless EO APIs were
enabled.

elm: Don't install private headers

Thanks tom for the report.

efl_ui_flip: renamed elm_flip to efl_ui_flip and updated the eo class

Reviewers: woohyun, raster, jpeg, cedric

Reviewed By: jpeg, cedric

Subscribers: cedric, jpeg

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

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

elementary/colorselector: use layout instead of edje object for components.

Summary:
Issue: As edje object is used instead of layout for components mode,  the content widgets like buttons are added as
subobject of colorselector hence when we unset the edje object on mode change the buttons still remain subobject of colorselector causing issues when all subobjects of colorselector is queried(sub objects are queried during atspi mode).
Soln: Use layout instead of edje object.

Signed-off-by: Shilpa Singh <shilpa.singh@samsung.com>
@fix

Test Plan: Change the mode to Palette mode and access the children of colorselector.

Reviewers: Hermet, cedric

Subscribers: govi, rajeshps, kimcinoo, cedric, jpeg

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

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

elementary/colorselector: colornames updated with spaces.

Summary:
Colornames updated with spaces so that any TTS engine
can read them out in an understandable manner.
Signed-off-by: Shilpa Singh <shilpa.singh@samsung.com>
@fix

Test Plan:
Enable ATSPI mode
Tap colors and listen to colorname's

Reviewers: cedric

Subscribers: rajeshps, jpeg

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

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

elementary/tooltip: adjust the coordinate if tooltip was located out of screen

Summary:
When reconfigure the tooltip as orient, the tooltip can located out of screen.
This patch will adjust the coordinate so that can be seen whole tooltip.

Signed-off-by: Minkyu Kang <mk7.kang@samsung.com>
Reviewers: cedric, Hermet

Subscribers: seoz, jpeg

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

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

evas: fix typo in description of the engine functions.

Reviewers: Hermet, raster, cedric

Reviewed By: cedric

Subscribers: jpeg

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

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

evas: fix order binding attributes to gl.

Summary:
In same cases(devices) can lead to mix up attributes in shaders.
@fix

Reviewers: cedric, Hermet, raster

Subscribers: jpeg

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

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

evas: button_mask as unsigned, UB fixes.

Summary:
* pointer.button is DATA32 which is unsigned, so this changes the
  definition of pointer_button_down_mask accordingly.

* Avoids UB in mask generation:
lib/evas/canvas/evas_events.c:1348:37: runtime error: left shift of 1 by 31 places cannot be represented in type 'int'

Subscribers: cedric, jpeg

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

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

ecore: make sure lifecycle of Efl.Loop.Fd is correct.

ecore: properly reset fd handler when parent get deleted.

elementary: Fix elm_code theme lookup and use correct naming

ecore: add a tests that ensure we do implement lifecycle correctly.

ecore: migrate ecore_timer lifecycle to rely cleanly on Eo lifecycle.

eina: ensure resource destruction on failure

In case of failure within eina_lock_new() (posix), a
pthread_mutexattr_t would have been left in an initialized state,
without any deinitialization being called.

Consequences would have been implementation defined.

ecore-xcb: Fix XCB to be usable again

Summary:
Well mostly, it seems there is an issue with multi-key events and
enlightenment. Let's merge this first before opening a ticket.

Reviewers: devilhorns

Subscribers: cedric, jpeg

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

@fix

elementary: export symbol used by tests suite.

eina_thread.c: fix include guard.

DragonFlyBSD has pthread_setname_np but no pthread_setaffinity_np;
we still need to include pthread_np though.

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

eio: fix pointer comparison.

Fix a warning on dragonfly.

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

ecore_audio: uncondionally include libgen.h.

Basically everyone should have it.  If not, maybe we should consider
to test the value of _POSIX_C_SOURCE before testing platform specific
defines, or adding a feature check.

Fixes the following warning on DragonFlyBSD:

lib/ecore_audio/ecore_audio_obj_out_pulse.c:282:60: warning: implicit declaration of function 'basename' [-Wimplicit-function-declaration]
        class_vars.context = pa_context_new(class_vars.api, basename(argv[0]));
                                                            ^
lib/ecore_audio/ecore_audio_obj_out_pulse.c:282:60: warning: passing argument 2 of 'pa_context_new' makes pointer from integer without a cast [-Wint-conversion]
In file included from /usr/local/include/pulse/pulseaudio.h:29:0,
                 from lib/ecore_audio/ecore_audio_private.h:17,
                 from lib/ecore_audio/ecore_audio_obj_out_pulse.c:18:
/usr/local/include/pulse/context.h:172:13: note: expected 'const char *' but argument is of type 'int'
 pa_context *pa_context_new(pa_mainloop_api *mainloop, const char *name);
             ^

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

ecore_time.c: do not mess with the representation of clockid_t.

Clockid_t should be used as an opaque type. Some platform might want
to (and even do, e.g. DragonFlyBSD) declare clockid_t as an unsigned.

On such platform, testing the sign of clockid_t is never false, and
assigning it a negative value is an UB, which makes this code unlikely to
work as intended. Fixes black window on dragonfly!

Thanks to gcc for spotting this.

  CC       lib/ecore/lib_ecore_libecore_la-ecore_time.lo
  In file included from ../src/lib/eina/Eina.h:215:0,
                   from lib/ecore/Ecore.h:304,
                   from lib/ecore/ecore_time.c:18:
                   lib/ecore/ecore_time.c: In function 'ecore_time_get':

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

ecore_signal.c: factoring out the platform test to improve readability.

Turns out there is no PRI?SIGATOMIC in the C99 standard. Work around
that by deducing the effective integer type by comparing the
SIG_ATOMIC_MAX with integers *MAX.

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

elput: Fix crash whem sd_session_get_seat fails

If systemd fails to give us a valid session id, then any further calls
to get a seat from that session are going to fail. As such, check for
a valid seat return from systemd before calling any systemd functions
which require session id.

Fixes T3785

@fix

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

eio: remove eio_model that should already be part of Eio.h.

edje_edit: fix map API (light, perspective, rotation)

it should be possible to set light or perspective to NULL
and return -1 when nothing is set
also ability to send NULL point to store rotation

@fix

elementary: fix definition of external symbols

Introduced by elm_code. Broke the build (link) Mac OS X.

eolian: we don't support unnamed structs/enums, so don't check

eolian: remove the c_type_named_get APIs

These have inconsistent behavior and are generally unnecessary, so I'm
removing them. Use the c_type_get funcs instead.

eolian: add a new references system to help replace pointers

It's now possible to mark struct fields and function params as "references",
which causes them to become pointers in C (in bindings, they become whatever
is necessary). They're not a part of the type and are much more restricted
than pointers, allowing bindings to be easier. This system will be gradually
utilized and expanded as required.

@feature

examples: eio: remove unused variable

examples: eio: mark unused parameter as such

Really shutting up warnings here to see what else the compiler has to say.

tests: evas: remove unused variable

We set it but never going to use it afterwards. We already tests the
efl_gfx_buffer_colorspace_get() elsewhere in this tests so we can ignore it
for now until we need it and bring it back together with a user.

elementary_code: Fix test failures post-merge

Looks like some were legitimate failures that were not being caught
in the previous setup

elementary_code: Use correct theme file post-merge

Merge elm code, a code editing widget into the efl

It was decided to merge this and mark the code as BETA for now.
It's currently being used by EDI and should also be used by enventor and etc.

benchmark: remove unused variable

Looks like this was forgotten during some re-factoring of the code.

benchmark: mark parameter as unused to ignore compile warning

eolian_cxx: add missing libgen.h include for basename() to fix build on OS X

On Linux this went through and has gone unnoticed. basename() needs libgen.h
though and eolian_cxx was not including it.

edje_edit: return lost API *_span_row/col_set declaration

I'm lost this declaration while add API for manager part items by
index and mark as deprecated old funcs

eio: Fix leaks in tests

elm: Add support for Efl.Model in Elm.Fileselector

Elm.Interface.Fileselector now supports Efl.Model objects, allowing users
to work with paths from different data models.

Example of model attribution:

ELm_Fileselector *fs = eo_add(EFL_FILESELECTOR_CLASS, NULL);
Efl_Model *model = ...;
elm_interface_fileselector_model_set(fs, model, NULL);

eo-cxx: Remove commented headers

eolian-cxx: Fix compilation error on clang with const UDT initialization

eolian-cxx: fix compilation errors an olders gcc

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.