platform/upstream/efl.git
4 years agodocs: Fix typos and wrap EO files to 120 chars
Xavi Artigas [Mon, 2 Sep 2019 14:24:26 +0000 (16:24 +0200)]
docs: Fix typos and wrap EO files to 120 chars

As per our style guide:
https://www.enlightenment.org/contrib/docs/eo-guide.md

4 years agoeolian: move all tests to use the new @move syntax and API
Daniel Kolesa [Mon, 2 Sep 2019 13:23:21 +0000 (15:23 +0200)]
eolian: move all tests to use the new @move syntax and API

4 years agoelua: remove old is_owned api bindings
Daniel Kolesa [Mon, 2 Sep 2019 13:07:01 +0000 (15:07 +0200)]
elua: remove old is_owned api bindings

4 years agoeolian_gen: use new is_move APIs to check ownership transfers
Daniel Kolesa [Mon, 2 Sep 2019 13:05:34 +0000 (15:05 +0200)]
eolian_gen: use new is_move APIs to check ownership transfers

4 years agoRevert "eolian_mono: remove underscore of Evt_Args"
Jaehyun Cho [Mon, 2 Sep 2019 12:41:20 +0000 (21:41 +0900)]
Revert "eolian_mono: remove underscore of Evt_Args"

This reverts commit ac99e2ac9410d5b2ef6225fa1aaaf9ffcd6578fb.

This patch is reverted because D9692 covers this patch.

4 years agocsharp: add SetKeyValue and GetKeyValue to EoWrapper
Jaehyun Cho [Mon, 2 Sep 2019 12:26:58 +0000 (21:26 +0900)]
csharp: add SetKeyValue and GetKeyValue to EoWrapper

Summary:
SetKeyValue adds a value object associated with a key object to hash
table.
GetKeyValue returns a value object associated with a key object from
hash table.

Reviewers: felipealmeida, lauromoura, vitor.sousa, woohyun, cedric

Subscribers: zmike, bu5hm4n, segfaultxavi, #reviewers, #committers

Tags: #efl

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

4 years agoeolian_mono: remove underscore of Evt_Args
Jaehyun Cho [Mon, 2 Sep 2019 12:14:44 +0000 (21:14 +0900)]
eolian_mono: remove underscore of Evt_Args

Since C# does not use underscore in type name, the underscore of
Evt_Args is removed.

4 years agoefl_canvas_animation_player: delete previous target event callback
Yeongjong Lee [Mon, 2 Sep 2019 11:55:51 +0000 (20:55 +0900)]
efl_canvas_animation_player: delete previous target event callback

Summary:
When target is changed, its event callback related `animation_player` should be
deleted.

Reviewers: Jaehyun_Cho

Reviewed By: Jaehyun_Cho

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

4 years agorelease: Update NEWS and bump version for 1.23.0-alpha1 release
Stefan Schmidt [Mon, 2 Sep 2019 09:47:11 +0000 (11:47 +0200)]
release: Update NEWS and bump version for 1.23.0-alpha1 release

4 years agodocs: Switch remaining #doxygen-style refs to @eo-style
Xavi Artigas [Mon, 2 Sep 2019 09:57:27 +0000 (11:57 +0200)]
docs: Switch remaining #doxygen-style refs to @eo-style

4 years agoeolian: fix validation of ownability with hashes
Daniel Kolesa [Sat, 31 Aug 2019 12:11:48 +0000 (14:11 +0200)]
eolian: fix validation of ownability with hashes

4 years agoeolian: properly handle non-beta restriction of __undefined_type
Daniel Kolesa [Sat, 31 Aug 2019 12:02:20 +0000 (14:02 +0200)]
eolian: properly handle non-beta restriction of __undefined_type

4 years agoeolian: always validate inner types of complex types for @move
Daniel Kolesa [Sat, 31 Aug 2019 00:09:46 +0000 (02:09 +0200)]
eolian: always validate inner types of complex types for @move

4 years agoelua: add eolian bindings for type_is_move
Daniel Kolesa [Sat, 31 Aug 2019 00:06:05 +0000 (02:06 +0200)]
elua: add eolian bindings for type_is_move

4 years agoeolian: add API to check if an inner type of complex type is @move
Daniel Kolesa [Sat, 31 Aug 2019 00:04:13 +0000 (02:04 +0200)]
eolian: add API to check if an inner type of complex type is @move

This complements the equivalent APIs of parameters and so on.
It is not the same as the older type_is_owned API, which applied
to everything.

4 years agoeolian: refactor ownability validation to accommodate new syntax
Daniel Kolesa [Fri, 30 Aug 2019 23:49:18 +0000 (01:49 +0200)]
eolian: refactor ownability validation to accommodate new syntax

This refactors ownability checks so that they're only performed
in contexts where @move can be specified, which both helps
with performance and enables later removal of ownability info
from types themselves.

4 years agoeolian: validate @by_ref correctness
Daniel Kolesa [Fri, 30 Aug 2019 23:30:00 +0000 (01:30 +0200)]
eolian: validate @by_ref correctness

The @by_ref qualifier can only be used with value types, to
guarantee exactly one level of pointers.

4 years agocxx: Fix example after popup changes.
Lauro Moura [Fri, 30 Aug 2019 19:34:14 +0000 (21:34 +0200)]
cxx: Fix example after popup changes.

Summary: Popup api changed in 1ca07b72da944

Reviewers: zmike, felipealmeida, bu5hm4n

Reviewed By: bu5hm4n

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

4 years agoefl_ui_collection: prevent the invalid insertion of items.
Marcel Hollerbach [Fri, 30 Aug 2019 18:50:32 +0000 (14:50 -0400)]
efl_ui_collection: prevent the invalid insertion of items.

