elc_hoversel: just little fix of expand/noitem behavior

when hoversel has no item at all and use clicks on it, then it goes into
state called "expanded".
and so then, no matter how many items user would try to add, hoversel won't work
anymore.

@fix

examples: update various gitignore files for new example binaries

These have been forgotten to be added recently.

atspi_image: improve existing documentation

Mixin and property descriptions.

atspi_widget_action: add mixin description

elm_scrollable: add mixin description

atspi_component: improve existing documentation

Ad more descriptions, consolidate property descriptions and fix smaller
problems.

atspi_action: improve existing documentation

Fix typos, add parameter descriptions and consolidate property describtions.

atspi_accessible: improve existing documentation

Make sure we describe whole properties and not getters and setters only. Adding
some more docs while being there.

eolian: add support for static and terminated arrays

Adds two new type types, STATIC_ARRAY and TERMINATED_ARRAY. Static arrays are
only allowed as struct members right now - they translate to regular C static
arrays (allowing them elsewhere wouldn't be good, as C isn't very good at
working with the size information). Terminated arrays are basically sequences
of data terminated at the end. The base type of static arrays can be any type
that is not marked ref (explicit ref may get allowed later). The base type of
terminated arrays has the same restriction plus that it has to be either
implicitly reference type (i.e. translating to pointer in C), integer type
or a character. In case of ref types, the terminator is NULL. In case of
integer types, the terminator is a zero. In case of character types, the
terminator is also a zero (null terminator like C strings).

@feature

ecore-wl2: Fix issue where hash would not get destroyed on error

Coverity reports that we could not reach the eina_hash_free here on
erorr condition. This was due to an erroneous return NULL above the if
(hash_create) check.

Fixes Coverity CID1357171

@fix

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

ecore-drm2: Don't leak return from _output_name_get function

Coverity reports that we end up leaking the return from
_output_name_get as the storage is never freed.

Fixes Coverity CID1357161

@fix

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

ecore-wl2: Fix issue of passing negative number to close() and read()

We should be checking the return value of ecore_main_fd_handler_fd_get
calls as they can return a negative number...which cannot be passed to
the close() or read() functions.

Fixes Coverity CID1357152 and CID1357153

@fix

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

ecore-wl2: Check the return value of ecore_main_fd_handler_fd_get

This patch fixes an issue where ecore_main_fd_handler_fd_get could be
returning a negative number and passing that to close() which cannot
accept negative numbers.

Fixes Coverity CID1357152

@fix

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

ecore-evas-wayland: Fix issue of passing wrong values to resize

Coverity detected that the same code was being passed to resize
regardless of canvas orientation. This patch fixes the issue by
passing the proper values to window_resize.

Fixes Coverity CID1357150

@fix

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

ecore-wl2: Fix derefence after null check

'window' here could potentially be null so we should be checking that
it is valid before trying to assign event window

Fixes Coverity CID1357148

@fix

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

elementary: Fix explicit null dereference

This patch fixes an issue reported by coverity where 'type' variable
could be null and passing null to strcmp is not good ;)

Fixes Coverity CID1357147

@fix

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

evas-wayland-shm: Remove logically dead code

Coverity reports that this bpp check is actuall dead code due to the
fact that bpp can never be < 0.

Fixes Coverity CID1357144

@fix

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

elput: Check return of read function call

If we fail to read any bytes from the fd, then we should be returning
a failure here.

Fixes Coverity CID1357143

@fix

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

ecore-evas: Fix error handling issues in wayland_shm engine

This patch fixes 2 Coverity issues where engine_info_set was being
called, but the return was never being checked.

Fixes Coverity CID1357141 and CID1357142

@fix

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

ecore-input: Check return fo ecore_main_fd_handler_fd_get

This patch fixes 2 coverity Error handling issues.
ecore_main_fd_handler_fd_get can return a negative fd on error so we
should be checking that return before trying to make use of the fd.

Fixes Coverity CID1356632 and CID1356624

@fix

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

edje: fix return type for part_text_cursor_content_get

Another spot that should've stayed "char *" and not "string". Also added the
missing 'own'.
Found due to a warning after ee36fdc2183a3de9569cac5ae74c0fb30d3d76db.

ector: add basic description to renderer mixins

ector: further document gl buffer base class

efl: gfx_shape: add some more documentation and improve existing one

efl: gfx_shape: document property itself and not the getters and setters

