Elementary: remove needless structs in Efl_Ui_List

elementary: efl_ui_list change SegArray tree to Efl.Object

elementary: efl_ui_list model_set update values fixed

elementary: efl_ui_list_segarray fixed remove last block item

elementary: efl_ui_list focus manager fix
fixed focus changed with keyboard
fixed scroll in focus
fixed focus in example

ecore_wl2: Take a library reference on surface create

Since surface handling is now done via modules, we need to ensure
the library can't be shutdown while a surface exists.  Otherwise,
we get a segfault trying to call a function we've unmapped.

Fixes a bug on shutdown for some wayland clients using software
rendering.

efl_ui_focus_manager_sub: remove needless call

Summary: focus manager is not assigned in constructor

Reviewers: bu5hm4n

Reviewed By: bu5hm4n

Subscribers: cedric

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

elm_hover: remove focus_layer related code

Summary:
because elm_hover implements Efl.Ui.Focus.Layer, focus_layer related code
dosen't needed.

Test Plan: elementary_test - hover

Reviewers: bu5hm4n

Reviewed By: bu5hm4n

Subscribers: cedric, jpeg

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

efl - fix build again after removal of install of public header

put elm_code_widget_legacy.eo.h back in the includes to install as it
is required by other public headers...

test_ui_table: fix uninitialized variable warning

maybe right, maybe wrong, maybe compile warnings shouldn't be added
in reviewed patches

ref D5668

gl_drm: NULL check img

If we set native surface on a new object we can get here with NULL
engine_info from the caller.  We should still create a native surface.

elementary: fix legacy widget type name for backward compat

Summary:
For example, the widget type of elm_button was "Elm_Button".
But, the object which is created by elm_button_add() will
return its widget type "Efl.Ui.Button_Legacy".
It is not legacy name. It should be fixed to return "Elm_Button".

I don't know when but eolian start to make class name with ".".
So, it should be converted to "_" for all widgets.

@fix

Test Plan:
All test cases are included in this patch.
Run "make check"

Reviewers: raster, cedric, jpeg, taxi2se

Reviewed By: cedric

Subscribers: taxi2se, woohyun

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

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

eina: improve documentation for eina_bezier.

Summary:
Adds missing @param docs and fixes an incorrectly documented one.
Clarifies difference between 'length' and 'position', specifying the
latter is a number between 0.0 and 1.0.  Improves verbage here and there
for grammatical correctness and internal consistency.

Subscribers: cedric

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

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

eina: make doxygen @notes consistent in eina_hash

Summary:
These routines all have slight permutations of the same basic note, but
are each formatted a bit differently.  Fix that and a few punctuation
irregularities.

Similarly for a couple @warnings, and escalate one @note to a @warning
since what it describes might be a security issue so deserves
highlighting.

Subscribers: cedric

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

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

eina: fix minor spelling mistake in eina-private.h doxygen

Reviewers: cedric

Reviewed By: cedric

Subscribers: cedric

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

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

eina: use matrix compose in place of equivalent code in matrix multiply

Summary:
The eina_matrix3_compose and eina_matrix3_multiply API's are
mathematically identical (even though the implementations are
reversed... weird), except that the latter also includes a fastpath for
identity matrices.

Having two functionally equivalent APIs is redundant, so ideally one or
the other would be dropped.  But in order avoid API breakage, just have
one routine wrapper the other and eliminate the internal redundancy.

(Note that the parameter signatures of the two routines are different -
eina_matrix3_compose() takes the two input matrices first, and the
output matrix last, while eina_matrix3_multiply() takes the parameters
in the reverse order.  This inconsistency in the API style could result
in accidentally erroneous usage and would be an argument for deprecation
of one of the two APIs.)

Signed-off-by: Bryce Harrington <bryce@osg.samsung.com>
Reviewers: cedric

Reviewed By: cedric

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

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

ecore_file: modify check for wd in file monitor del

Summary:
Make checks consistent in ecore_file_monitor_backend_del.
inotify_add_watch can return 0 as well on success.
So, even if wd is 0, inotify_rm_watch need to be invoked.

Reviewers: cedric, jpeg

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

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

eina: cleanup doxygen for eina_rectangle.

Summary:
Some spelling/punctuation/grammar/formatting fixes to make the
rectangles doxygen more consistent.  Simplify wording in several places,
including removing redundant documentation of return values.  Reword the
rectangle cutting API's to be more clearly worded.  Make clearer mention
of parameters that get changed by the function call, and ones that
allocate memory.

Reviewers: cedric

Reviewed By: cedric

Subscribers: cedric

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

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

evas/vg: reset the path data in destructor.