Summary:
the group hirachy that you can build up with group items is flattend
down in the collection, as a linear list. Based on that a user might be
able to insert items at invalid positions, like between groups, without
having set the proper group item, which ... is a problem.

This commit ensures that no new item ever destroys the neightboor hood
between two items of the same group or a item with a group and its group
leader.

Reviewers: stefan_schmidt, zmike, segfaultxavi, SanghyeonLee

Reviewed By: SanghyeonLee

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

4 years agoefl_ui/pan: remove pan,viewport,changed event
Mike Blumenkrantz [Fri, 30 Aug 2019 18:48:14 +0000 (14:48 -0400)]
efl_ui/pan: remove pan,viewport,changed event

Summary:
this is just a duplicate of entity geometry changed events, users of
pan should just use those instead

ref T7708
Depends on D9791

Reviewers: segfaultxavi

Reviewed By: segfaultxavi

Subscribers: segfaultxavi, cedric, #reviewers, #committers

Tags: #efl_widgets

Maniphest Tasks: T7708

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

4 years agoefl_ui/pan: only emit size change event on pan content size change
Mike Blumenkrantz [Fri, 30 Aug 2019 18:48:09 +0000 (14:48 -0400)]
efl_ui/pan: only emit size change event on pan content size change

Summary:
changing the size of the pan content does not require that the pan
position also changes, so let users of pan sort this out using the correct
events

ref T7708
Depends on D9790

Reviewers: segfaultxavi

Reviewed By: segfaultxavi

Subscribers: cedric, #reviewers, #committers

Tags: #efl_widgets

Maniphest Tasks: T7708

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

4 years agoefl_ui/pan: add position info to pan,content_position,changed event
Mike Blumenkrantz [Fri, 30 Aug 2019 18:48:04 +0000 (14:48 -0400)]
efl_ui/pan: add position info to pan,content_position,changed event

Summary:
changed events should always send the related data when possible

ref T7708
Depends on D9789

Reviewers: segfaultxavi

Reviewed By: segfaultxavi

Subscribers: segfaultxavi, cedric, #reviewers, #committers

Tags: #efl_widgets

Maniphest Tasks: T7708

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

4 years agoefl_ui/pan: add pan,content_size,changed event
Mike Blumenkrantz [Fri, 30 Aug 2019 18:47:58 +0000 (14:47 -0400)]
efl_ui/pan: add pan,content_size,changed event

Summary:
this is a shortcut so pan users don't have to manually hook the content
resize event
Depends on D9788

Reviewers: bu5hm4n

Reviewed By: bu5hm4n

Subscribers: cedric, #reviewers, #committers

Tags: #efl_widgets

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

4 years agoefl_ui/pan: rename pan,position,changed -> pan,content_position,changed
Mike Blumenkrantz [Fri, 30 Aug 2019 18:47:52 +0000 (14:47 -0400)]
efl_ui/pan: rename pan,position,changed -> pan,content_position,changed

Summary:
the pan object is not changing its position, it's moving the content

ref T7708
Depends on D9787

Reviewers: segfaultxavi

Reviewed By: segfaultxavi

Subscribers: segfaultxavi, cedric, #reviewers, #committers

Tags: #efl_widgets

Maniphest Tasks: T7708

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

4 years agoefl_ui/pan: remove pan content,changed event
Mike Blumenkrantz [Fri, 30 Aug 2019 18:47:46 +0000 (14:47 -0400)]
efl_ui/pan: remove pan content,changed event

Summary:
this was being emitted as a catch-all for pan changes, where something else
should have been monitored/used instead

ref T7708

Reviewers: bu5hm4n

Reviewed By: bu5hm4n

Subscribers: bu5hm4n, cedric, #reviewers, #committers

Tags: #efl_widgets

Maniphest Tasks: T7708

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

4 years agoevas_filter_parser: remove dereferenced NULL
Shinwoo Kim [Fri, 30 Aug 2019 13:17:59 +0000 (09:17 -0400)]
evas_filter_parser: remove dereferenced NULL

Summary:
This is detected by static analysis tool.
The variable last could be NULL when it is dereferenced.

Reviewers: Hermet, zmike, bu5hm4n

Reviewed By: zmike

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

4 years agoelm sel mgr - remove event handlers when no drop targets are left
Carsten Haitzler (Rasterman) [Fri, 30 Aug 2019 17:17:38 +0000 (18:17 +0100)]
elm sel mgr - remove event handlers when no drop targets are left

case left over that this didnt handle right and this breaks e's dnd
once a file preview popup appears.

@fix

4 years agotests: use void cast with util callback
Mike Blumenkrantz [Fri, 30 Aug 2019 17:02:55 +0000 (13:02 -0400)]
tests: use void cast with util callback

fixes a lot of compiler warnings

Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D9802

4 years agoefl_ui/popup: timeout -> closing_timeout
Mike Blumenkrantz [Fri, 30 Aug 2019 17:02:01 +0000 (13:02 -0400)]
efl_ui/popup: timeout -> closing_timeout

simple property rename

ref T7902

Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D9801

4 years agodocs: Proper documentation for Collection and Item classes
Xavi Artigas [Fri, 30 Aug 2019 09:53:40 +0000 (11:53 +0200)]
docs: Proper documentation for Collection and Item classes

Reviewers: bu5hm4n

Reviewed By: bu5hm4n

Subscribers: cedric, #reviewers, #committers

Tags: #efl, #documentation

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

4 years agoelua: expose new tags via eolian bindings
Daniel Kolesa [Fri, 30 Aug 2019 15:14:09 +0000 (17:14 +0200)]
elua: expose new tags via eolian bindings

4 years agoeolian: add API to expose the new @move and @by_ref tags
Daniel Kolesa [Fri, 30 Aug 2019 15:04:44 +0000 (17:04 +0200)]
eolian: add API to expose the new @move and @by_ref tags