edje_edit: remove useless pointer grab as it conflicts with elementary combobox

Summary:
So currently having an issue related to clip_to function from
EdjeEdit API. I am not sure about how and why combobox use pointer grabbing
(just started to use it) but when setting NULL into clip_to of edje edit it
suddenly block entire screen away from mouse. Only when point is moved out of
window and back, only then it will be able to access widgets again.

Take a look at video and maybe try example to reproduce problem

I am not sure if deleting those functions from edje_edit is actually right
decision. But it will fix the problem of course.

Test Plan: See attached video and examples in attached files

Reviewers: NikaWhite, reutskiy.v.v, Hermet, cedric, raster

Subscribers: jpeg

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

Revert "elm: Fix wrong use of char instead of string and void out parameter"

This reverts commit 61f52a26fb5a512e77d3141e398182107b1cf752.

This fixes the reported break on T4000.

slider: fix edje warning for vertical style

Follow-up of D4117

slider: fix edje warning

Summary:
ERR<18878>:edje lib/edje/edje_calc.c:2767 _edje_part_recalc_single() file /usr/local/share/elementary/themes/default.edj, group elm/slider/horizontal/default has a non-fixed part 'elm.swallow.icon'. You should add 'fixed: 1 0'. But in order to optimize the edje calc, we add it automatically.
ERR<18878>:edje lib/edje/edje_calc.c:2767 _edje_part_recalc_single() file /usr/local/share/elementary/themes/default.edj, group elm/slider/horizontal/default has a non-fixed part 'elm.swallow.end'. You should add 'fixed: 1 0'. But in order to optimize the edje calc, we add it automatically.

Test Plan: Launch an app with slider, open the slider and observe the console output

Reviewers: cedric, singh.amitesh, jpeg

Subscribers: seoz, minkyu

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

elm_widget: Fix mouse wheel event propagation

The wrong type was propagated. Luckily it didn't crash.

Thanks @Hermet for the report.

edje: Lower ERR to INF for message about non fixed parts

This message really is just informational at best, since it's
not an error, and we even implemented an optimization.

It should be done at compile (edje_cc) time though.
See comments on D3801

evas object: Move display_mode to Efl.Gfx.Size.Hint

win: Remove font & image cache functions

Those actually belong to elm_config, or rather Efl.Config:
  efl_config_int_set("cache_image_cache_size", 42);
  efl_config_int_set("cache_font_cache_size", 1337);

layout: Add NULL check in _elm_layout_text_set function

When the parameter 'text' is NULL in elm_layout_text_set function,
the sub object data with the same part name is removed
from the layout's sub object list and the function returns immediately.
However, if the text part doesn't exist in the list,
a new sub object data is appended to the list even though the text is NULL.
This patch adds NULL check so the function can return in such cases.

photocam: add missing EOLIAN prefix

ecore-wl2: Fix formatting

NB: No functional changes

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

elementary: Fix erroneous usage of EINA_UNUSED

'data' parameter is actually used in this function, so remove mis-use
of EINA_UNUSED in function

@fix

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

elementary: Fix over-zealous optimization attempt

This is a fix for some over-zealous optimization attempt. Basically,
we cannot optimize out multiple calls to ecore_wl2_window_input_get as
the window can change in the for loop.

@fix

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

elementary: Fix bug in wl dnd where all subsequent mimetypes would be
accepted

This fixes an issue where in wl dnd, if one mimetype was accepted by a
drop, then all subsequent types would have been accepted also..

@fix

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

elementary: Reformat elm_cnp to be readable

NB: No functional changes, just formatting so I can read this hot mess

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

Revert "layout: Add NULL check in _elm_layout_text_set function"

This reverts commit e3fd84d1de9885aa05021603a50c0287d21d2765.

NULL checking point is wrong.

layout: Add NULL check in _elm_layout_text_set function

When the parameter 'text' is NULL in elm_layout_text_set function,
the sub object data with the same part name is removed
from the layout's sub object list and the function returns immediately.
However, if the text part doesn't exist in the list,
a new sub object data is appended to the list even though the text is NULL.
This patch adds NULL check so the function can return in such cases.

ecore: Add basic test case for efl version

Not very useful test but still good to make sure we
preserve the API.

elm: Add app's EFL information to EFL_MAIN and ELM_MAIN

Bindings should do the same.

efl: Add Efl.Version struct and APIs

