Efl: Introduce app-side min size hint

Now called "hint_min", not sure if it's the proper name for it.
At first I wanted to reuse the request size hint instead of
adding a new hint, but doing that would break Terminology
or any app that already used size_hint_request.

One problem with hint_request is that the legacy function
already exists but its support is practically not implemented.

@feature

Edje: Fix some compilation warnings

Efl: Add geometry property (combines position and size)

It's a helper function that can also in the future be
used to implement atomic move+resize operations.

Efl: rename size hint content_min to restricted_min

Still not sure about which keyword is the best to use here,
but anyway, it's a protected function so users should realize
it's probably not what they want.

Efl.Ui.Box: Simplify layout code with combined_min

Simplify the code and fix an issue.

Efl: Introduce combine_min size hint

It combines the content min size and the requested min size
by taking the MAX value. This will help support request size
without major code changes.

edje: cached the vg tree generation and fixed the interpolation betwwen two vg tree.

Reviewers: cedric, jpeg

Subscribers: cedric, jpeg

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

edje : fix the typo in svg loader.

Reviewers: SanghyeonLee, cedric, jpeg

Reviewed By: jpeg

Subscribers: cedric, jpeg

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

edje: unamed part are completely legit as long as no program try to interact with them.

elmentary: trigger selected event when mouse is up in index.

Summary:
- selected callback should be called when the user releases a mouse button
  and selects an item.

Reviewers: cedric, jpeg, Hermet, woohyun

Subscribers: conr2d, cedric, jpeg

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

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

evas: fix possible memory leaks in e3d_drawable_texture_rendered_pixels_get

Summary:
@fix
CID: 1356243, 1356242, 1356242, 1356242

Reviewers: cedric, Hermet, raster

Subscribers: jpeg

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

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

elementary: add implementation for elm_multibuttonentry_item_selected_get API

Summary:
Added definition for item_selected_get API
Signed-off-by: Shilpa Singh <shilpa.singh@samsung.com>
@fix

Test Plan: select an item, call item_selected_get API to check status of the item

Reviewers: cedric, CHAN

Reviewed By: CHAN

Subscribers: CHAN, rajeshps, cedric, jpeg

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

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

elementary: cleanup event handler setup for hoversel.

elementary: rearrange hoversel when its parent is resized

Summary:
Hoversel will be rearranged when its parent is resized.
This patch was written by @godlytalias on tizen side.

Reviewers: cedric

Subscribers: jpeg, godlytalias

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

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

edje: add new svg part to edc

Reviewers: Hermet, cedric

Reviewed By: cedric

Subscribers: jpeg, cedric, Hermet

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

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

edje: little bit of refactoring, more needed.

edje: add svg support to parse and store a svg file as a vector object in .edj.

Summary:
This is the svg loader which will parse the svg file and store it in a intermediate structure.
It provides the helper function to get EET descriptor to save the structure into eet file.

NOTE: Not all svg attributes are handled yet. but most common attributes are handled .

Reviewers: Hermet, cedric

Subscribers: Hermet, cedric, jpeg

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

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

edje_edit: add missed part and description initializations

@fix

eolian-cxx: Fix some C++ quirks that makes clang warn about

eolian-cxx: Fix generation of some primitive types

Fix translation of primitive types for C++ binding

eolian: refine the ref system to suit more cases

Now references are first class (but still restricted to one level). Unlike
pointers they only mark the type instead of introducing a whole new type.

elput: Fix build of tests out of tree

elm video: implement player interface

Reviewers: raster, cedric, jpeg, singh.amitesh

Reviewed By: singh.amitesh

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

Efl: Add screen_ prefix to all Efl.Screen apis

This will resolve conflicts between Gfx.size and Screen.size.

Efl: Add hint_ prefix to all size hints

This covers:
- base
- step
- aspect

The prefix hint_ is useful for bindings, and does not change
the generated C API.

Efl: Move evas size hints to Efl.Gfx.Size.Hint

This includes:
- align
- min (now content_min)
- request
- max
- padding (now margin)
- weight

This does NOT include:
- display mode (compress, expand... a hint used by naviframe only).

win: Implement aspect control with Efl.Gfx.Size.Hint

This changes the API from a double to int, int.
Still not sure which API variant is the best (int,int or double).

win: Remove inexisting EAPI definitions from legacy

Evas: Move aspect ratio to Efl.Gfx.Size.Hint

Problem:
- edje aspect ratio is defined by 1 enum and 2 double (min, max)
- window aspect ratio is defined by only 1 double
- evas object aspect ratio is defined by 1 enum and 2 ints (w, h)

Which one is the best interface? Are min/max a better option?

Also, not sure how to call the enum...

win: Forward render_post events from evas to window

To do: optimize event forward when no one is listening.

Evas: EO-ify Evas_Event_Render_Post

As Efl.Gfx.Event.Render_Post.
To be implemented by Efl.Ui.Window

efl_ui_win: Disable legacy for name and type set

Those are only valid during eo_add, before finalize.
Both of these are constructors.
Add again jp's code.

efl_ui_win: Add back title_set as text.set

add jp's code again.

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