4 years agoeolian: add parsing for @move and @by_ref
Daniel Kolesa [Fri, 30 Aug 2019 14:08:31 +0000 (16:08 +0200)]
eolian: add parsing for @move and @by_ref

The @by_ref qualifier is now allowed on parameters, returns and
struct fields in the usual qualifier section. It will mean that
this type is passed around by reference, and will only be allowed
on types that are not already pointer-like.

The @move qualifier will replace @owned as one with a clearer
meaning. It means "transfer of ownership". It has the same semantics
as the current @owned, i.e. on return values it transfers ownership
of the value to the caller, on parameters it transfers ownership
to the callee (the inverse is the default when not specified).
On struct fields, it means the field will transfer together with
the struct that contains it.

4 years agoelm_entry: add null check logic on elm_entry_text_set API
Bowon Ryu [Fri, 30 Aug 2019 12:47:19 +0000 (08:47 -0400)]
elm_entry: add null check logic on elm_entry_text_set API

Summary: If current_text is NULL, a segmentation fault is occured.

Test Plan: N/A

Reviewers: cedric, Hermet, Jaehyun_Cho, zmike

Reviewed By: zmike

Subscribers: zmike, cedric, #reviewers, #committers

Tags: #efl

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

4 years agotest_ui_slder: remove meaningless printf when reaching the limit
WooHyun Jung [Fri, 30 Aug 2019 12:42:41 +0000 (08:42 -0400)]
test_ui_slder: remove meaningless printf when reaching the limit

Summary:
Without new internal min/max properties, this may be the only way
to not give redundant printing of the meaningless value.

@ref T8188

Test Plan:
1. elementary_test
2. EFl.Ui.Slider
3. On the limit slider, move the knob above the limit

Reviewers: bu5hm4n

Reviewed By: bu5hm4n

Subscribers: cedric, #reviewers, #committers

Tags: #efl

Maniphest Tasks: T8188

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

4 years agoeio - don't call cb's after a cancel
Carsten Haitzler (Rasterman) [Fri, 30 Aug 2019 11:43:24 +0000 (12:43 +0100)]
eio - don't call cb's after a cancel

i saw a segv in e/efm .. an eio call was called after it had been
canceled. this should never happen. ever. so ensure it does not once
an eio file has been canceled.

@fix

4 years agoefl_ui_datetime: change min/max to date_min/date_max
WooHyun Jung [Fri, 30 Aug 2019 10:08:16 +0000 (12:08 +0200)]
efl_ui_datetime: change min/max to date_min/date_max

Summary:
To keep consistency with other classes, change the property name
properly.

@ref T7869

Reviewers: zmike, segfaultxavi, bu5hm4n

Reviewed By: segfaultxavi

Subscribers: cedric, #reviewers, #committers

Tags: #efl

Maniphest Tasks: T7869

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

4 years agocsharp: Fix factory inheritance.
Lauro Moura [Fri, 30 Aug 2019 05:33:28 +0000 (14:33 +0900)]
csharp: Fix factory inheritance.

Summary:
C#'s factory must inherit from `LayoutFactory` so we can access its
implementation of `FactoryBind`.

Depends on D9759 to avoid test failure on PropertyBound events.

Reviewers: felipealmeida, cedric, SanghyeonLee

Reviewed By: SanghyeonLee

Subscribers: #reviewers, #committers

Tags: #efl

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

4 years agoedje/optimization: refactor edje_color_class_set() api.
subhransu mohanty [Thu, 29 Aug 2019 18:31:52 +0000 (14:31 -0400)]
edje/optimization: refactor edje_color_class_set() api.

Summary:
During _elm_config_color_overlay_apply() application can call this api
100's of time depending on how many overlay it wants add . As each color_class
set triggers the global color class apply chain 3 times (for color , outline and shadow)
just club them once and emit single color_class,set event.

Reviewers: Hermet, ali.alzyod, zmike

Reviewed By: zmike

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

4 years agoedje/style: append file prefix to textblock_style api that affects on file level
subhransu mohanty [Thu, 29 Aug 2019 14:07:25 +0000 (10:07 -0400)]
edje/style: append file prefix to textblock_style api that affects on file level

Summary:
All those api's are working on the Edje_file level so by appending the file prefix
and changing the argument to Edje_File leads to easy maintainability.

Depends on D9776

Reviewers: Hermet, ali.alzyod, zmike

Reviewed By: zmike

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

4 years agoelm_systray: add back beta constructor
Marcel Hollerbach [Thu, 29 Aug 2019 14:06:42 +0000 (10:06 -0400)]
elm_systray: add back beta constructor

Summary:
1/2 year  ago we have removed the automatic generation of legacy
headers. In this go we also removed the installation of legacy eo beta
APIs, as they definitly should not be used. However, there is the legacy
systray API which does not depend directly on eo. Apps could use this
before due to specifying the BETA define, now they only can call the
API, but cannot construct the object they need. This adds this back
under the beta tag, so those apps can continue working.

I am not sure myself if i like the commit or not, its a beta thing, why
should we fix it. However, Its quite a easy thing to add such a
constructor method for us, and enables app to continue working.

Reviewers: zmike, segfaultxavi

Reviewed By: zmike

Subscribers: jf_simon, cedric, #reviewers, #committers

Tags: #efl

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

4 years agoelm config -> allow people to set desktop entry and magnifier config
Carsten Haitzler (Rasterman) [Thu, 29 Aug 2019 16:29:38 +0000 (17:29 +0100)]
elm config -> allow people to set desktop entry and magnifier config

4 years agoelm config - add config api's to set desktop style entry flag/mode
Carsten Haitzler (Rasterman) [Thu, 29 Aug 2019 16:15:32 +0000 (17:15 +0100)]
elm config - add config api's to set desktop style entry flag/mode