The original idea behind knowing the app's version of EFL is not
a great story. It comes from the fact that some bugs exist in
earlier versions of EFL, and some things need to be fixed. But
those fixes may break behaviour for older apps. This patch is
opening the way to the slippery slope of bug compatibility.

Unfortunately this is a requirement if we want to be able to move
forward and not break apps when we fix bugs (behaviour or ABI).

I hope we will not need to implement too many (if any) workaround
such issues. For now, this will only be used as debugging info.

EFL_MAIN() and ELM_MAIN() will both set the app's EFL version
automatically at startup time. Some internal helpers can be added
later to check how the app build-time and run-time version of
EFL differ.

@feature

loop: Add main class property to return the main loop

Note: this is both @class and @property. Hope that's ok for
all bindings.

This returns same as ecore_main_loop_get() (which now uses the eo
api instead).

Ping @cedric (so he can check this patch).

win: Change type of composite interface for Efl.Config

Use Efl.Config.Global instead, since it's what the window will
expose, and it provides access to the "profile" config options.

evas: Add & fix legacy APIs for text filters

Those are marked as EINA_DEPRECATED since they are for legacy only,
and EDC is the preferred method.
Note: these are new in EFL 1.18.

I would rather have not added them as legacy APIs. This patch is
here to align to Tizen which added them by mistake. Also, remove
the "name" argument from filter_program_set as it's not required,
and also not there in Tizen's code base.

Sorry for the mess.

evas: Add 2 EO examples for maps

Those are copy & paste & eo-ified versions of the legacy
samples. I did not touch the other legacy APIs, only maps.

evas: Implement evas map support in eo as a mixin

It relies a bit on evas legacy APIs and will only work on
evas objects (Efl.Canvas.Object) for now.

The main difference with Evas_Map is that there is no
separate map object, as the functions apply directly to
any canvas object.

For convenience, most functions will automatically populate
the map if there was no previous map info. While this may
be convenient, the object's size changes still need to
be tracked to update the map info.

evas: Move evas_map APIs from common to legacy header

+ define Evas_Map struct in Evas_Legacy.h (not eo)

Otherwise, just a simple cut & paste

evas: Move map to a separate mixin

Evas map is not translated to proper eo apis yet.
Efl.Canvas.Map will be a mixin implementing the evas map
API in a bindable API.

ecore_drm: Don't pass uninit garbage to drm ioctls

Hasn't actually caused us any problems so far, but valgrind
whines about it, so fixing anyway.

elementary: try to keep const type in config test.

ecore: fix typo in test.

Thanks Stefan.

ecore_wl2: use ECORE_WL2_EVENT_DATA_SOURCE_END

this event is emited each time the dragging is cancled. So the dragwin
should be deleted here.

elementary/fileselector: Do not use deprecated APIs.

ecore-wl2: Prevent duplicated mouse up events

If you touch downs and ups about all of fingers,
that is sufficient condition to call ecore_wl_input_ungrab().
And then one mouse up event is generated from that function.
So if you set a callback about ECORE_EVENT_MOUSE_BUTTON_UP,
you can watch two mouse up events are occurred.

This is a port of the patch for Ecore_Wayland library (deprecated) to
the newer Ecore_Wl2 library.

ref D4112

@fix

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

ecore-wayland: Prevent duplicated mouse up events.

Summary:
Prevent duplicated mouse up events generated from input_ungrab.
         If you touch downs and ups about all of fingers,
         that is sufficient condition to call ecore_wl_input_ungrab().
         And then one mouse up event is generated from that function.
         So if you set a callback about ECORE_EVENT_MOUSE_BUTTON_UP,
         you can watch two mouse up events are occurred.

Test Plan: Touch and release. And watch mouse up events.

Reviewers: raster, zmike, devilhorns

Reviewed By: devilhorns

Subscribers: ohduna, input.hacker, cedric, jpeg

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

eo: Fix composite objects destruction

This walks the list and removes objects from it. Not
using EINA_LIST_FREE as the actual list_remove is done
inside eo_composite_detach.

evas: Use proper type for stringshares

evas: Fix double stringshare_del

A stringshare for font "style" was not properly duplicated,
resulting in lots of invalid stringshare pointers.

Ping @id213sin (commit 2ce33e73a7d48d4b54a075c07f12c0162).

tests: elm: make cnt unsigned for correct compare

Not really somethign we would hit here but still better to change this.