Summary: As the gfx_path mixin class destructor is not called hence explicit call of reset.

Reviewers: jpeg, SanghyeonLee

Subscribers: cedric

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

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

efl: general distcheck fixes

Summary:
- Added missing C++ header
- Added missing elementary header
- Removed generated header from elementary_SOURCES
  (Was added by raster in 42dfee37)
  Not sure of what would be the best place for it, though.
- Removed previously removed files from elementary examples Makefile.

Test Plan: Run 'make distcheck'

Reviewers: stefan, felipealmeida

Subscribers: cedric

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

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

efl.ui.layout: change signal name for icon/text

Summary:
when set/unset icon and text,
signal "elm,state,[part],visible/hidden" is emitted.
This is wrong because visible/hidden should be handled by
Efl.Gfx.visible, not Efl.Text nor Efl.Content.
This should be changed into elm,state,[part],set/unset"
All relating edc should be fixed.

Test Plan: run elementary_test->button, Efl.Ui.Button

Reviewers: jpeg, cedric, woohyun, Jaehyun_Cho

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

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

efl_wl: Update native surface for dmabuf test too

Should've added this to the last commit - though this change is purely
cosmetic as the test can never trigger direct scanout.

efl_wl: Update native surface usage

We should be zeroing out the dmabuf scanout stuff.

efl-wl: check for existing keyboard enter before sending leave

ref 4a32e2456eaca80cee0b1ce70cea00cb41e11979

ecore_wl2_surface_dmabuf: fix memory leak on surface destroy

We need to free the surface private data on destroy.

ecore_wl2: Fix memory leak on shutdown

The output list wasn't properly cleaned up on shutdown/disconnect.

efl_selection_manager: shut up shut up shut up ecore-x when running in wayland

fix T6643

efl_gesture_manager: free hash in destructor

every canvas object has had this leak for some time:

==25217== 64 bytes in 1 blocks are definitely lost in loss record 1,669 of 3,515
==25217==    at 0x4C2FB6B: malloc (vg_replace_malloc.c:299)
==25217==    by 0x688D839: eina_hash_new (eina_hash.c:755)
==25217==    by 0x688D940: eina_hash_string_superfast_new (eina_hash.c:791)
==25217==    by 0x6265F8C: _efl_gesture_manager_efl_object_constructor (efl_gesture_manager.c:66)

eolian: properly initialize warned (to avoid random parse failures)

elua: add eina in environment variable library path

In-tree libeina.so library path is missing while running elua.

http://autobuild.buildroot.net/results/2bb/2bbe89694c206d764d51a5dd9c75bc4aa844e98d/build-end.log

@ref T6670

Cc: Daniel Kolesa <d.kolesa@samsung.com>
Signed-off-by: Daniel Kolesa <d.kolesa@osg.samsung.com>

evas drm engines: Fix variable type for output

We already include the Ecore_Drm2 header for these engines, so there
is no need for the 'output' field to be a void pointer here.

@fix

Signed-off-by: Chris Michael <cp.michael@samsung.com>

ecore-evas-drm: Use output viewport in ecore_evas_animator_tick calls

@fix

Signed-off-by: Chris Michael <cp.michael@samsung.com>

ecore-drm2: Add API functions to get/set output relative mode

These API functions will be used inside Enlightenment in order to
determine the proper extended screen size based on outputs relative
position.

@feature

Signed-off-by: Chris Michael <cp.michael@samsung.com>

ecore-evas-drm: Include Evas_Internal header

We need to include this header so that we can use the new
efl_canvas_output functions

Signed-off-by: Chris Michael <cp.michael@samsung.com>

efl_ui_nstate: make "changed" callback is called properly

- "changed" callback is not called when theme_apply is called
- elm_check_state_set does not call "changed" callback to keep
  backward compatibility

Revert "elm: do not call "changed" callback when setting style"

This reverts commit b1ee35e35ce3132566244abbbd3a804167ef9c00.

Revert "Efl.Ui.Nstate: do not call "changed" callback"

This reverts commit fd8e9f9c4e8dbda6c379dc464f562f5b2c681d31.

eolian: print all duplicate warnings before aborting build

Instead of aborting at the first warning, accumulate the warnings
and stop at the end of validation.

elm focus ... set focus in focus in if focus is not already on something

let's actually set focus in focus_in on window (ecore evas) IF focus
is not set. this seems to make terminology still work...

efl ui win - if job runs after destruction of evas so dont flush pending

elm - dont get top widget for non-ui objects

efl ui win - fix focus break for terminology (at least)

