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

efl-wl: no need to check exe tag on app del, just remove pid always

efl-wl: add functions for managing allowed client pids

@feature

elm: fix typo - remove line to debug

elm: use Dbus securely

The Efl.Access.Attribute is using key and value.
The value could be NULL. If the value is NULL, then following error occurs.

*error:
arguments to dbus_message_iter_append_basic() were incorrect,
assertion "_dbus_check_is_valid_utf8 (*string_p)" failed
in file ../../dbus/dbus-message.c line 2712.
This is normally a bug in some application using the D-Bus library.
Array or variant type requires that type string be written, but end_dict_entry
was written.
The overall signature expected here was 'a{ss}' and we are on byte 3 of that
signature.

elm: fix for invalid attribute matching

Summary:
Fixes invalid attribute matching in at-spi's collection interface's GetMatches***
functions.

Reviewers: stanluk

Reviewed By: stanluk

Subscribers: lukasz.stanislawski, cedric

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

evil: final step : remove completely the binary, useless now that the unit test is theree

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

efl ui did - dont segv if createicon func is not provided

Use ERR instead of CRI if *DATA_GET* returns NULL

This patch set is for remains.

elm: Use ERR instead of CRI if *DATA_GET* returns NULL

eina tests: add missing semicolumn

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

evil: remove useless binary tests

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

eolian: no need to store 'parsed' table anymore either

eolian: simplify parsed checking

eolian: no need to track currently parsing file anymore

eolian: all dependency parsing is now deferred

efl_ui_focus_parent_provider_gen: code refactoring

Summary: remove needless function call and else statement

Reviewers: bu5hm4n

Subscribers: cedric, jpeg

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

efl_ui_focus_manager_sub: avoid a call to NULL

Summary:
we need to consider the manager change to NULL

@ref T6616

Reviewers: bu5hm4n

Subscribers: cedric

Maniphest Tasks: T6616

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

elm_menu: avoid a call to NULL

Summary: the provider_find can be called before parent_set in constructor.

Test Plan:
1. EINA_LOG_LEVELS=eo:2 elementary_test -to menu
2. check that there is no warning message about efl_provider_find

Reviewers: jpeg, bu5hm4n

Reviewed By: bu5hm4n

Subscribers: cedric

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

efl_ui_focus_composition: remove elements_flush

this can also be done when calling prepare_logical

Efl.ui.progressbar: remove duplicate call

eina_value_set() was already getting called

Efl.Ui.Progressbar: implement range min max

wl2_surface_dmabuf: Trim the buffer queue after a while

If we have more buffers than we need for 100 frames then drop the oldest.

This can happen if we're on a hardware plane and then removed from it, or
really whenever the compositor feels like holding onto a few frames.

Trimming the queue too soon could result in having to do a costly full
frame redraw, so we wait a while to make sure we don't need one again.

Having more frames than we need costs us a little every draw since we
always use the oldest available.  It also wastes memory.

ecore_wl2_surface: Flush surfaces for offscreen windows

When a surface leaves all outputs we can discard its buffers to save
memory.

Currently most compositors don't send leave events for iconify, so this
pretty much just saves us a cursor buffer under weston for now, but in
the future it could be used for freeing resources of offscreen (fully
occluded or iconified) windows.

wl2_surface_dmabuf: improve surface flush

We used to abandon all buffers even if they were locked. This can't
actually free them until they're all released by the compositor.

Instead just free any buffer the compositor doesn't have locked, so we can
still re-use them when they're released without needing a full redraw.

There's probably room for additional cleverness here.  If we get a new
frame event before the buffer release we may want to keep it, and if we
get the release first we may want to abandon it.

ecore_wl2: Add new event for windows not displayed on any output

When a window leaves all outputs we can free its render buffers to save
memory.  This new event is generated when a window leaves all outputs.

ecore_wl2: Track outputs a surface is present on

Keep a list of Ecore_Wl2_Output * a surface is present on.

ecore_wl2: Add internal _ecore_wl2_output_find

This finds the Ecore_Wl2_Output * for a given struct wl_output *

ecore: add function for prepending an event handler

@feature

eolian: deferred parsing of inherits + better static checks

This change finally introduces deferred parsing of inherits to
Eolian, after a long time and many iterations. That means instead
of parsing inherits directly as part of the main file's parse pass,
they're pushed into a queue and parsed later. The validation engine
was modified to properly fill in the remaining info afterwards.