tests/elementary/elm_test_config.c: In function ‘elm_config_profiles’:
tests/elementary/elm_test_config.c:206:12: warning: comparison between signed and unsigned integer
expressions [-Wsign-compare]
         fail_if(cnt != eina_list_count(list));

tests: elm: remove wrong const for Eina_List

We are assigning somethign new to this list so no chance for const here.

tests/elementary/elm_test_config.c: In function ‘_eina_list_string_has’:
tests/elementary/elm_test_config.c:174:11: warning: assignment discards ‘const’ qualifier from
pointer target type [-Wdiscarded-qualifiers]
    EINA_LIST_FOREACH(list, l, s)

popup: remove max_sc_w because it's duplicate job

Summary:
max_sc_w decide popup's "content" area size when popup scrollable is TRUE.
but in popup's edc, "elm/popup/content/XXXX" has min size when scrollable is TRUE. (and it is content area layout)
max_sc_w has fixed value, so I think deciding by edc is better than fixed code.

Test Plan:
1. elementary_test -to popup
2. check "Enable popup scroll"
3. click any list item.
4. observe popup and compare before this patch.

Reviewers: singh.amitesh, herb, Hermet, raster

Subscribers: cedric, jpeg

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

evas: add pattern "style" when evas query fonts via fontconfig

Summary:
Some fonts can have weird style and weight value.
If a font has a style name as "medium" and a weight value as "semi-bold",
Evas can't load the font using "font=Somefont:style=Medium".
It only can be load with "font=Somefont:style=SemiBold" or
"font=Somefont:weight=SemiBold".
On the other hand, it could be loaded when I tested the following commands.
fc-match -s ":family=Somefont:style=Medium" or
fc-match -s ":family=Somefont:weight=SemiBold"

Evas also should load font based on font's style name.
@fix

Test Plan: N/A

Reviewers: tasn, herdsman, cedric, woohyun, raster

Reviewed By: raster

Subscribers: Blackmole, z-wony, jpeg

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

elementary/layout: changed to Evas_Object * from Elm_Layout * in legacy header.

if EFL_EO_API_SUPPORT is not defined in user side, this will break the build.
because Elm_Layout type is defined in eo header file.

Just keep using Evas_Object * in legacy yet.

efl: Remove eo struct Efl.Gfx.Color

So far this was protected behind ifdef EO_API_SUPPORT. It also
was not used internally. Dropping this before the release, since
we will soon have a (hopefully) better solution to handle various
color representations.

edje: refactor edje color class API

Summary:
refactor setter and getter of edje color class and
move edje_color_class_set/get() into Edje.Object as class functions

Reviewers: cedric, jpeg

Reviewed By: jpeg

Subscribers: kimcinoo

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

Note by jpeg:
The EO APIs introduced here will be revisited in upcoming patches,
in particular:
1. avoid non-premultiplied colors
2. move to a common color class interface

elm_atspi_bridge: add bounds-changed signal sending code.

Summary:
Add bounds-changed signal, this signal can be sent on geometry change
of any evas object when atspi mode is enabled.

Test Plan:
connect to "object:bounds-changed" signal on screen-reader.
when signal BoundsChanged is emitted from elm using elm_interface_atspi_accessible_bounds_changed_signal_emit, the signal should be received
and we should be able to access geometry.

Signed-off-by: Shilpa Singh <shilpa.singh@samsung.com>
Reviewers: cedric, lukasz.stanislawski, kimcinoo

Subscribers: govi, rajeshps, cedric, jpeg

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

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

elementary/layout: attach edje object API with eo compositing

Test Plan: make check

Reviewers: Hermet, jpeg, cedric

Subscribers: slotus.lee

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

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

label: set sliding only if label size is valid

Summary:
When call slide_go before label is resized, setting the sliding is a meaningless.
This patch is for reducing redundant operations.

Signed-off-by: Minkyu Kang <mk7.kang@samsung.com>
Signed-off-by: Deepjyoti Dutta <deepjyoti.d@samsung.com>
Test Plan: elementary_test -to "label slide"

Reviewers: cedric, Hermet

Subscribers: seoz, jpeg

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

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

ecore_wl2: Fix dnd breakage when focus leaves window

Summary:
If dnd in the same window is activated, the focus goes away from the
ecore_wl2 window to the dnd window, after that focus.pointer is NULL.
After focus.pointer is NULL ev->win of all the events will be 0 which
breaks dnd-motions, drop and end.