at least terminology was broken by this. perhaps other efl apps. if an
efl app has decided to manually play with focus itself, like set focus
to some obj then leav ie, this line breaks that and interferes with
the object focused whenever the window gets focus set. if there is a
focus manager at least this is a new thing and so let the focus
manager set focus, but dont set old evas focus.

this fixes the break added in 56beb861e83cbc830c82db7473e0a88461e2e213

evas_vg: implement interpolate from Efl.Gfx.Path

interfaces: merge all bounds_get methods into efl.gfx.path

ref 884fc35160152712decdf7fd43082ef3912c0856

evas3d: use a type setter function with finalize instead of a custom constructor

resolves a conflict with the 'constructor' method name

evas3d: add "unknown" node type for use with errors

interfaces: remove Efl.Text.Properties

this is provided in Efl.Text.Font

efl_text_font: copy docs from efl_text_properties

efl_vg: use efl.object's name property instead of implementing a new one

ector: rename ector_renderer "bounds_get" method to "boundary_get"

this conflicted with a method from Efl.Gfx.Path

ector: fix property/method name conflict for "fill"

Shape implements a property, sw/gl/cairo implements a method

rename method to op_fill

eolian: convert all of cedric's tabs to spaces in .eo files

eolian: stop build with EOLIAN_WARN_FUNC_DUPLICATES > 1

ecore_wl2: Send window hide in response to xdg popup done

Clients should be sent this event when the compositor sends popup done.

efl_ui_win: move inital focus set to focus_in handler

it seems that focus changes to FOCUS=FALSE are causing autodel windows
to kill themself, so we only set the focus on the window if the window
manager calls focus in AND we dont have anything to focus and nothing is
focused yet.

edje - entry - fix empty item handling

tjhis should fix empty items which might have a null item string...
deal with it

fixes T6668

@fix

eina - tiler -= return true if rect alreadt there as its not a failure

docs say return true on succesas, false on failure. adding a rect we
already added is not a failur. it's an optimization to a NOP. so fix.
this was brought up by and fixes T6669 ... but in the opposite way.

elm - fix new em_config func prototypes in header to have real param decl

efl ui format - fix format checking to be correct

the format string checking was just ... wrong. it didnt' handle %%. it
didn't handle constant strings with no format in them... now it does.

fixes T6697

@fix

elm progressbar - fix val set

set initial min and max to 0 and 1 respectively so val set works at
the start...

fixes T6697

@fix

elm - make progress test code a little more solid and nice to read

use sizeof the buffer rather than constant in snprint and 0.5 rather
than .50 which almost looks like a typo... :) also use correct escape
for % (%%%% so it becoems %% which is correct for a fmt string yto
produce %)

elm: Fix double shutdowns of ecore_x and ecore_wl2

This is now done elsewhere.  wayland clients were ERRing on shutdown.

ref 9cf61c2f6d215e3adfa1567154a229eb49116769

wayland_imf: Clear hide timer when deleting context

Otherwise the callback can fire after the module is unloaded leading to some
very entertaining to debug crashes at shutdown.

Elm Progressbar test: Add in examples of efl regressions to progressbar.

ecore_wl2: Destroy offscreen event handler in surface destroy

Or we can get the event after destroying the surface and crash.

efl_mono: Distcheck fixes

- Export required sources
- Avoid generated sources being passed as static ones

eina_mono: Avoid calling eina_init directly from the modules.

They must be initialized from the eina.Config.Init() method.

ci: make sure we fail the build and pass on the non zero exit code

So far we just kept going on when one of the make targets failed with
the next one.

efl_part: call Efl.Text of each class' part

Summary:
Macro ELM_PART_TEXT_DEFAULT_IMPLEMENT calls
super class' Efl.Text interface, rather than this class.

Reviewers: jpeg, woohyun, Jaehyun_Cho, herdsman

Subscribers: cedric, id213sin

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

efl_part: use efl_ui_widget's default_part_get

Summary:
efl_part macros are using each widget's internally defined
default_part_get() functions to get default part name.
This might potentially cause errors when future widgets
inherits the widget but not overriding Efl.Text.text and
Efl.Content.content.

Reviewers: jpeg, cedric, woohyun, Jaehyun_Cho

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

ector: Updated the ector_buffer_pixels_set() api with stride info

Reviewers: jypark, jpeg

Subscribers: cedric

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

elm: remove ecore-x init from quicklaunch post-fork

this is invalid for basically every possible case. allow components
to naturally call this function as needed

@fix

elm: call ecore_app_args_set() after quicklaunch fork

ensure that apps get expected behavior when calling args_get()

@fix

ecore-imf: do ecore-x init during ctx creation instead of module init

