Bruno da Silva Belo [Mon, 23 Sep 2019 23:50:27 +0000 (20:50 -0300)]
efl-mono: Additional blank line between all methods.
Summary:
For better indentation, adding a blank line in the end of
methods.
Test Plan:
using efl_ui_layout.eo.cs, efl_access_object.eo.cs, efl_part.eo.cs and efl_ui_factory_bind.eo.cs as references,
apply changes to the code then check the diffs of after and before of the references files
Reviewers: lauromoura, felipealmeida
Reviewed By: lauromoura
Subscribers: cedric, felipealmeida, lauromoura
Tags: PHID-PROJ-uhnmnvlvunw6jgoqdnd4, #efl_language_bindings, #refactoring
Maniphest Tasks: T8167
Differential Revision: https://phab.enlightenment.org/D9712
Hosang Kim [Tue, 24 Sep 2019 03:48:08 +0000 (12:48 +0900)]
elm_conform: initialize variables for resizing part.
Change-Id: I21ef8eb27f6a2c37aaa5ae190d8869a1ce2ce4cb
JunsuChoi [Tue, 24 Sep 2019 02:52:30 +0000 (11:52 +0900)]
efl_gfx_path: Add optimized path command
Summary:
If 'L' is removed due to optimization, it should be supported like 3b1f7be
If the previous command is 'M', use 'L'.
Test Plan: N/A
Reviewers: Hermet, smohanty, kimcinoo
Reviewed By: Hermet
Subscribers: cedric, #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D10104
Change-Id: I97478dffc8c1612d3e60e905f4af37de0b6e5d9d
JunsuChoi [Tue, 24 Sep 2019 01:20:43 +0000 (10:20 +0900)]
efl_gfx_path: Support 'l' command for optimized path.
Summary:
If there is no 'l' command for path optimization, refer to the previous cmd.
If the previous command is 'm', use 'l'.
Test Plan:
[Example SVG]
'''
<svg xmlns="http://www.w3.org/2000/svg" version="1.1" viewBox="0 0 40 40">
<g fill="none" fill-rule="evenodd">
<rect width="40" height="40" fill="#f00" opacity="0"/>
<path d="m12 20 13-12-13 12zm0 0 13 12-13-12z" fill-rule="nonzero" opacity=".12" stroke="#fff" stroke-linecap="round" stroke-linejoin="round" stroke-width="2"/>
</g>
</svg>
'''
Reviewers: Hermet, smohanty, kimcinoo
Reviewed By: Hermet
Subscribers: cedric, #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D10074
Change-Id: I0e78949fcd0fdbff1d7a7314735319c2ce9924d4
Jaehyun Cho [Mon, 23 Sep 2019 08:06:20 +0000 (17:06 +0900)]
eo_mono: remove Efl.Access.Reading_Info_Type_Mask from workaround.cs
Since Efl.Access.Reading_Info_Type_Mask is defined in
efl_access_object.eo, the previous definition in workaround.cs is
removed.
Change-Id: I080a5c928318a7ba990d76d3e71cb9e831599414
WooHyun Jung [Tue, 24 Sep 2019 01:15:21 +0000 (10:15 +0900)]
efl_canvas_object: set default value to false for not breaking compatibility
@tizen_fix
Change-Id: I56e28fd87d92f5f6605297e24687f58cb4341184
Jongmin Lee [Mon, 23 Sep 2019 21:49:02 +0000 (06:49 +0900)]
build: resolve build error during migration
gl_generic module should be copmpiled prior to the other modules.
@tizen_fix
Change-Id: I8be4330792836b4ba18b7b0b2493d950950b0227
Xavi Artigas [Mon, 23 Sep 2019 20:51:17 +0000 (16:51 -0400)]
docs: Add documentation for Efl.Ui.Scroller
Summary:
These docs were missing.
Ref T7891
Test Plan: Only docs changed
Reviewers: zmike, woohyun, bu5hm4n
Reviewed By: zmike
Subscribers: cedric, #reviewers, #committers
Tags: #efl
Maniphest Tasks: T7891
Differential Revision: https://phab.enlightenment.org/D10019
Carsten Haitzler (Rasterman) [Mon, 23 Sep 2019 20:14:50 +0000 (21:14 +0100)]
osx - require full opengl for gl support on osx
the default still is gl-es as documented. you need to explicitly turn
on full opengl to get gl support on osx because thats all the engine
does. document this exception for osx (having gl magically being full
on osx and gle-es elsewhere is another option but then we need
different os specific defaults for this which is kind of bad in other
ways).
Xavi Artigas [Mon, 23 Sep 2019 20:09:36 +0000 (16:09 -0400)]
theme: add 'required' to efl/tab_pager group parts
Summary: ref T8231
Reviewers: zmike, cedric
Reviewed By: zmike
Subscribers: #reviewers, #committers
Tags: #efl
Maniphest Tasks: T8231
Differential Revision: https://phab.enlightenment.org/D10087
Marcel Hollerbach [Mon, 23 Sep 2019 20:03:26 +0000 (16:03 -0400)]
efl_ui_pack: allow NULL as existing parameter in after and before
Summary:
with this commit all implementations of Efl.Pack_Linear to permit NULL
as existing parameter, this is verified with a spec test unit.
fixes T8210
Reviewers: zmike, segfaultxavi
Reviewed By: zmike
Subscribers: woohyun, Jaehyun_Cho, YOhoho, segfaultxavi, zmike, cedric, #reviewers, #committers
Tags: #efl
Maniphest Tasks: T8210
Differential Revision: https://phab.enlightenment.org/D10023
Mike Blumenkrantz [Mon, 23 Sep 2019 20:09:22 +0000 (22:09 +0200)]
doc: wrap comment for EFL_UI_CLICKABLE_PART_BIND docs in edc
Reviewers: segfaultxavi
Reviewed By: segfaultxavi
Subscribers: segfaultxavi, cedric, #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D10089
Carsten Haitzler (Rasterman) [Mon, 23 Sep 2019 19:54:35 +0000 (20:54 +0100)]
ci - osx - switch to full gl option on osx
Xavi Artigas [Mon, 23 Sep 2019 19:27:50 +0000 (15:27 -0400)]
theme: add 'required' to efl/button:anchor group parts
Summary: ref T8231
Reviewers: zmike, cedric
Reviewed By: zmike
Subscribers: #reviewers, #committers
Tags: #efl
Maniphest Tasks: T8231
Differential Revision: https://phab.enlightenment.org/D10086
Lauro Moura [Mon, 23 Sep 2019 19:27:49 +0000 (15:27 -0400)]
tests: Initialize variables and quiet warning.
Summary: The switch does not cover all Align enum values.
Reviewers: zmike
Reviewed By: zmike
Subscribers: cedric, #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D10085
Xavi Artigas [Mon, 23 Sep 2019 19:27:47 +0000 (15:27 -0400)]
theme: add 'required' to efl/spin_button group parts
Summary: ref T8231
Reviewers: zmike, cedric
Reviewed By: zmike
Subscribers: #reviewers, #committers
Tags: #efl
Maniphest Tasks: T8231
Differential Revision: https://phab.enlightenment.org/D10084
Xavi Artigas [Mon, 23 Sep 2019 19:27:45 +0000 (15:27 -0400)]
theme: add 'required' to efl/timepicker parts
Summary: ref T8231
Reviewers: zmike, cedric
Reviewed By: zmike
Subscribers: #reviewers, #committers
Tags: #efl
Maniphest Tasks: T8231
Differential Revision: https://phab.enlightenment.org/D10083
Cedric Bail [Mon, 23 Sep 2019 19:27:34 +0000 (15:27 -0400)]
efl: improve test in checking the return value during Eina_Value get operation.
Summary: Depends on D10061
Reviewers: zmike, bu5hm4n, SanghyeonLee, segfaultxavi, felipealmeida, lauromoura
Reviewed By: bu5hm4n
Subscribers: #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D10062
Xavi Artigas [Mon, 23 Sep 2019 19:27:14 +0000 (15:27 -0400)]
docs: Clarify behavior of NULL in Efl.Pack_Linear.
Summary: Depends on D10023
Reviewers: zmike, bu5hm4n
Reviewed By: bu5hm4n
Subscribers: cedric, #reviewers, #committers
Tags: #efl
Maniphest Tasks: T8210
Differential Revision: https://phab.enlightenment.org/D10024
Carsten Haitzler (Rasterman) [Mon, 23 Sep 2019 17:59:27 +0000 (18:59 +0100)]
build - update README and meson options to match echother in defaults
update some defaults like move to gl-es by default, tslib off by
default as this really makes efl simpler to confgure ... by default
with less things to change to get stuff working.
Mike Blumenkrantz [Fri, 20 Sep 2019 17:38:08 +0000 (13:38 -0400)]
theme: add 'required' to efl/bg group parts
ref T8231
Reviewed-by: Cedric BAIL <cedric.bail@free.fr>
Differential Revision: https://phab.enlightenment.org/D10056
Mike Blumenkrantz [Fri, 20 Sep 2019 17:38:21 +0000 (13:38 -0400)]
theme: add 'required' to efl/popup (and related group) parts
ref T8231
Reviewed-by: Cedric BAIL <cedric.bail@free.fr>
Differential Revision: https://phab.enlightenment.org/D10057
Mike Blumenkrantz [Fri, 20 Sep 2019 17:56:23 +0000 (13:56 -0400)]
theme: add 'required' to efl/scroller group parts
ref T8231
Reviewed-by: Cedric BAIL <cedric.bail@free.fr>
Differential Revision: https://phab.enlightenment.org/D10058
Mike Blumenkrantz [Fri, 20 Sep 2019 17:59:32 +0000 (13:59 -0400)]
theme: add 'required' to efl/datepicker parts
ref T8231
Reviewed-by: Cedric BAIL <cedric.bail@free.fr>
Differential Revision: https://phab.enlightenment.org/D10059
Marcel Hollerbach [Sat, 21 Sep 2019 09:06:26 +0000 (11:06 +0200)]
efl_ui_position_manager: review cache accesses
in position manager grid and list, there are caches for the sizes, so
the access within placing the item is faster. The cache sometimes is
invalidated, which means, it must be rebuild before accessing it.
This commit is the result of reviewing all the accesses, that the
correct item is available.
Reviewed-by: Cedric BAIL <cedric.bail@free.fr>
Differential Revision: https://phab.enlightenment.org/D10065
Marcel Hollerbach [Sat, 21 Sep 2019 09:09:24 +0000 (11:09 +0200)]
efl_ui_focus_managr_sub: fix a err message
with this commit there are a few errors less, for the case a object is
destroyed while registered as a border element in the sub manager.
This adds a few more callbacks to the list of callbacks, however, i
could not messure a big performance impact.
Reviewed-by: Cedric BAIL <cedric.bail@free.fr>
Differential Revision: https://phab.enlightenment.org/D10067
Woochanlee [Mon, 23 Sep 2019 06:17:45 +0000 (06:17 +0000)]
ecore_thread: Fix arguments to match with the function call.
see the _eina_thread_internal() function
r = c->func((void*) c->data, eina_thread_self());
The second param has been missed in ecore_thread_worker, ecore_direct_worker functions.
Reviewed-by: Cedric BAIL <cedric.bail@free.fr>
Differential Revision: https://phab.enlightenment.org/D10073
Mike Blumenkrantz [Mon, 23 Sep 2019 14:06:26 +0000 (10:06 -0400)]
build: fix EFL_VERSION_MICRO define
this was never pulled in from the build system and thus was never actually
defined
Reviewed-by: Cedric BAIL <cedric.bail@free.fr>
Differential Revision: https://phab.enlightenment.org/D10078
Mike Blumenkrantz [Mon, 23 Sep 2019 13:30:03 +0000 (09:30 -0400)]
efl_ui/layout: move 'frozen' struct member to be a bool bitflag
this is already handled by edje, all we really want here is a flag
to avoid more eo lookups internally when calling canvas_group_change
Reviewed-by: Cedric BAIL <cedric.bail@free.fr>
Differential Revision: https://phab.enlightenment.org/D10079
Lauro Moura [Mon, 23 Sep 2019 16:33:52 +0000 (13:33 -0300)]
cxx: Fix some warnings from -Wextra
Summary:
- As we don't have C++17's [[fallthrough]], rely on GCC'd detection
of fallthrough comments.
See https://developers.redhat.com/blog/2017/03/10/wimplicit-fallthrough-in-gcc-7/
- Replaced some copy constructors with typecast operators.
Previously, a constructor with remove_const/remove_cv were used
to allow const iterators to be constructed from non-const iterators.
This had the side effect of making these constructors actual copy
constructors for non const lists. As we did not define other
special constructors/operators, the rule of 5 were violated for these
cases.
This commit replaces these constructors with actual typecast operators
that promote non const iterators to their const counterparts.
- Cast a Eina_Bool/bool narrowing conversion
- Add a missing break statement from D10050
Reviewers: zmike, brunobelo, felipealmeida
Reviewed By: brunobelo
Subscribers: cedric, #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D9934
Xavi Artigas [Mon, 23 Sep 2019 16:05:07 +0000 (18:05 +0200)]
docs: Efl.Ui.Widget_Scroller_Content updated docs
Summary: Ref T8254
Reviewers: zmike
Reviewed By: zmike
Subscribers: cedric, #reviewers, #committers
Tags: #efl
Maniphest Tasks: T8254
Differential Revision: https://phab.enlightenment.org/D10082
Vincent Torri [Mon, 23 Sep 2019 14:00:25 +0000 (14:00 +0000)]
eina_array: do not mix declaration and definition
fix warning when used in C90 code.
Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org>
Differential Revision: https://phab.enlightenment.org/D10077
Wonki Kim [Fri, 20 Sep 2019 05:26:41 +0000 (05:26 +0000)]
gesture: fix it not to lose information by dividing
division of integer by integer results integer.
this patch fix the case to make a division result more precise.
Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com>
Differential Revision: https://phab.enlightenment.org/D10039
Mike Blumenkrantz [Fri, 20 Sep 2019 16:51:18 +0000 (12:51 -0400)]
theme: efl/photocam -> efl/image_zoomable
the theme group for a class should match its class name
ref T8231
Reviewed-by: Xavi Artigas <xavierartigas@yahoo.es>
Differential Revision: https://phab.enlightenment.org/D10052
Mike Blumenkrantz [Fri, 20 Sep 2019 17:01:04 +0000 (13:01 -0400)]
theme: remove dead popup theme groups
these widgets were removed
ref T8231
Reviewed-by: Xavi Artigas <xavierartigas@yahoo.es>
Differential Revision: https://phab.enlightenment.org/D10054
Marcel Hollerbach [Mon, 23 Sep 2019 12:11:51 +0000 (14:11 +0200)]
eina_log: ensure that we do not spam journald with our intree binaries
EFL_RUN_IN_TREE is set for all binaries that are executed during build
time, and test suites. With this commit we are ensuring that we are not
sending messages during that time to journald, otherwise we would waste
a lot of time + we would create a lot of unneccessary log messages.
(eina test suite runtime with journald hook: 11.21s without journald
hook: 0.57s)
Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com>
Differential Revision: https://phab.enlightenment.org/D10076
Daniel Kolesa [Mon, 23 Sep 2019 13:26:23 +0000 (15:26 +0200)]
eolian: allow read-only stringshare in events
While the type cannot be moved to the callee, it at least provides
a hint that it's a stringshare, leeting people ref it instead of
copying.
Mike Blumenkrantz [Thu, 19 Sep 2019 13:28:57 +0000 (09:28 -0400)]
ci: add back target to do out-of-tree test build
this is necessary in order to ensure that our headers are usable
ref T8228
Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org>
Differential Revision: https://phab.enlightenment.org/D10036
Jaehyun Cho [Thu, 19 Sep 2019 09:49:39 +0000 (09:49 +0000)]
elementary: unref objects when memory allocation fails in factory
The objects referenced are unreferenced when memory allocation fails in
elementary factory.
Differential Revision: https://phab.enlightenment.org/D9968
Hermet Park [Mon, 23 Sep 2019 12:12:07 +0000 (21:12 +0900)]
ector engine: +null checking.
Null pixels is allowed when drawing region is 0,
In that case, the pixel buffer can be destroyed earlier by vg.
@fix
Jaehyun Cho [Mon, 23 Sep 2019 10:37:10 +0000 (19:37 +0900)]
efl_ui_tab_pager: fix infinite selected calls
Since tab_pager's _select() and tab_bar's _selection_change_cb() cause
calling each other infinitely.
Mike Blumenkrantz [Mon, 23 Sep 2019 10:56:11 +0000 (12:56 +0200)]
elm_test: mark image_zoomable test as non-legacy
Reviewers: segfaultxavi
Reviewed By: segfaultxavi
Subscribers: cedric, #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D10053
Marcel Hollerbach [Mon, 23 Sep 2019 07:42:22 +0000 (09:42 +0200)]
eina_log: do not fallback to INFO when we run with systemd support
INFO is a way too verbose log domain, this lead to the fact that we
spammed journald before the first frame with ~200-300 messages
(depending on the setup) which is quite a lot. Additionally, this also
lead to the fact that we often TIMEOUT our own tests.
Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org>
Differential Revision: https://phab.enlightenment.org/D10075
Stefan Schmidt [Fri, 20 Sep 2019 10:11:39 +0000 (12:11 +0200)]
elementary_codegen: check before use to avoid use after free
These might or might not have been freed already, depending on the
breaks form the above loop. Before we are having a use after free here
better check on it before free'ing another time.
Coverity IDs: 1402671, 1402701
Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com>
Differential Revision: https://phab.enlightenment.org/D10047
Stefan Schmidt [Fri, 20 Sep 2019 09:19:51 +0000 (11:19 +0200)]
efl_ui_exact_model: fix always true condition
The tbuf binbuffer has already been freed the line above, which means
this condition would always be true as !tbuf would always result in true.
As a result the rest of the function has never been executed.
Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com>
Differential Revision: https://phab.enlightenment.org/D10046
Stefan Schmidt [Fri, 20 Sep 2019 08:46:02 +0000 (10:46 +0200)]
efl_ui_selection_manager: free allocated memory after local use
This temporary allocation is only used in this block and goes out of
scope when leaving. Make sure we free the resource before this happens.
Coverity ID: 1402653
Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com>
Differential Revision: https://phab.enlightenment.org/D10045
Stefan Schmidt [Fri, 20 Sep 2019 08:36:17 +0000 (10:36 +0200)]
efl_ui_textpath: make sure variables are not uses uninitialized
First and last could have been uninitialized for their first use in some
cases. Make sure we set the x y coordinates to 0 to begin with.
Coverity IDs: 1401458, 1404747
Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com>
Differential Revision: https://phab.enlightenment.org/D10044
WooHyun Jung [Mon, 23 Sep 2019 08:48:02 +0000 (10:48 +0200)]
efl_ui_scrollable: remove ui_scrollable_interactive and move all into ui_scrollable
Summary:
Based on the discussion T7883, there is no reason to keep those
interfaces separately. So, now, efl_ui_scrollable will support
the events and methods together.
@ref T7883
Reviewers: eagleeye, bu5hm4n, zmike, segfaultxavi
Reviewed By: segfaultxavi
Subscribers: cedric, #reviewers, #committers
Tags: #efl
Maniphest Tasks: T7883
Differential Revision: https://phab.enlightenment.org/D10072
Wonki Kim [Sat, 21 Sep 2019 04:28:24 +0000 (04:28 +0000)]
test: Add a logic to check a size of Check
If I pack a check without any api calls into a box,
then it doesn't have the hint_min size which is defined on check edc.
I guess there are problems on sizing logic currently, so this patch helps people
to test the problem.
Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D10038
Mike Blumenkrantz [Fri, 20 Sep 2019 13:04:16 +0000 (09:04 -0400)]
elm/layout: remove check for finalize in efl_canvas_group_change impl
this used to be a thing in order to prevent widgets from spamming themselves
with recalcs during construction, but since that's no longer possible we
can remove this and also resolve an issue where legacy widgets failed
to correctly calculate their min size if no text was set
Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D10043
Ross Vandegrift [Sun, 22 Sep 2019 23:30:25 +0000 (23:30 +0000)]
Use EFL version for doc tarball, not meson version
Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D10070
Xavi Artigas [Mon, 23 Sep 2019 08:41:46 +0000 (10:41 +0200)]
docs: Typos in Efl.Object
SangHyeon Jade Lee [Fri, 20 Sep 2019 01:40:01 +0000 (10:40 +0900)]
efl_ui : fix possible memory leak on model compressing
Summary: fix memory leak of model compressing on binbuf and allocated data.
Reviewers: cedric, bu5hm4n, zmike
Reviewed By: cedric, zmike
Subscribers: #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D10037
SangHyeon Jade Lee [Tue, 27 Aug 2019 01:21:36 +0000 (10:21 +0900)]
genlist: skip unnecessary process call while genlist clearing
item_process is calling while genlist clearing when
the item is relatvely added.
skiping unnecessary process calls improve deletion performances.
Change-Id: Ic12c4ea190799a79fb2f369569b453d5061f8019
Signed-off-by: SangHyeon Jade Lee <sh10233.lee@samsung.com>
SangHyeon Jade Lee [Fri, 20 Sep 2019 01:55:26 +0000 (10:55 +0900)]
elementary: fix derefrenced pointer accessing on adjusting item
Change-Id: I2392de287318168c1097aa6826e0a54190bb8b3a
Signed-off-by: SangHyeon Jade Lee <sh10233.lee@samsung.com>
SangHyeon Jade Lee [Fri, 20 Sep 2019 01:51:09 +0000 (10:51 +0900)]
elementary: prevent zero-division in gengrid internal circle patch position calculate
Change-Id: I130deac7a6b04d1bdb07aae1f51f10508162e4ee
Signed-off-by: SangHyeon Jade Lee <sh10233.lee@samsung.com>
Jongmin Lee [Sun, 22 Sep 2019 21:22:39 +0000 (06:22 +0900)]
build: resolve build error during migration
Change-Id: Id3d2437e84e3dbbebc45ff784f7e4b6345be7dc3
Marcel Hollerbach [Sat, 21 Sep 2019 09:08:46 +0000 (11:08 +0200)]
efl_ui_collection: protect against invalid access
i am not sure why this should ever happen. However, a error here is
better than a access out of the bounds of the array.
Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com>
Differential Revision: https://phab.enlightenment.org/D10066
Marcel Hollerbach [Sat, 21 Sep 2019 08:15:18 +0000 (10:15 +0200)]
efl_ui_bg: implement load_controller and efl.gfx.image via composition
With this commit the API of load controller and efl.gfx.image is
implemented via composition, which makes eolians API checker happy
ref T7880
Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com>
Differential Revision: https://phab.enlightenment.org/D10064
Marcel Hollerbach [Sat, 21 Sep 2019 15:53:53 +0000 (17:53 +0200)]
efl_gfx_image: remove ptr()
this is not needed here, and not permitted in a none beta context.
Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com>
Differential Revision: https://phab.enlightenment.org/D10069
Marcel Hollerbach [Sat, 21 Sep 2019 15:37:07 +0000 (17:37 +0200)]
efl_ui_image: implement Efl.Gfx.Image and Efl.Gfx.Image_Load_Controller
at first i tried to implement that with composition, however, this does
not work, caused by the fact that the edje_object does not implement
efl.gfx.image nor load controller. which would mean i would have to
differentiate based on the type what kind of composition i would attach,
additionally, i would have to write an additional object which would
implement the correct behaviour.
To shortcut this overdone approach, this commit simply implements the
APIs in the image object.
Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com>
Differential Revision: https://phab.enlightenment.org/D10068
Marcel Hollerbach [Sat, 21 Sep 2019 11:10:53 +0000 (13:10 +0200)]
elementary: make our tests work again
73219114682f64fdbfe0809be3f2d991016dd3f1 moved the constructor to to
top, which leads to the fact that a few widgets cannot fetch the shared
data of the window. Which made the test suite fail.
With this commit we are just moving the sub_object_add after the
constructor, which fixes the flip crash.
This fixes the test suite
Carsten Haitzler (Rasterman) [Sat, 21 Sep 2019 09:26:46 +0000 (10:26 +0100)]
efl ui widget - call parent constructor before doing more init
the object is not properly set up yet for child classes -0 parent like
the main evas object or smart obj havent been constructed yet so call
this really early, not in the middle of construction of an efl ui
widget...
this fixes a segv in elementary_test -to filp where it segv's on
object construction where evas object ->cur/prev are NULL (not set up
yet).
Lauro Moura [Fri, 20 Sep 2019 22:22:22 +0000 (19:22 -0300)]
csharp: Guard hash usage behind EFL_BETA.
Summary: After
718f3cd49586, hashes are currently beta-only.
Reviewers: zmike, brunobelo
Reviewed By: brunobelo
Subscribers: cedric, #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D10060
Jaehyun Cho [Fri, 20 Sep 2019 22:14:02 +0000 (19:14 -0300)]
eolian_mono: add more verbs to generate C# method beginning with verb
Summary:
C# language binding generates C# method name beginning with verb by
checking verb list in name_helpers.hh.
Some verbs used in eo method are missing in the current verb list. So
they are added into the verb list.
In this scheme, there is a problem that some words can be used as a verb
in some methods but they also can be used as a noun in other methods.
(e.g. drag_start as a verb / cursor_word_start as a noun)
So "start" and "scroll" are not added to verb list in this patch.
To resolve the above problem, we may need a new eo syntax to rename or
reverse the name in language bindings.
Test Plan: Check C# method names in .eo.cs files if the name begins with verb.
Reviewers: felipealmeida, SanghyeonLee, woohyun, Jaehyun_Cho, segfaultxavi
Reviewed By: segfaultxavi
Subscribers: cedric, #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D10001
Jongmin Lee [Fri, 20 Sep 2019 21:26:32 +0000 (06:26 +0900)]
Revert "build: resolve build error"
This reverts commit
11d6f49658f44a2ad09a76fa28095077a1b438a1.
Daniel Kolesa [Fri, 20 Sep 2019 16:02:07 +0000 (18:02 +0200)]
eolian: rename any_value_ptr -> any_value_ref for consistency
Daniel Kolesa [Fri, 20 Sep 2019 15:46:56 +0000 (17:46 +0200)]
eolian: string(share) is always const, allow in events out of box
Daniel Kolesa [Fri, 20 Sep 2019 15:03:58 +0000 (17:03 +0200)]
eolian: make hashes beta-only for now
They're not completely settled on and we don't even know if we want
them at all. So let's make them beta only for now.
Ref T8050.
Xavi Artigas [Fri, 20 Sep 2019 14:38:18 +0000 (11:38 -0300)]
Use proper Eolian syntax for default values instead of docs
Summary:
Eolian supports reporting the defaults for parameters and return values, but in some
places we have been writing this information in the documentation instead.
This patch moves it to its proper place, where documentation generators can pick it up
and render it in a consistent manner.
Ref T8171
Reviewers: zmike, bu5hm4n, lauromoura
Reviewed By: lauromoura
Subscribers: cedric, #reviewers, #committers
Tags: #efl
Maniphest Tasks: T8171
Differential Revision: https://phab.enlightenment.org/D10051
Lauro Moura [Fri, 20 Sep 2019 14:05:01 +0000 (16:05 +0200)]
csharp: Add support for default param references in documentation
Summary:
This also adds the information of default values in `parameter_def`.
This will help adding actual default parameter support in the C#
bindings.
Thanks to @segfaultxavi for improved formatting and escaping.
Fixes T8224
Reviewers: segfaultxavi, felipealmeida, brunobelo
Reviewed By: segfaultxavi
Subscribers: cedric, #reviewers, segfaultxavi, #committers
Tags: #efl
Maniphest Tasks: T8224
Differential Revision: https://phab.enlightenment.org/D10050
Marcel Hollerbach [Fri, 20 Sep 2019 13:16:25 +0000 (15:16 +0200)]
efl_ui_factory: improve docs
Summary:
and remove TAAABS
Depends on D10038
Reviewers: segfaultxavi
Reviewed By: segfaultxavi
Subscribers: cedric, #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D10042
Hosang Kim [Fri, 20 Sep 2019 10:42:50 +0000 (12:42 +0200)]
efl.ui.scrollbar: fix event name for scrollbar
Summary: Event name is applied in the past tense.
Reviewers: segfaultxavi
Reviewed By: segfaultxavi
Subscribers: cedric, #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D10040
Stefan Schmidt [Fri, 20 Sep 2019 07:35:36 +0000 (09:35 +0200)]
build: switch build type to release for now
This should have been done with the first alpha, but there was a bug
preventing this. With commit
6960751c4580af9e902cd7719c1acafab4ce7bc5
the problem is fixed and we can finally enable it.
WooHyun Jung [Fri, 20 Sep 2019 02:09:02 +0000 (11:09 +0900)]
efl_ui_scroll_util: add null check after calloc
Marcel Hollerbach [Wed, 18 Sep 2019 09:32:08 +0000 (11:32 +0200)]
efl_io_model: next try to fix this race condition
what is happening is that a file gets announced through eio_model
listing code, at this point of time, the monitor does not yet know about
the file. If the file now gets deleted between the annoncing and the
learning of the file from the monitor, then the file got an ADD event,
but no DEL event. Which is a bug.
With this commit there is a new API which asks the monitor if the file
already has the knowledge about the files existance, or not. A few
monitors like win32 inotify or cocoa do not have context about the file
directly, if the OS is now having the same bug, then we are again in
trouble, however, we canot do anything about that. In the case of kevent
or poll, this asks the context of the monitor if the file is already
there.
Reviewed-by: Cedric BAIL <cedric.bail@free.fr>
Differential Revision: https://phab.enlightenment.org/D10006
Cedric BAIL [Fri, 23 Aug 2019 21:29:55 +0000 (14:29 -0700)]
elementary: Efl.Ui.Widget_Factory bind widget property before finalizing the widget.
In the same vain as previous patch this will initialize more of the widget during its
creation and reduce unecessary recalc.
Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D9949
Cedric BAIL [Fri, 23 Aug 2019 19:48:16 +0000 (12:48 -0700)]
elementary: leverage sizing information from model if available to avoid unecessary calc.
With the new Efl unified infrastructure, we do delay a lot of the computation to finalize,
by filling the object information before finalize we reduce unecessary computation.
Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com>
Differential Revision: https://phab.enlightenment.org/D9948
Marcel Hollerbach [Sun, 15 Sep 2019 09:54:51 +0000 (11:54 +0200)]
efl_ui_position_manager: a way to announce new entities
there are situations where the entity is not ready yet when the initial
placing does happen. With this API you can tell the position manager
that the placing of the items can be reapplied at the entities are
availble now.
Differential Revision: https://phab.enlightenment.org/D9947
Cedric Bail [Thu, 19 Sep 2019 02:30:17 +0000 (19:30 -0700)]
efl: improve Efl.Boolean_Model test to enforce Eina_Value type too.
Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com>
Differential Revision: https://phab.enlightenment.org/D10031
Cedric Bail [Wed, 18 Sep 2019 17:33:23 +0000 (10:33 -0700)]
ecore: remove unused piece of code.
Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com>
Differential Revision: https://phab.enlightenment.org/D10030
Cedric Bail [Wed, 18 Sep 2019 02:03:48 +0000 (19:03 -0700)]
ecore: properly initialize all field in event generated by Efl.Generic_Model
Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com>
Differential Revision: https://phab.enlightenment.org/D10029
Cedric Bail [Wed, 18 Sep 2019 01:42:51 +0000 (18:42 -0700)]
ecore: properly initialize the event structure in Efl.Generic_Model.
Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com>
Differential Revision: https://phab.enlightenment.org/D10028
Cedric Bail [Sun, 15 Sep 2019 02:12:44 +0000 (19:12 -0700)]
eina: add code to help debug leaking Eina_Value.
Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com>
Differential Revision: https://phab.enlightenment.org/D9942
Cedric Bail [Sat, 14 Sep 2019 17:54:05 +0000 (10:54 -0700)]
eina: add eina_mempool_iterator_new to slowly iterate every allocated pointer in a mempool.
Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com>
Differential Revision: https://phab.enlightenment.org/D9941
Marcel Hollerbach [Wed, 18 Sep 2019 12:59:44 +0000 (14:59 +0200)]
efl_ui_layout_part_table: implement missing API
setting of these properties does not work, as they are not available in
legacy, but edje is still creating legacy objects.
Reviewed-by: Cedric BAIL <cedric.bail@free.fr>
Differential Revision: https://phab.enlightenment.org/D10007
Marcel Hollerbach [Tue, 17 Sep 2019 14:41:49 +0000 (16:41 +0200)]
efl_ui_layout_part_*: declare stable
Reviewed-by: Cedric BAIL <cedric.bail@free.fr>
Differential Revision: https://phab.enlightenment.org/D9995
Marcel Hollerbach [Tue, 17 Sep 2019 13:59:07 +0000 (15:59 +0200)]
efl_ui_widget_*: declare parts stable
Reviewed-by: Cedric BAIL <cedric.bail@free.fr>
Differential Revision: https://phab.enlightenment.org/D9994
Marcel Hollerbach [Tue, 17 Sep 2019 14:39:55 +0000 (16:39 +0200)]
efl_ui_layout_part_box/table: remove real_part_set
real part set was used to set the real part on the part, so the obj
pointer of Layout_Part_Data points to the correct evas object. With this
commit, this API is removed from the public .eo files, and is refactored
into the same API which is private.
Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com>
Differential Revision: https://phab.enlightenment.org/D9993
Marcel Hollerbach [Tue, 17 Sep 2019 14:25:37 +0000 (16:25 +0200)]
efl_ui_layout_part_table: implement Efl.Pack.pack API
this api was missing, the pack method here is behaving in the same
manner as Efl.Ui.Table.
Reviewed-by: Cedric BAIL <cedric.bail@free.fr>
Differential Revision: https://phab.enlightenment.org/D9992
Marcel Hollerbach [Tue, 17 Sep 2019 13:51:54 +0000 (15:51 +0200)]
efl_ui_widget_part: implement all the missing API
Efl.Color and Efl.File cannot be done due to composition. Efl.Gfx.Image
can, so we can solve this via composition, this also resolves all the
other cases for Efl.Gfx.Image.
Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com>
Differential Revision: https://phab.enlightenment.org/D9991
Marcel Hollerbach [Wed, 18 Sep 2019 15:42:28 +0000 (17:42 +0200)]
efl_test_multi_selectable: test for the correct event
EFL_UI_EVENT_ITEM_SELECTED / UNSELECTED is the wrong event here. We
should check of the selection changed event. ITEM_SELECTED is the legacy
event. this was not found based on the fact that we test for this event
to NOT be executed.
Reviewed-by: Cedric BAIL <cedric.bail@free.fr>
Differential Revision: https://phab.enlightenment.org/D10011
Marcel Hollerbach [Thu, 19 Sep 2019 12:43:31 +0000 (14:43 +0200)]
elm_test_widget_focus: cleanup callbacks correctly
we need to delete the callback here, otherwise the objects are deleted
later on, which:
- changes focus, which leads to
- callbacks executed with stacked addresses from a dead function
this fixes a testsuite failure on the release build.
Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com>
Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org>
Differential Revision: https://phab.enlightenment.org/D10020
Yeongjong Lee [Tue, 17 Sep 2019 09:15:05 +0000 (09:15 +0000)]
efl_ui_collection: fix null pointer dereferences
This will fix Coverity issue.
Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com>
Differential Revision: https://phab.enlightenment.org/D9986
Marcel Hollerbach [Thu, 19 Sep 2019 07:27:59 +0000 (09:27 +0200)]
eo: custom created legacy events should be unfreezable
a while back we have moved event submission from custom lists to eo
events. We also merged together the freeze and thaw functions, however,
this brought up one bug. smart_callbacks and the likes (those that used
custom lists before) are not effected by the legacy freeze and thaw
functions. This means, we should declare these legacy wrapper objects
unfreezable in order to obtain this behaviour.
Reviewed-by: Cedric BAIL <cedric.bail@free.fr>
Reviewed-by: SangHyeon Jade Lee <sh10233.lee@samsung.com>
Differential Revision: https://phab.enlightenment.org/D10016
Mike Blumenkrantz [Thu, 19 Sep 2019 14:54:47 +0000 (10:54 -0400)]
efl_ui: further unbreak headers
move more header includes around to fix building with Efl_Ui.h
ref T8228
Reviewed-by: Cedric BAIL <cedric.bail@free.fr>
Differential Revision: https://phab.enlightenment.org/D10022
Mike Blumenkrantz [Thu, 19 Sep 2019 18:52:23 +0000 (14:52 -0400)]
ecore/signal: increase maximum signal throughput
this adds 4 more signal handling fds and loops over them for reading/writing
signal info in order to handle more signals when the buffer of one (or more)
pipes is full
also update the unit test to verify that we are receiving all the events without
dropping any and bump the number of signals to 2000 since we should now be able to
handle that many
Reviewed-by: Cedric BAIL <cedric.bail@free.fr>
Differential Revision: https://phab.enlightenment.org/D10027
Mike Blumenkrantz [Thu, 19 Sep 2019 18:15:45 +0000 (14:15 -0400)]
tests/ecore: add stress test for main loop signal handling
this explodes after about 500 signals, so make it 1000 for a good test
Reviewed-by: Cedric BAIL <cedric.bail@free.fr>
Differential Revision: https://phab.enlightenment.org/D10026
Mike Blumenkrantz [Thu, 19 Sep 2019 18:03:59 +0000 (14:03 -0400)]
ecore/signal: also use nonblock for writing side of signal pipe
if any efl-based process receives a bunch of signals in a short period of
time, it will deadlock in the signal handler. this is unavoidable given the
current signal handling architecture
by setting nonblock, we can at least avoid deadlocking even if it means we'll
be losing signal events
@fix
Reviewed-by: Cedric BAIL <cedric.bail@free.fr>
Differential Revision: https://phab.enlightenment.org/D10025
Changyeon Lee [Mon, 16 Sep 2019 12:31:52 +0000 (21:31 +0900)]
ecore_evas_tbm: set tbm_surface_queue to ecore window
Change-Id: I598666204d347c84a7af233865addf94bb07a868