This may introduce breakages but I haven't noticed any. It also
properly unlocks cyclic dependency support in Eolian.

Additionally, this also introduces checks for duplicates in class
inherits (class Foo(Bar, Bar) is no longer possible) and it adds
stricter name checks, so you can no longer have a class Foo.Bar
and refer to it as Foo_Bar in implements. This was actually never
meant to be supported, but the check was previously broken.

@feature

eolian: move impl/ctor fill to validation stage

Eina test: delete tempory file on Windows in eina_test_file_mktemp

Evil: remove "symlink" code (it was for .lnk files anyway...)

Eio: disable the symlink code on Windows

Efreet_Mime: update comment about symlinks on Windows

Eina: ignore EINA_FILE_LNK on Windows

Ecore_File: remove symlinks support on Windows (library and test)

Windows: remove symlink() usage in edje_decc and evil binaries

eina: Cleanup grammar in eina_hash doxygen

Summary:
Corrects some grammatical errors, and rephrases wording of some passages
for better clarity.  Also fix a few doxygen formatting inconsistencies.

Subscribers: cedric, jpeg

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

edje_object: fix typo in EAPI edje_object_color_class_description_get

Reviewers: woohyun, jpeg, cedric, conr2d

Reviewed By: jpeg

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

elm_test: add elementary_test for Efl.Ui.Button

Test Plan: Run elementary_test->Efl.Ui.Button

Reviewers: jpeg, cedric, Jaehyun_Cho

Reviewed By: jpeg

Subscribers: jsuya

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

Efl.Ui.Image: remove previous image when NULL is set

Summary:
when Efl.Ui.Image has an image,
         efl_file_set(efl_added, NULL, NULL) is not working.
         I think this should remove prevous image and go back to empty image.
         @fix

Reviewers: jpeg, cedric, eunue, woohyun, Jaehyun_Cho

Subscribers: Blackmole

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

elm_index: fix documentation of wrong param name

Reviewers: jpeg, woohyun, eunue, conr2d, Jaehyun_Cho

Reviewed By: Jaehyun_Cho

Subscribers: cedric

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

efl_ui_bg_widget_legacy: ignore ELM_SCALE

Summary:
For legacy efl_ui_bg_widget or elm_bg, ELM_SCALE is ignored
when calculating internal image size.

Test Plan:
compare elementary_test->bg option with
ELM_SCALE=2.0 elementary_test->bg option

Reviewers: woohyun, jpeg, Jaehyun_Cho

Reviewed By: Jaehyun_Cho

Subscribers: cedric, id213sin

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

elm: make elm_layout_text_set use efl_text_markup_set

The elm_layout_text_get is using efl_text_markup_get by following commit.

    commit c07a40c745c5df1b1f6f0bbf666b233d8d072ca7
    elm: make elm_object_text_get return markup info as well.

    This commit solves following issue

    https://phab.enlightenment.org/T6642

    If I set object text as below
    elm_object_text_set(btn, "Some<br>text");
    then elm_object_text_get(btn) returns "Some text" not "Some<br>text".

So using efl_text_markup_set makes sense.

edje_cc: check HAVE_SYS_RESOURCE_H presence for rlimit usage

fix T6645

efl_ui_focus_manager_calc: keep the entry for the redirect seperated

logicals on the focus stack can cause there a lot of confusion.

This fixes a missed focus out event from the entry in elementary_test

ecore_wl2: Make surface managers modular

This allows loading modules to handle wayland surfaces, and makes the
existing dmabuf manager a module.

eina_module: Drop ridiculous ERR messages from module_list_load/unload

The documentation for these functions claims that passing a NULL array
results in doing nothing - that should also include logging nothing.

EINA_SAFETY_ON_NULL_RETURN() logs an ERR message and should be reserved
for usage when NULL is not actually a valid state.

Additionally, it's entirely possible to turn off EINA_SAFETY_CHECKS, at
which point these functions would stop behaving as the documentation
says they do.  Not great.

ecore_wl2: Use wl_buffer_get in dmabuf_surface_post

This is the final place where this code was reliant on internal structures,
it now solely uses (beta) public api.

ecore_wl2: fix ecore_wl2_buffer_wl_buffer_get

It was currently only used internally and had the side effect of
creating a new buffer instead of just returning the existing one.

Now it's useful to external callers, as it only returns the existing
wl_buffer and has no freaky side effects.