this avoids a number of issues with efl under wayland, msot notably the
automatic triggering of xwayland during every app startup

@fix

elm: move ecore-x and ecore-wl2 connection init to selection manager

this seems to be the only place where the related components are
explicitly used

neither of these components have fork-safe connections, so there is no
benefit to calling them during quicklaunch init

elm: print error if setsid() fails during quicklaunch fork

eio: remove broken pid-based monitor reset copied from ecore-file

this has no technical basis for existing and should be handled by
individual monitor backends

@fix

ecore: fix event flushing to not erase+leak events

stealing the message data here prevents events which aren't being flushed from
ever being usable again and is unnecessary since the free callback will be
automatically called during the destructor

ref 5dd52fd09b7d79c70b3134423a87aa6400a2d994

Evas: remove depth usage in GDI and DirectDraw engines

Test Plan: compilation and elementary_test

Reviewers: raster

Reviewed By: raster

Subscribers: cedric, raster

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

elm_part_helper: Fix to use registered default text part

Efl.Ui.Nstate: do not call "changed" callback

Before solving following problem in Efl.Ui.Check

https://phab.enlightenment.org/T6673

The changed callback is called with opposite value,
if application is using Efl.Ui.Check with state pointer.

The reason is that the changed callback is called in
efl_ui_nstate_activate, and the value refered by state pointer
is changed only after the efl_ui_nstate_activate is called.

static void
_activate(Evas_Object *obj)
{
   EFL_UI_CHECK_DATA_GET(obj, sd);

   efl_ui_nstate_activate(obj);
   if (sd->statep) *sd->statep = efl_ui_nstate_value_get(obj);

   elm_genlist: fix rebase build error related with on_focus_update to Focus.Object

   widget: Move on_focus_update to Focus.Object

   Following @taxi2se's recommendation. This is indeed a focus method, and
   Widget already inherits from Focus.Object.

   Ping @bu5hm4n who probably wants to adapt this further.

   Ref T5363

   efl_spec: fix build error related with dmabuf of ecore_wl2

   ecore_wl2_tbmsurface: modify tbmsurface based on ecore_wl2_surface change of opensource

ecore-drm2: Ensure output changed event is sent for removed outputs

If an output gets disconnected, we would still like to be notified of
that. In the _outputs_update function, we mark an output as
disconnected by setting output->connected and output->enabled to
FALSE. With this line in place (in the output_event_send function) we
would never get notified if an output was disconnected.

@fix

Signed-off-by: Chris Michael <cp.michael@samsung.com>

disable async mode (use sync mode) for ibus when keymap changes

so scenario:

1. use ibus
2. have at least english input and japanese input (or korean etc.)
3. have 2 kbd layouts (english and greek).
4. enable "use system keyboard layout" in ibus advanced settings
5. switch to english input mode
6. switch to greek key layout
7. type and get english, not greek input as you should

@fix for both terminology and elm/efl entry/ytext input.

elm: do not call "changed" callback when setting style

efl.ui.progressbar: add checks for min & max value

this improves the validity check for min & max values
provided from user side.

efl_ui: Disable includes that don't compile

Efl.Ui.Text:
The EO file contains a lot of references to legacy Elm types, which are
defined in elm_general. They should be checked and moved over to
efl_ui.eot if necessary.

Efl.Ui.Multibuttonentry:
This class was originally supposed to be based on a Model Item but as of
now the API is still uncertain, so MBE itself hasn't been worked on
more. Disable this from EO-only apps until its API is fixed.

Ref T6666

eina_tiler: fix typo of rect comparing

eio: make inotify monitors fork-safe

@fix

eeze: simplify watch code and make it fork-safe

avoid reads from parent process's udev fds

ecore-file: make monitoring truly fork-safe

add a fork reset callback and attempt to preserve existing monitors
during reset

@fix

elm: call ecore_event_init() during init

events from this component are used internally in elm, don't rely on
other components to do it

@fix

ecore: make dbus-using modules fork-safe

reset dbus connections to ensure continued functionality

eldbus: make connections fork-safe

after a fork, any existing connection objects can no longer be used,
but it's up to the user to destroy them. internally, this prevents
existing connections from ever being returned as valid connections
and creates new ones after a fork

also destroy fd handlers for connections to ensure that no data is
accidentally clobbered before the connections are cleaned up

efreet: reset ipc connection after fork

ensure that existing connection is not reused

eldbus: do not require error return when adding/removing signal matches

this forces the functions to be synchronous and blocks (potentially forever)

quicklaunch: use XDG_RUNTIME_DIR instead of hardcoded /tmp

efl-wl: add function for passing ecore-exe flags to launched exes

@feature