With prev_pointer beeing the last focused window, we can simply set this
window as event window. After that dnd with jesus works perfectly.

@fix

Reviewers: devilhorns

Subscribers: cedric, jpeg

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

Ui text: make elm test editable

Canvas text: fix format annotations deletion

Ui text: defer decorations to one job

Also, remove callbacks on the deletion.

Ui text: port item support from Entry

This uses the annotations API to retrieve items and their geometries, instead of
the old format API.

Ui text: add cursor_new method

The cursor object is meant to be instantiated in the following manner:
   eo_add(EFL_CANVAS_TEXT_CURSOR_CLASS, text_obj,
      efl_canvas_text_cursor_text_object_set(eo_self, text_obj));

This can't be done directly on the Ui Text object, so it has to be with an
method call, sadly.

@feature

evas: Fix bad clipping with proxy

evas-images2 example was broken due to excessive clipping.
This is one more issue with the cached clip geometry.

By default proxy_src_clip is true. The doc (that I wrote)
mentions that it means both objects (image proxy & source)
share the same clipper. So, this does not mean that the proxy
is clipped to the geometry of the source (this can be useful
in some cases, when replacing an object with a proxy to apply
some kind of effect on it... but this could be rare).

Thanks Amitesh for the report.

@fix

ecore: Replace TABs with spaces

elm_main: Remove useless shadowing variable

elm test: Remove unused variable

Evas events: Implement support for hold event

Fix 'maybe used uninitialized' warning

Summary:
    bin/edje/edje_cc_out.c: In function ‘data_write_images’:
    bin/edje/edje_cc_out.c:1164:7: warning: ‘f’ may be used uninitialized in this function [-Wmaybe-uninitialized]
        if (!f) return EINA_FALSE;
           ^
    bin/edje/edje_cc_out.c:1154:15: note: ‘f’ was declared here
        Eina_File *f;
                   ^

Reviewers: cedric, jpeg

Reviewed By: jpeg

Subscribers: seoz, jpeg

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

evas: Make shaders regen an opt-in behaviour

While the shaders automatic regeneration is very useful when
working on shaders, it has been the source of too many
build breaks.

So I'm making this regeneration an opt-in.

You work on shaders?
Set the env var EFL_SHD_REGEN=1

Revert "evas: Fix compilation when using different build directory"

This reverts commit 3c13ef14e3511611cabf184ff0b83c264b3e620d.

This introduced new build breaks, especially with in-tree builds.
In particular, DIR was not defined. See T3975.

I'll make the shaders regen an opt-in rather than on by default.

elm_entry: restrict focus region to inside of entry object area

Summary:
Focus region must be located in entry object.
Therefore if it get out of entry,
 it just returns last cursor position that can be shown.

@fix

Test Plan:
1. elementary_test "Entry on Page Scroll"
2. click 2nd btn and close popup
3. page should not be scrolled

Reviewers: raster, herdsman, id213sin, woohyun, tasn, cedric

Subscribers: jpeg

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

evas object: Hide "del" event

In EO world, we should stick to EO_EVENT_DEL.

elementary: add test for external icons

The ExtIcon elementary test shows how edje external icons
can be used.

edje_external: fix external icon handling

External icons were handled with a global variable, which reason
to exist seem quite weird. It seems to me it was used to retain
some states of the actual parameters to get them later without having
access to the real parameters.

I don't really get how this would have worked with more than one
icon. When I tried to animate an external icon, with two states,
there were leaks and memory corruption... Now both are gone.

@fix

eina: fix typos in eina_rectangle doc

elementary: include Ecore_Cocoa.h normally

Guards are not necessary anymore.

ecore_cocoa: release Ecore_Cocoa as stable

ecore_cocoa: mark clipboard API as BETA

Also, Since the API is beta, don't make is available to Doxygen.

elm: Always connect to wl when enforced

This is a followup on ca55a920d9a5f6a4274e40fcf901a89e622c3007.

This fixes cnp/dnd for applications started outside of the weston
session. (There are still more problems with dnd and cnp)

edje/example: add a missing closing bracket in svg.

evas: Fix compilation when using different build directory

elementary: return an exit code at exit with the new EFL_MAIN macro.

ecore: add the possibility to return an exit code when leaving the main loop.

elementary: introduce EFL_MAIN and rely on Eo_Event to call the initialisation function.

FIXME: Find a way to return an exit value. Maybe on efl_loop_quit ?