4 years agoevas -gl engines - buffer age - bring back fix for changing buffer age
Carsten Haitzler (Rasterman) [Tue, 27 Aug 2019 22:05:18 +0000 (23:05 +0100)]
evas -gl engines - buffer age - bring back fix for changing buffer age

so we USED to... if buffer age changed between frames, force a full
redraw and add that to the pipeline... somehow someone removed that.
this brings that back as it's really needed to deal with updating
correctly.

@fix

4 years agomono-docs: Update DocFX version and use online symbol lookup
Xavi Artigas [Thu, 29 Aug 2019 15:44:16 +0000 (17:44 +0200)]
mono-docs: Update DocFX version and use online symbol lookup

Version 2.45 of DocFX seems much much faster than previous 2.40, but could just
be a matter of stale caches. Anyway, no breakages have been detected.
Using an online symbol resolution server (https://xref.docs.microsoft.com)
adds some more links for .NET symbols (and no noticeable slowdown).

4 years agoefl: implement efl.file::unload for classes that implement load
Mike Blumenkrantz [Thu, 29 Aug 2019 13:26:15 +0000 (09:26 -0400)]
efl: implement efl.file::unload for classes that implement load

unload is a crucial method for classes that implement load, as this is
the method which is called during e.g., efl_file_simple_load() when
a new file is specified in order to destroy the previous object data

Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D9786

4 years agoefl_ui/alert_popup: always create button layout
Mike Blumenkrantz [Wed, 28 Aug 2019 13:47:25 +0000 (09:47 -0400)]
efl_ui/alert_popup: always create button layout

this can be safely done in the constructor without issues now

Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D9785

4 years agomvvm: Fix Layout_Factory property bound event
Lauro Moura [Tue, 27 Aug 2019 23:58:06 +0000 (23:58 +0000)]
mvvm: Fix Layout_Factory property bound event

We should keep the key alive at least until the `property,bound` event
is emmitted.

Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D9759

4 years agofix build: remove deleted header file
Marcel Hollerbach [Thu, 29 Aug 2019 13:27:07 +0000 (15:27 +0200)]
fix build: remove deleted header file

- sorry.

4 years agoeolian: disallow legacy keyword in enums in stable api
Daniel Kolesa [Thu, 29 Aug 2019 13:05:24 +0000 (15:05 +0200)]
eolian: disallow legacy keyword in enums in stable api

This also marks a bunch of users of that as beta, as they were
effectively beta.

4 years agoeolian: reverse keywords '@transfer' and '@by_ref'
Daniel Kolesa [Thu, 29 Aug 2019 12:43:41 +0000 (14:43 +0200)]
eolian: reverse keywords '@transfer' and '@by_ref'

The '@transfer' keyword is a clearer replacement for '@owned'.
It will be specified on params/returns/... to specify that
ownership will be transferred between caller/callee.

4 years agoeina_test: resolve build warning
Ali Alzyod [Thu, 29 Aug 2019 07:44:22 +0000 (07:44 +0000)]
eina_test: resolve build warning

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

4 years agoefl_ui_tab_pager: make this compatible with Efl.Pack_Linear test suite
Marcel Hollerbach [Fri, 23 Aug 2019 17:34:53 +0000 (19:34 +0200)]
efl_ui_tab_pager: make this compatible with Efl.Pack_Linear test suite

the testsuite now is happy with the behaviour in there. Additionally,
the internals are cleaned up, fields that are not used anymore are
deleted, and the logic maintaining them as well.

Reviewed-by: WooHyun Jung <woohyun0705@gmail.com>
Differential Revision: https://phab.enlightenment.org/D9732

4 years agoefl_ui_tab_pager: remove the setter for tab_bar
Marcel Hollerbach [Fri, 23 Aug 2019 17:16:56 +0000 (19:16 +0200)]
efl_ui_tab_pager: remove the setter for tab_bar

the setter of the tab_bar here is quite useless. If you set it after
something else was set, the item situation is not clear anymore, there
is more in the spotlight than in the bar. You cannot use the widget
without tab bar. So the logical consequence for me was that the pager
create the tab bar by itself, and you can just access it via a getter.

Reviewed-by: WooHyun Jung <woohyun0705@gmail.com>
Differential Revision: https://phab.enlightenment.org/D9731

4 years agoefl_ui_tab_bar: migrate to Efl.Pack_Linear
Marcel Hollerbach [Fri, 23 Aug 2019 16:20:32 +0000 (18:20 +0200)]
efl_ui_tab_bar: migrate to Efl.Pack_Linear

with pack linear it is a lot more convinient and error prover to add
items to this widget, this also comes with the pack test suite, which
verifies that this behaviour is not broken. Additionally we have fewer
APIs that a user has to learn.

Reviewed-by: WooHyun Jung <woohyun0705@gmail.com>
Differential Revision: https://phab.enlightenment.org/D9730

4 years agoefl_ui_tab_bar: implement efl.ui.single_selectable
Marcel Hollerbach [Fri, 23 Aug 2019 15:51:49 +0000 (17:51 +0200)]
efl_ui_tab_bar: implement efl.ui.single_selectable

this is for now the simples way IMO, the one or another implementation
thing in efl_ui_tab_pager is a little bit clumsy, but that should be
able to be improved once Efl.Pack_Linear is implemented.

Reviewed-by: WooHyun Jung <woohyun0705@gmail.com>
Differential Revision: https://phab.enlightenment.org/D9729

4 years agoefl_ui_tab_page: remove part
Marcel Hollerbach [Fri, 23 Aug 2019 15:33:50 +0000 (17:33 +0200)]
efl_ui_tab_page: remove part

I introduced the property for item a few patches earlier. I do not want
to use a part for this, as i would have to implement the class twice
(Once in the part, once as a selfstanding object).
It would also be kind of weird to have code like:
efl_content_set(efl_part(efl_part("item"), "tab"),bla).

The purpose of this part also got a little bit replaced when the item
was introduced, as this is the central point of data for the tab bar
icon. So need for an additional editition part.

Reviewed-by: WooHyun Jung <woohyun0705@gmail.com>
Differential Revision: https://phab.enlightenment.org/D9728

4 years agoefl_ui_tab_bar: remove update event
Marcel Hollerbach [Thu, 29 Aug 2019 12:31:24 +0000 (21:31 +0900)]
efl_ui_tab_bar: remove update event

Summary:
before the previous commit this was used to update the tab_bar icon.
However, this is not needed anymore, as the item stuff can directly be
updated. So I do not see a reason for this event anymore.

Reviewers: zmike, segfaultxavi, cedric, woohyun, bowonryu

Reviewed By: woohyun

Subscribers: #reviewers, #committers

Tags: #efl

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

4 years agoefl_ui_*_default_item: improve docs
Marcel Hollerbach [Thu, 29 Aug 2019 12:13:46 +0000 (14:13 +0200)]
efl_ui_*_default_item: improve docs

Summary:
ref T7717
Depends on D9782

Reviewers: segfaultxavi

Reviewed By: segfaultxavi

Subscribers: cedric, #reviewers, #committers

Tags: #efl

Maniphest Tasks: T7717

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

4 years agoefl_ui_grid/list: update single class doc
Marcel Hollerbach [Thu, 29 Aug 2019 12:13:42 +0000 (14:13 +0200)]
efl_ui_grid/list: update single class doc

Summary:
its not much more, but a little bit. All other information about the
features in this widget can be found via the implemented interfaces.

ref T7881
ref T7870

Reviewers: segfaultxavi

Reviewed By: segfaultxavi

Subscribers: cedric, #reviewers, #committers

Tags: #efl

Maniphest Tasks: T7881, T7870

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

4 years agointroduce efl_ui_tab_bar_default_item
Marcel Hollerbach [Thu, 29 Aug 2019 12:07:47 +0000 (21:07 +0900)]
introduce efl_ui_tab_bar_default_item

Summary:
Tab bar used a direct layout to implement all this functionality by its
own. However, it seems that we can reuse a lot of object functionality
that is already part of efl.ui.item. With this commit the features that
are not part of Efl.Ui.Item are moved to Efl.Ui.Tab_Bar_Default_Item.

The tab bar is changed in a way that you do not need to pass the icon
and label by hand anymore, you can rather just pass the tab_bar default
item. Additionally, the item for a tab_page can now directly be
generated from a page.

This is the first commit in order to cleanup efl_ui_tab_bar, tab_page &
tab_pager. The goal is also to implement our interfaces for inserting
elements on those objects. So our common test suites can also be used.

Reviewers: zmike, segfaultxavi, cedric, woohyun, bowonryu

Reviewed By: woohyun

Subscribers: #reviewers, #committers

Tags: #efl

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

4 years agoeolian: drop @cref
Daniel Kolesa [Thu, 29 Aug 2019 11:46:35 +0000 (13:46 +0200)]
eolian: drop @cref

In the end this was just a failed experiment that didn't turn
out to be practical. For now, revert back to ptr(const(T)) until
a proper replacement for pointer syntax is added.

4 years agoevas_textblock: allocator use same heap if it is large enough
Ali Alzyod [Thu, 29 Aug 2019 09:43:00 +0000 (18:43 +0900)]
evas_textblock: allocator use same heap if it is large enough

Summary:
allocator use same heap if it is large enough

I am also think to move this struct/functionality into common place, I think we can use it in other parts too

Reviewers: smohanty, Hermet, bowonryu, SanghyeonLee

Reviewed By: smohanty, SanghyeonLee

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

4 years agotests: eldbus: fix heap use after free error detected by address sanitizer
Stefan Schmidt [Thu, 22 Aug 2019 09:56:11 +0000 (11:56 +0200)]
tests: eldbus: fix heap use after free error detected by address sanitizer

Using the just unref'ed message in the call next line is not going to
end well.

==26321==ERROR: AddressSanitizer: heap-use-after-free on address 0x603000001210 at pc 0x7fc456fd37f5 bp 0x7fff37a00a50 sp 0x7fff37a00a40
READ of size 4 at 0x603000001210 thread T0

    #0 0x7fc456fd37f4 in eldbus_message_path_get ../src/lib/eldbus/eldbus_message.c:193
    #1 0x42f6ab in utc_eldbus_message_ref_unref_p ../src/tests/eldbus/eldbus_test_eldbus_message.c:877
    #2 0x7fc456a27744 in srunner_run_tagged (/lib64/libcheck.so.0+0x6744)
    #3 0x406a01 in _efl_suite_run_end ../src/tests/eldbus/../efl_check.h:282
    #4 0x407357 in _efl_suite_build_and_run ../src/tests/eldbus/../efl_check.h:403
    #5 0x407925 in main ../src/tests/eldbus/eldbus_suite.c:49
    #6 0x7fc45685df32 in __libc_start_main (/lib64/libc.so.6+0x23f32)
    #7 0x405d8d in _start (/home/stefan/EFL/efl/build/src/tests/eldbus/eldbus_suite+0x405d8d)

ref T8142

Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com>
Differential Revision: https://phab.enlightenment.org/D9755

4 years agoefl_ui_radio_group: migrate to Efl.Ui.Single_Selectable
Marcel Hollerbach [Thu, 22 Aug 2019 13:06:22 +0000 (15:06 +0200)]
efl_ui_radio_group: migrate to Efl.Ui.Single_Selectable

with this the whole thing is migrated to single_selectable. The group is
added to the spec test suite. The elm_test case of radio_group now also has
a fallback option to demonstrate the usage of it.

This also fixes a broken testcase, where a flag was forgotten to be
checked.

ref T8024

Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com>
Differential Revision: https://phab.enlightenment.org/D9706

4 years agoefl_ui_layout: remove efl.end
Marcel Hollerbach [Wed, 28 Aug 2019 16:05:14 +0000 (18:05 +0200)]
efl_ui_layout: remove efl.end

someone decided it should be named efl.extra, the previous commits are
ensuring that.

Reviewed-by: SangHyeon Jade Lee <sh10233.lee@samsung.com>
Differential Revision: https://phab.enlightenment.org/D9773

4 years agoefl_ui_text: theme rename efl.end -> efl.extra
Marcel Hollerbach [Wed, 28 Aug 2019 16:04:34 +0000 (18:04 +0200)]
efl_ui_text: theme rename efl.end -> efl.extra

Reviewed-by: SangHyeon Jade Lee <sh10233.lee@samsung.com>
Differential Revision: https://phab.enlightenment.org/D9772

4 years agoEdje: ease the warning level.
Hermet Park [Thu, 29 Aug 2019 05:34:43 +0000 (14:34 +0900)]
Edje: ease the warning level.

We already take care the case, it's not so serious as to error.

4 years agoefl_ui : add efl.extra part as layout swallow part list.
SangHyeon Jade Lee [Thu, 29 Aug 2019 01:18:35 +0000 (10:18 +0900)]
efl_ui : add efl.extra part as layout swallow part list.

Summary:
efl.extra part is extra part of the layout swallow,
which can be generally used on overall layout object.
To be supported layout part infrastructures,
add efl.extra on layout_swallow_part[] list.

Depends on D9233

Reviewers: cedric, bu5hm4n, zmike, segfaultxavi

Reviewed By: bu5hm4n

Subscribers: #reviewers, #committers

Tags: #efl

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

4 years agoefl_object: set @beta to name_find
WooHyun Jung [Wed, 28 Aug 2019 12:23:00 +0000 (12:23 +0000)]
efl_object: set @beta to name_find

This property should not be used by application developers
with current status.
There is no way to trace child objects from parents, because
many codes are internally calling efl_parent_set.

Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D9768

4 years agotests/ecore_wl2: Skip aux_hints_supported test if not run in Enlightenment
Christopher Michael [Wed, 28 Aug 2019 11:59:13 +0000 (07:59 -0400)]
tests/ecore_wl2: Skip aux_hints_supported test if not run in Enlightenment

This test will fail when run in all situations (weston, command line,
etc) except when run under Enlightenment, so add a small env check so
that it can be skipped if we are not running E.

ref T8016

4 years agomono-docs: Remove useless semicolon from doc comments
Xavi Artigas [Wed, 28 Aug 2019 11:42:28 +0000 (13:42 +0200)]
mono-docs: Remove useless semicolon from doc comments

4 years agoEfl.Pack.Layout docs: remove refs to protected methods from public methods
Xavi Artigas [Wed, 28 Aug 2019 09:18:06 +0000 (11:18 +0200)]
Efl.Pack.Layout docs: remove refs to protected methods from public methods

4 years agopyolian: Update after event and binbuf keywords
Lauro Moura [Wed, 28 Aug 2019 08:21:18 +0000 (10:21 +0200)]
pyolian: Update after event and binbuf keywords

Reviewers: q66, DaveMDS

Reviewed By: q66

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

4 years agoeolian_aux: this must be recursive
Marcel Hollerbach [Wed, 28 Aug 2019 08:20:48 +0000 (10:20 +0200)]
eolian_aux: this must be recursive

Summary:
in order to get all callables, this must be recursive, otherwise deeper
callables are forgotten.

Reviewers: q66

Reviewed By: q66

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

4 years agoedje: refactor _edje_object_file_set_internal() part 2.
subhransu mohanty [Wed, 28 Aug 2019 07:23:55 +0000 (16:23 +0900)]
edje: refactor _edje_object_file_set_internal() part 2.

Summary:
- split functionality into smaller helper class
  _edje_process_colorclass() , _edje_process_sizeclass() and
 _edje_process_physics() for easy redability and maintenance.

Reviewers: Hermet, zmike

Reviewed By: Hermet

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

4 years agoefl_canvas_vg_object/evas_object_vg: Change to legacy naming rule.
JunsuChoi [Wed, 28 Aug 2019 05:09:13 +0000 (14:09 +0900)]
efl_canvas_vg_object/evas_object_vg: Change to legacy naming rule.

Summary:
   Enum and object decleared in efl_canvas_vg_object_eo.legacy.*
   use Efl_Canvas_Vg_* nameing rule.
   This skips the definition of type declared in Efl.Canvas.Vg.Object class.
   So change these to legacy names.

Test Plan: N/A

Reviewers: Hermet, smohanty, kimcinoo

Reviewed By: Hermet

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

4 years agoefl_ui_layout: restore sizing_eval logic to keep backward compatibility 81/213781/2
Yeongjong Lee [Wed, 11 Sep 2019 06:56:29 +0000 (15:56 +0900)]
efl_ui_layout: restore sizing_eval logic to keep backward compatibility

Since sizing eval refactoring patch(627eb17f1 ~ a25d8642b),
sizing_eval(group_calculate) is called during `main_loop_iterate` which means
if you call finction related to `size_get` before main_loop_iterate, it return
wrong value.

Test Plan:
```
 #include <Elementary.h>

EAPI_MAIN int
elm_main(int argc EINA_UNUSED, char **argv EINA_UNUSED)
{
   Evas_Object *win = NULL;
   Evas_Object *btn = NULL;
   Evas_Object *label = NULL;
   Evas_Object *scroller = NULL;
   Evas_Coord vw = 0;
   Evas_Coord vh = 0;

   elm_policy_set(ELM_POLICY_QUIT, ELM_POLICY_QUIT_LAST_WINDOW_CLOSED);

   win = elm_win_util_standard_add("Greetings", "Hello, World!");
   elm_win_autodel_set(win, EINA_TRUE);
   evas_object_show(win);

   scroller = elm_scroller_add(win);
   label = elm_label_add(win);
   evas_object_size_hint_min_set(label, 500, 150);

   elm_object_content_set(scroller, label);
   elm_scroller_child_size_get(scroller, &vw, &vh);

   printf("vw : %d, vh : %d\n", vw, vh);

   return 0;
}
ELM_MAIN()
```
vw, vh should be (500, 150).

@tizen_fix

Change-Id: I036b76379b7f0cc7ee6faa914ddd56e9f38504d7

4 years agoelm_slider: change the default value for indicator_show 52/213852/1
WooHyun Jung [Mon, 16 Sep 2019 01:16:22 +0000 (10:16 +0900)]
elm_slider: change the default value for indicator_show

The default value for visibility of indicator should be kept as
EINA_FALSE to keep backward compatibility in Tizen.

@tizen_fix

Change-Id: I1019d54e9466953b09525244c8d069988e4a703c

4 years agoecore_wl2: set opaque state while changing alpha or transparent value of window 53/213753/1
Doyoun Kang [Wed, 11 Sep 2019 04:30:38 +0000 (13:30 +0900)]
ecore_wl2: set opaque state while changing alpha or transparent value of window

There was a bug that the window didn't call wl_surface_set_opaque_region
when its alpha or transparent value was changed.

@tizen_fix

Change-Id: I8fb11a6153a647835e1abc83486b021b2952d725

4 years agoeldbus: Temporary remove critical log level. 99/213699/1
JunsuChoi [Tue, 10 Sep 2019 08:25:15 +0000 (17:25 +0900)]
eldbus: Temporary remove critical log level.

If eldbus_shutdown is called, if there is any connection information left,
it will print the critical log.
Critical logs may make abort depending on the situation.
eldbus_shutdown prints the connection information remaining after log output.
And eldbus_shutdown should call the rest of the shutdowns.

Tizen does not provide a APIs for user side interface of eldbus.
Therefore, if this problem occurs from the user's side, this is the area that the user cannot control.
So Temporarily lower the log level.

@Tizen_only

Change-Id: I9a56c49b89e67ef07cedd14c2cebcf64b4db33dd

4 years agoecore_imf: Modified to allow sending negative coordinate 08/213508/2
InHong Han [Fri, 6 Sep 2019 05:26:10 +0000 (14:26 +0900)]
ecore_imf: Modified to allow sending negative coordinate

Change-Id: Iaab502563b17f642a9e4bdfcb163ae65db0bab13

4 years agoelm perf_test: fix warning of implicit conversion of floating point 04/213604/2
Jaehyun Cho [Mon, 9 Sep 2019 05:46:01 +0000 (14:46 +0900)]
elm perf_test: fix warning of implicit conversion of floating point

The result of integral division is converted into floating point.
To fix the warning by assigning the result to Evas_Coord variable, type
cast is done.

Change-Id: Ic25c4c4889678b56c286d31586a60b1f66fab9d4

4 years agoefl_ui_spotlight: fix warning of implicit conversion of floating point 03/213603/1
Jaehyun Cho [Mon, 9 Sep 2019 05:23:26 +0000 (14:23 +0900)]
efl_ui_spotlight: fix warning of implicit conversion of floating point

The result of integral division is converted into floating point.
To fix the warning by assigning the result to integer variable, type
cast is done.

Change-Id: Icfdd614bbd100e5b54b8ed09dcf272b4dfcbde90

4 years agoRevert "efl.spec: Enable pulseaudio." 82/213582/2
Woochanlee [Mon, 9 Sep 2019 04:34:01 +0000 (13:34 +0900)]
Revert "efl.spec: Enable pulseaudio."

This reverts commit 363c3cf602945866659b1f9266874ac0c94303ad.

Change-Id: I5cd23ff07812721d55fcc39166d7ecdb74e03672
Signed-off-by: Woochanlee <wc0917.lee@samsung.com>
4 years agoevas_events: do not to cause abort by input event feed from post event callback 50/213550/3
Jaehyun Cho [Fri, 6 Sep 2019 12:16:24 +0000 (21:16 +0900)]
evas_events: do not to cause abort by input event feed from post event callback

New input events feed from a post event callback may be risky because it
may cause recursive calls. (T3144)
However, it is extremely difficult to cause recursive calls in this
case.
Therefore, the CRI is replaced with the DBG not to cause abort by new
input events feed from post event callback.

e.g. "drag,start" callback of elm_interface_scroller is called in mouse
move post event. If app sets size of an widget in "drag,start" callback,
then it may cause mouse move event again in the mouse move post event
and the CRI log message is called. But since "drag,start" callback is
called only once, the mouse move event is not called recursively.

Change-Id: Id1aea1dd6916a4bd723564218acc17761556b9ad

4 years agoector: code refactoring. 31/213531/1
Hermet Park [Fri, 6 Sep 2019 08:41:52 +0000 (17:41 +0900)]
ector: code refactoring.

Current vector drawing requires several methods for compositing,
it's shouldnt have only masking but other blending functions
such as matte in lottie, porter&duff in android, etc.

Previously we didn't specify this method name,
just reserved the options and now we started to define the methods
with ector_renderer_composite_method() with Efl.Gfx.VG_Composite_Method
instead of mask_set().

Change-Id: I36235c23e1da6d3f138096a9425c6173eca93b88

4 years agoeldbus_proxy: Prevent dangling pointer. 96/213496/1
JunsuChoi [Fri, 6 Sep 2019 02:25:22 +0000 (11:25 +0900)]
eldbus_proxy: Prevent dangling pointer.

Change-Id: Iea7fbe36b983180a07a7e66a206fb6d7df21a625

4 years agoedje_edit: Prevent dangling pointer. 95/213495/1
JunsuChoi [Fri, 6 Sep 2019 02:08:59 +0000 (11:08 +0900)]
edje_edit: Prevent dangling pointer.

Change-Id: Id1b04acab9f48b762fada91f12a1b2c57b894a8d

4 years agocanvas 3d: remove potential dangling pointer. 92/213492/1
Hermet Park [Thu, 5 Sep 2019 09:09:36 +0000 (18:09 +0900)]
canvas 3d: remove potential dangling pointer.

Change-Id: I0815fd4da50838dd28a554fa9c3d151aa57e86ba

4 years agoethumb: fixed a potential dangling pointer. 91/213491/1
Hermet Park [Thu, 5 Sep 2019 09:04:07 +0000 (18:04 +0900)]
ethumb: fixed a potential dangling pointer.

fixed by null handling.

Change-Id: I890789f907eb6778b2c3ccb9a83eca5832c3870a

4 years agoefl.spec: Enable pulseaudio. 04/213404/3 accepted/tizen/unified/20190905.232221 submit/tizen/20190905.064609
Woochanlee [Thu, 5 Sep 2019 04:51:37 +0000 (13:51 +0900)]
efl.spec: Enable pulseaudio.

This is using in edc program(PLAY_SAMPLE) to play sound.

Change-Id: I34f3baa548a1e19225be2b817f601554e5e964c1
Signed-off-by: Woochanlee <wc0917.lee@samsung.com>
4 years agoector: fix a wrong target buffer size. 58/213358/1
Hermet Park [Wed, 4 Sep 2019 10:56:57 +0000 (19:56 +0900)]
ector: fix a wrong target buffer size.

the target buffer is allocated by backend engine,
The buffer size must be specified with the stride, not the width.
It's real dimension can be different with the our request.

Change-Id: Ia17f166176ce67ea03433102f48974b76f05b40b

4 years agoefl_ui_widget: reorder constructor code to fix event callback registration issue 39/213339/1
Yeongjong Lee [Wed, 4 Sep 2019 08:20:34 +0000 (17:20 +0900)]
efl_ui_widget: reorder constructor code to fix event callback registration issue

`callback_add` function should be called after `evas_object_callback_init` that
is called in `_efl_canvas_object_efl_object_constructor`. `callback_add` that is
called in `sub_object_add` didn't work.
This patch will fix that issue.

Note: it is needed more time to make solution that respects author's intent.
This is quick fix for only tizen which means this commit should be reverted
after fixing event callback registration issue in upstream.

@tizen_only

Change-Id: Id17bced316bb64e5bf4637f7f45ea66dc19327ab

4 years agovg_common_svg : Initialize "display" attribute 88/213288/2
JunsuChoi [Wed, 4 Sep 2019 03:08:56 +0000 (12:08 +0900)]
vg_common_svg : Initialize "display" attribute

Summary:
In e850e3e, the code to initialize this property is missing.
That makes the object invisible when printing svg through edje.
This is a patch to fix it.
@fix

Test Plan: N/A

Reviewers: Hermet, smohanty, kimcinoo

Reviewed By: Hermet

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

Change-Id: I36370ab055bb89ddb201d8a4763136bfccbf9ab2

4 years agofreetype: Prevent lose of data when fixed point divide calculation. 87/213287/2
JunsuChoi [Wed, 4 Sep 2019 03:07:08 +0000 (12:07 +0900)]
freetype: Prevent lose of data when fixed point divide calculation.

Summary:
In environments that long is 4byte, fixed-point division calculations will cause data loss.
fixed-point division need to more space.
Therefore, change all long types to long long types.

Test Plan: N/A

Reviewers: Hermet, kimcinoo, smohanty

Reviewed By: Hermet

Subscribers: vtorri, cedric, #reviewers, #committers

Tags: #efl

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

Change-Id: I473aacd8461bc0f02894836998fc7f81c8f5b531

4 years agoector gradient: ++ optmization. 79/213179/2
Hermet Park [Fri, 30 Aug 2019 08:13:31 +0000 (17:13 +0900)]
ector gradient: ++ optmization.

we know what size will be used for every span,
this graident buffer will be overwritten with new one,

So it doesn't need to clear every spans.

Change-Id: Ia65a442f45d87e4e1906755d8713989644ae9c98

4 years agoector: remove unnecessary blending processing. 78/213178/2
Hermet Park [Fri, 30 Aug 2019 04:42:54 +0000 (13:42 +0900)]
ector: remove unnecessary blending processing.

gradient buffer in masking doesn't need to apply anti-aliasing
since it would be taken by masking buffer.

This also fixes memory overflow case if the tbuffer is lack of size.

Change-Id: I88de96feb0d9d11fc14c5a24e4aacaef868ccfce

4 years agoector: code refactoring. 77/213177/2
Hermet Park [Thu, 29 Aug 2019 12:51:48 +0000 (21:51 +0900)]
ector: code refactoring.

just renamed internal variables, no logical changes.

Change-Id: I8c5ec644a61d5f99deca30d04c33e3b342030b6c

4 years agoedje/textstyle/optimization: Optimize code as textblock style handle logic changes. 80/213280/2
subhransu mohanty [Wed, 4 Sep 2019 00:51:50 +0000 (09:51 +0900)]
edje/textstyle/optimization: Optimize code as textblock style handle logic changes.

As tizen keeps a per object textblock style we don't
need to clear the file level textstyle as we never refer it. So cleanup the logic
which will be similar to opensource and performs much better than current logic.

Change-Id: I73905ee2e90a4e11496f902c04cbb7de9da38bb9