elementary: switch to legacy API for tests.

edje: fix unused variable when gettext is disabled

ecore_cocoa: simplify clipboard functions naming

ecore_cocoa_selection_clipboard_xxxx() have been renamed into
ecore_cocoa_clipboard_xxxx() and they have nothing to do with
selection, only with clipboard.

Since the API is not stable yet, I can change it.

ecore_cocoa: move the default cursor definition out of the enum

This makes the code more explicit and less awkward.

ecore_cocoa: improve documentation

elementary: Remove set but unused variable

This variable looks like it may be used in the future, but for now
just causes additional compiler warnings so comment it out for now.

@fix

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

ecore-wl2: Support wl_seat_listener version 4

This patch adds support for wl_seat_listener version 4 (which is used
in wayland 1.11). We do not actually make use of the seat callback for
name yet, but when the rest of multi-seat issues are sorted, we may
need this.

@fix

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

Revert "elput: Missing pointer initialization."

Reverting this as the initialization of char *s was already set to
NULL in a previous patch, so the 'if (s)' check here is pointless
really as 'man 3 free' says passing NULL to free() is ok.

This reverts commit ad2e21b02399c02a748bb8b8ab0dc10f85e6662d.

edje_edit: text style should be able to set into NULL

@fix

efl_ui_win: rename elm_win to efl_ui_win

eolian: support @ref on returns

This adds basic support for @ref on return types.

edje_edit: fix copy&paste type for map functions which should return int

zplane and focal actually returns int, not Eina_Bool

@fix

elput: Missing pointer initialization.

Summary:
Calling free() on an uninitialized pointer leads to crash, that can occurs
when sd_session_get_seat fails.
Fixes T3785.

Reviewers: devilhorns

Subscribers: cedric, jpeg

Maniphest Tasks: T3785

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

@fix

Evas: Fix propagation of key events

UP/DOWN arrows were having effects on both focus and entry cursor
moves. This was due to an invalid propagation of the changes to
event_flags (on_hold).

Fixes T3845

examples: ecore: do not try to install files which are not available

The ecore_audio examples have been disabled and now do not compile right now.
But as they are disabled they never land in the tarball and thus we error out
with files not aviavle for install. Found by the OpenSUSE build service. Thanks
Simotek for reporting.

[ 2172s] /home/abuild/rpmbuild/BUILD/efl-1.17.99.47400/install-sh: ecore_audio_custom.c does not
exist.

examples: eio: update gitignore for new binaries

examples: evas: update gitignore for new binary

efl: Bump required wayland version

This makes our wayland libraries require newest wayland release of 1.11

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

elementary: Remove unused variable

Gcc says that 'model' is unused here, so comment it out

@fix

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

elm_win: Disable legacy for name and type set

Those are only valid during eo_add, before finalize.
Both of these are constructors.

Revert "Revert "Revert "Revert "elm ctxpopup: fix auto_hide mode to work correctly""""

This reverts commit 95620c6b60643a2326d73141e64447385c28e6e5.

Revert again, because this obviously fixes a bug...

Revert "Revert "Revert "elm ctxpopup: fix auto_hide mode to work correctly"""

This reverts commit 0907217be5cf68a377e0ceacf776535acdc38531.

append comment there.

Revert "Revert "elm ctxpopup: fix auto_hide mode to work correctly""

This reverts commit 5ddc7c36f9ed3b995b43c31eab3fdf4d27db74df.

Revert "elm ctxpopup: fix auto_hide mode to work correctly"

This reverts commit a428e1e7125f28e41532c659b9e31a8ff0bb5d11.

please don't accept patches if the discussion hasn't been finalizied between a submitter and the reviewers.

po: follow file rename for elm_flip to efl_ui_flip change

elm_win: Add back title_set as text.set

Title and name are two different things.
See 466d9fbaf1319db4df9d576bdbe196f2557f6.

elm ctxpopup: fix auto_hide mode to work correctly

@fix

Summary: fixx auto_hide mode to work correctly

Test Plan:
1. launch elementary_test - ctxpopup
2. click normal ctxpopup and auto_hide ctxpopup sample
3. check the auto_hide function whether it is work well or not

Reviewers: Jaehyun_Cho, raster, jpeg, cedric, singh.amitesh, DaveMDS, Hermet

Reviewed By: DaveMDS

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

pdf: Fix build with poppler >= 0.40

This disables PDF load if poppler >= 0.40 (current is 0.42 on
my machine). Those image loaders definitely should be explicitely
enabled or disabled, not enabled based on the system.

ecore_wl2: Fix uninitialized fields in function array

Ping @devilhorns

wl_shm: Fix uninitialized warning

warning: variable 'surf' is used uninitialized whenever 'if' condition
is true [-Wsometimes-uninitialized]

gl_drm: Fix shadow warning

eio: Fix build for windows

dllimport vs dllexport problem due to invalid include order.
Thanks @vtorri for the report

See 1ba06bb6429d6199714

tests: Fix previous patch (forgot to amend changes)

elm_test: Fix potential crash and garbage printf

gl_x11: Working GLES2 fallback.

Summary:
Disables GLES3 support when it cannot be determined that it is
available. Prevents to repeatly request a (missing) gles3 support and
fail.
Fixes T3795.

Reviewers: jpeg

Subscribers: cedric, jpeg

Maniphest Tasks: T3795

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

tests: Fix fake event test (avoid ERR)

An event object was deleted carelessly.

Evas: Fix compilation for Windows

Fixes T3783

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.