Godly Thekkethottiyil Alias [Thu, 6 Feb 2020 15:01:34 +0000 (20:31 +0530)]
[Scrollable] Go To top Feature
[Model] Noblesse
[BinType] N/A
[Customer] N/A
[Issue#] FWGOVERSEA-1775
[Request] UX
[Occurrence Version] N/A
[Problem] N/A
[Cause & Measure] N/A
[Checking Method] N/A
[Team] (UIFW/EFL)
[Developer] Godly T.Alias
[Solution company] Samsung
[Change Type] New Feature
Change-Id: I6ea0c4130271709906cac840f27ef0afd7ad9f50
Signed-off-by: Godly Thekkethottiyil Alias <godly.talias@samsung.com>
Godly Thekkethottiyil Alias [Tue, 28 Jan 2020 13:40:33 +0000 (19:10 +0530)]
[Scrollable] Fix for clipper on go to top button
[Model] Noblesse
[BinType] N/A
[Customer] N/A
[Issue#] FWGOVERSEA-1520
[Request] UX
[Occurrence Version] N/A
[Problem] N/A
[Cause & Measure] Clipper added in edc creates flickering
Moving clipper addition to elementary
[Checking Method] N/A
[Team] (UIFW/EFL)
[Developer] Godly T.Alias
[Solution company] Samsung
[Change Type] New Feature
Change-Id: Ia3672265029226e3f880ba9704c8228b2a430337
Signed-off-by: Godly Thekkethottiyil Alias <godly.talias@samsung.com>
Godly Thekkethottiyil Alias [Tue, 28 Jan 2020 10:42:23 +0000 (16:12 +0530)]
[Scrollable] Go to top hook function
[Model] Noblesse
[BinType] N/A
[Customer] N/A
[Issue#] FWGOVERSEA-1520
[Request] N/A
[Occurrence Version] N/A
[Problem] N/A
[Cause & Measure] N/A
[Checking Method] N/A
[Team] (UIFW/EFL)
[Developer] Godly T.Alias
[Solution company] Samsung
[Change Type] New Feature
Change-Id: I62fd1fc0d27b5a830a5f142924af696ed692c418
Signed-off-by: Godly Thekkethottiyil Alias <godly.talias@samsung.com>
Godly Thekkethottiyil Alias [Thu, 23 Jan 2020 15:52:38 +0000 (21:22 +0530)]
[Scrollable Interface] Go To Top support
[Model] N/A
[BinType] N/A
[Customer] N/A
[Issue#] FWGOVERSEA-1520
[Request] N/A
[Occurrence Version] N/A
[Problem] Scroll To Top button for scroller
[Cause & Measure] N/A
[Checking Method] N/A
[Team] (UIFW/EFL)
[Developer] Godly T.Alias
[Solution company] Samsung
[Change Type] New feature
Change-Id: I5e8ac5f8f91bc0f635f1e79d0efdc44b9ca9d600
Signed-off-by: Godly Thekkethottiyil Alias <godly.talias@samsung.com>
Taehyub Kim [Thu, 13 Feb 2020 07:14:17 +0000 (16:14 +0900)]
elm_conform: fix casting warning for elm_win_wl_window_get API
Change-Id: Ib5cd3ba6c7bd50fe68136e4a56aafaff0c9e83e8
Woochanlee [Thu, 13 Feb 2020 10:25:27 +0000 (19:25 +0900)]
ecore_w2: Move some Ecore_Wl2 APIs to be internal
This commit to prepare ecore_wl2 APIs public work.
@tizen_only
Change-Id: I37cb82be7bd8d9bea24e91bf4f7340d6db6a8297
Taehyub Kim [Thu, 13 Feb 2020 06:38:18 +0000 (15:38 +0900)]
tizen_vector: remove unused variables for progressbar style
Change-Id: I2daaaf3a7c31d7971c1db8c055500e094af4d120
Taehyub Kim [Thu, 13 Feb 2020 06:47:23 +0000 (15:47 +0900)]
elm_entry: remove unused static _entry_selection_changed_signal_job_cb function
Change-Id: Id52905ccbcebdfd4a157c8d1f84adf9cba6ca7e5
Yeongjong Lee [Thu, 13 Feb 2020 10:01:48 +0000 (19:01 +0900)]
Evas : remove tizen only code to fix build erros related to beta APIs
This reverts commit
5b5e498929b81a71d696ecfd5a016066286d88fc.
Change-Id: Idb42cb78d4d88204b8e756837c9bee6aa79d5b8c
Taehyub Kim [Thu, 13 Feb 2020 07:34:30 +0000 (16:34 +0900)]
elm_toolbar: fix warning to compare integer and enum value
Change-Id: Ic662065109f32131f60731377afc8e759cd4b679
Hermet Park [Thu, 13 Feb 2020 10:16:01 +0000 (19:16 +0900)]
canvas textblock: + null handling
Change-Id: I661d1f47579cc9c5a143464abb4591b53c43a830
Hermet Park [Thu, 13 Feb 2020 10:05:43 +0000 (19:05 +0900)]
canvas textblock: + null safety
Change-Id: I43811fa5d80b6be8621172300d847813d5829d37
Hermet Park [Thu, 13 Feb 2020 09:53:33 +0000 (18:53 +0900)]
ui text: code refactoring
replace unreacheable code with EINA_UNUSED properly
Change-Id: I3efabb0a66895b121b45ee4a243640d4fb3808c7
Hermet Park [Thu, 13 Feb 2020 07:48:57 +0000 (16:48 +0900)]
canvas font: correct wrong masking length.
the masking length should be x2 - x1 otherwise,
it will try to mask over regions with dirty pixels.
@fix
Change-Id: If82fa94da99a24856bb3e177bf698c8a53c62d5f
Hermet Park [Wed, 12 Feb 2020 12:09:36 +0000 (21:09 +0900)]
edje marquee: remove useless code.
Change-Id: I957c42dfa86ab7b95f32965028cf054fb63dadf1
Yeongjong Lee [Wed, 12 Feb 2020 10:18:08 +0000 (19:18 +0900)]
evas_gl_texture: change tex_palette type to GLuint from GLint
Fixes compiler warnings
```
../src/modules/evas/engines/gl_common/evas_gl_texture.c: In function 'evas_gl_common_texture_update':
../src/modules/evas/engines/gl_common/evas_gl_texture.c:1478:35: warning: pointer targets in passing argument 4 of 'evas_glGenTextures_thread_cmd' differ in signedness [-Wpointer-sign]
1478 | GL_TH(glGenTextures, 1, &(tex->tex_palette));
```
```
../src/modules/evas/engines/gl_common/evas_gl_thread_gl_generated.h:67:66: note: expected 'GLuint *' {aka 'unsigned int *'} but argument is of type 'GLint *' {aka 'int *'}
67 | extern void GL_TH_FN(glGenTextures)(GL_TH_DP, GLsizei n, GLuint* textures);
```
@tizen_fix
Change-Id: Ib3ee6e34ff14720a31a56edb096ac967be9b6bbf
Yeongjong Lee [Wed, 12 Feb 2020 09:52:34 +0000 (18:52 +0900)]
evas_gl_texture: add missing type declaration
Fixes compiler warings.
```
[ 69s] ../src/modules/evas/engines/gl_common/evas_gl_texture.c:56:14: warning: type defaults to 'int' in declaration of 'palette_ifmt' [-Wimplicit-int]
[ 69s] 56 | static const palette_ifmt = GL_ALPHA;
[ 69s] | ^~~~~~~~~~~~
[ 69s] ../src/modules/evas/engines/gl_common/evas_gl_texture.c:57:14: warning: type defaults to 'int' in declaration of 'palette_fmt' [-Wimplicit-int]
[ 69s] 57 | static const palette_fmt = GL_ALPHA;
[ 69s] | ^~~~~~~~~~~
[ 69s] ../src/modules/evas/engines/gl_common/evas_gl_texture.c:105:52: warning: pointer targets in initialization of 'const GLenum *' {aka 'const unsigned int *'} from 'const int *' differ in signedness [-Wpointer-sign]
[ 69s] 105 | { MATCH_ANY, MATCH_ANY, EVAS_COLORSPACE_PALETTE, &palette_ifmt, &palette_fmt }
[ 69s] | ^
[ 69s] ../src/modules/evas/engines/gl_common/evas_gl_texture.c:105:52: note: (near initialization for 'matching_format[17].intformat')
[ 69s] ../src/modules/evas/engines/gl_common/evas_gl_texture.c:105:67: warning: pointer targets in initialization of 'const GLenum *' {aka 'const unsigned int *'} from 'const int *' differ in signedness [-Wpointer-sign]
[ 69s] 105 | { MATCH_ANY, MATCH_ANY, EVAS_COLORSPACE_PALETTE, &palette_ifmt, &palette_fmt }
```
@tizen_fix
Change-Id: I45b802841079f1233b9a3a24aa8fe06d931bd257
Yeongjong Lee [Thu, 13 Feb 2020 01:31:34 +0000 (10:31 +0900)]
evas_engine: remove unused variable
@tizen_fix
Change-Id: I3278b8e5690d89b83d7184a939ecbde5543b3523
Yeongjong Lee [Wed, 12 Feb 2020 09:34:58 +0000 (18:34 +0900)]
evas_proxy: correct initialization of default_proxy
ref
76b89b8e49f4b16ad490c04134a9bdddea27950f
@tizen_fix
Change-Id: I8bd92b0faf9d4ef6dfdfb2750acffb870edddee3
Yeongjong Lee [Wed, 12 Feb 2020 09:25:57 +0000 (18:25 +0900)]
evas_image_load: remove unused variable
Change-Id: Ie7317fa03fb591085ec5c4433bcc74b57c54f297
SangHyeon Jade Lee [Tue, 4 Feb 2020 10:55:26 +0000 (19:55 +0900)]
genlist : fix wrong condition of delete transition
Change-Id: Ieb5090489fa6623f5cd47ac3b406cbbf23c18c00
Signed-off-by: SangHyeon Jade Lee <sh10233.lee@samsung.com>
Stefan Schmidt [Wed, 12 Feb 2020 17:10:07 +0000 (12:10 -0500)]
ci: enable exactness build on travis
Summary:
With exactness merged into efl we can finally enable some exactness
testing on our CI. As a first step we will run around 47 tests from our
elementary_tests recordings.
The base data is already in the specific docker image to avoid
downloading on each run. We also need to use a dedicated branch from the
repo while we are still working out all the problems to make exactness
recordings really independent from their env.
Depends on D11317
Reviewers: zmike, bu5hm4n
Reviewed By: zmike
Subscribers: cedric, #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D11318
Stefan Schmidt [Wed, 12 Feb 2020 17:10:04 +0000 (12:10 -0500)]
exactness_player: fix running of exactness player after compile fix
Summary:
The last compile fix broke the execution of exactness. When replacing
_POSIX_PATH_MAX with PATH_MAX here we did not catch that it would
overwrite out of bounds here as the original _POSIX_PATH_MAX is limited
to 256 while PATH_MAX can go up to 4096. This screws up the size of the
memset.
Reviewers: zmike, bu5hm4n, vtorri, ali.alzyod
Reviewed By: zmike
Subscribers: cedric, #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D11317
João Paulo Taylor Ienczak Zanette [Wed, 12 Feb 2020 15:39:51 +0000 (16:39 +0100)]
csharp: Fix managed name when solving Get/SetType name clash
Summary:
If you declare a property called "type", the generated getter method
would be called "GetType", clashing with "System.Object.GetType" method. The
current workaround for that is to straight-forwardly insert the respective
class/abstract's Eolian name into the function name (becoming, for example
"GetMy_ClassType"), sometimes getting inconsistent to not only other methods
but also with documentation (leading to XML errors, since a cref doc would
reference "GetMyClassType", which doesn't exists).
This patch fixes that by wrapping the Eolian name with the already existing
`managed_name` function.
Test Plan: Run ninja tests.
Reviewers: zmike, segfaultxavi
Reviewed By: segfaultxavi
Subscribers: cedric, #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D11331
Mike Blumenkrantz [Thu, 23 Jan 2020 14:34:52 +0000 (09:34 -0500)]
tests/layout: modify api ordering test to verify theme signal emission
ensure that signals queued during construction are always emitted
Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D11162
Yeongjong Lee [Fri, 17 Jan 2020 07:50:07 +0000 (07:50 +0000)]
efl_ui_panes: remove duplicated press/unpress event
pressed/unpressed event will be emitted by efl_input_clickable_press/unpress.
Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D11116
Yeongjong Lee [Wed, 12 Feb 2020 14:05:42 +0000 (09:05 -0500)]
elc_fileselector: add NULL check for eina_value_type_get
Reviewers: Jaehyun_Cho, Hermet, zmike
Reviewed By: zmike
Subscribers: zmike, cedric, #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D11314
Yeongjong Lee [Wed, 12 Feb 2020 14:04:09 +0000 (09:04 -0500)]
elm_interface_scrollable: prevent calling APIs with NULL edje object
Summary:
Is is unnecessary to call edje_object_ functions if `edje_obj` is NULL.
Also, It prevents printing 6 eo warnings
```
WRN<30343>:eo ../src/lib/eo/eo.c:644 _efl_object_call_resolve() NULL passed to function efl_layout_signal_emit().
```
Test Plan:
`EINA_LOG_LEVELS=eo:2 elementary_test`
`EINA_LOG_LEVELS=eo:3 elementary_test -to focus`
Reviewers: bu5hm4n, Jaehyun_Cho, zmike
Reviewed By: zmike
Subscribers: zmike, cedric, #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D11315
Carsten Haitzler (Rasterman) [Wed, 12 Feb 2020 09:52:03 +0000 (09:52 +0000)]
eina - promise - be safe if no scheduled_entry is there
something is causing a scheduled entry to be called with no scheduled
entry: https://phab.enlightenment.org/T8490 - valgrind trace
aattached. i can't see it so i don't know what. the only protection i
can put in is this.
Xavi Artigas [Wed, 12 Feb 2020 09:44:06 +0000 (10:44 +0100)]
Tabs: remove them
They have been known to crash some old compilers and have
severe performance penalties on non-power-of-two architectures
like the Chitauri 13-bit processors.
Tabs also cause eye strain, vomits, brain tumors and bad breath.
Jérémy Zurcher [Wed, 12 Feb 2020 08:08:59 +0000 (09:08 +0100)]
eo : fix
8e6e1860c3, _efl_object_event_future_scheduler_get(…) returns a pointer
Wonki Kim [Wed, 12 Feb 2020 07:48:16 +0000 (16:48 +0900)]
evas: remove redundant conditions.
Summary:
conditions of the inner and outer statement are identical.
this patch removes a condition of them.
Reviewers: jsuya, Hermet
Reviewed By: Hermet
Subscribers: cedric, #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D11324
Wonki Kim [Wed, 12 Feb 2020 07:47:33 +0000 (16:47 +0900)]
eo: remove potentional NULL dereferencing
Summary: this patch removes a potentional NULL dereferencing
Reviewers: jsuya, Hermet
Reviewed By: Hermet
Subscribers: cedric, #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D11325
Wonki Kim [Wed, 12 Feb 2020 07:47:00 +0000 (16:47 +0900)]
edje_cc : remove potential NULL dereferencing
Summary: this patch removes potential NULL deferencing in edje_cc
Reviewers: jsuya, Hermet
Reviewed By: Hermet
Subscribers: cedric, #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D11326
Yeongjong Lee [Wed, 12 Feb 2020 07:45:40 +0000 (16:45 +0900)]
efl_ui_position_manager_list: add NULL check for memory allocation
Reviewers: Jaehyun_Cho, Hermet
Reviewed By: Hermet
Subscribers: cedric, #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D11312
Shinwoo Kim [Tue, 11 Feb 2020 14:03:43 +0000 (09:03 -0500)]
evas_render: initialize variable
Summary:
evas_object_image_load_region_get could be called with following stack.
(#0) evas_object_image_load_region_get
(#1) evas_render_proxy_subrender
(#2) evas_filter_context_proxy_render_all
(#3) evas_filter_object_render
(#4) evas_object_text_render
This means that evas_object_image_load_region_get is called by text object.
In this case, the load region value has garbabe, and it leads to invalid
memory access which is detected by Assan(T8610).
This patch initialize variable before using evas_object_image_load_region_set.
Reviewers: Hermet, jsuya, bu5hm4n, zmike
Reviewed By: zmike
Subscribers: cedric, #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D11316
Shinwoo Kim [Tue, 4 Feb 2020 03:14:23 +0000 (12:14 +0900)]
evas proxy: make it work for load_region
Summary:
This makes a proxy object use a selective region of a source object.
So far a proxy has not worked for load_region at all.
This should be better solution than https://phab.enlightenment.org/D10604
introducing new interface.
This is useful when the source is too big to allocate a proxy surface.
This will be used by elm_scroller to solve following issue.
[Issue]
If size of elm_sclloer content is too big, then the proxy of
elm_scroller to show loop effect does not work. Because
evas_gl_common_image_surface_new does not allow
bigger size surface than max_texture_size
Reviewers: Hermet, jsuya
Reviewed By: Hermet
Subscribers: cedric, #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D10626
Change-Id: I9a8d80eca88a5f2d1d1c1b5425294bcd4017bfa9
Wonki Kim [Wed, 12 Feb 2020 06:06:07 +0000 (15:06 +0900)]
wearable_position_mgr: remove a potentional null dereferencing
this patch removes a potentional null dereferencing
Change-Id: I6e5fc8fb3aef8376435eb0682c21720fc5a95c24
Hermet Park [Wed, 12 Feb 2020 03:23:06 +0000 (12:23 +0900)]
Revert "genlist :disable source clip set to prevent unwanted clipping issue on proxy"
This reverts commit
b2acde3ac2c7da734abeb71e2dd070aa08180876.
Fixed proxy regression bug -
d5d9e9a9aafad398c82c5302f36c1be8db53d455
Change-Id: Ie185ef59f19ec15c5405185a63071ecf034856df
Yeongjong Lee [Fri, 31 Jan 2020 05:24:13 +0000 (14:24 +0900)]
Revert "Revert "eolian: enforce that list<> can only be used with @beta API.""
Build error has been fixed by
8c28c4c6cc6164819d521e9834af537f4978f1ee.
This reverts commit
e77fe8c40b8e0b26bc1b86734f304d9a7825ea4b.
Change-Id: I3ed3b7ad9c3b01612b573f34bfc2eb5b1dadb612
Marcel Hollerbach [Fri, 31 Jan 2020 15:47:46 +0000 (16:47 +0100)]
eolian: finally error on missing since tags
this is actaully usefull ... for people not forgetting things.
Reviewed-by: Xavi Artigas <xavierartigas@yahoo.es>
Reviewed-by: Daniel Kolesa <daniel@octaforge.org>
Differential Revision: https://phab.enlightenment.org/D11266
Change-Id: Iae2c685be54e282201b2ffa1ef02b77ffb497a33
Yeongjong Lee [Wed, 12 Feb 2020 02:37:12 +0000 (11:37 +0900)]
evas_canvas3d_types: mark @beta Evas.Real
No build error anymore.
ref commit
10b2e407d0a002ef4bc5987cbdafea5fd62952e6.
@tizen_fix
Change-Id: I69fce86a6ba8019a73604eec0689f5a04a956fbc
Yeongjong Lee [Wed, 12 Feb 2020 02:34:55 +0000 (11:34 +0900)]
ecore_input_device: add since tags for ecore_input_device enums
@tizen_fix
Change-Id: I3403b7e1043e160feee0497cee9659d5a163f945
Jongmin Lee [Tue, 11 Feb 2020 22:15:18 +0000 (07:15 +0900)]
exactness: make it optional and disble it by default
@tizen_only
Change-Id: I3b5d3c904c9c10db73ec7533232004f0b26b872b
Yeongjong Lee [Tue, 11 Feb 2020 13:56:34 +0000 (08:56 -0500)]
elm perf_test: fix warning of implicit conversion of floating point
Summary:
The result of integral division is converted into floating point.
In order to fix this warning, it changes denominator to double from integer.
Reviewers: Jaehyun_Cho, Hermet, zmike
Reviewed By: zmike
Subscribers: cedric, #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D11313
Ali Alzyod [Tue, 11 Feb 2020 11:03:00 +0000 (12:03 +0100)]
efl.canvas.textblock: allow all white spaces in style string not just space
Summary:
style string can contain any kind of white spaces and it will be fine
For example
```
"font=sans font_size=30 color=red "
```
Is the same as
```
"font=sans\tfont_size=30\n color=red "
```
Reviewers: woohyun, segfaultxavi, tasn, zmike
Reviewed By: segfaultxavi
Subscribers: bu5hm4n, cedric, #reviewers, #committers
Tags: #efl
Maniphest Tasks: T8532
Differential Revision: https://phab.enlightenment.org/D11303
Xavi Artigas [Tue, 11 Feb 2020 10:03:49 +0000 (11:03 +0100)]
docs: ecore_animator's default framerate is 60 not 30
This was changed in
81242af6f95 without updating the docs.
Stefan Schmidt [Mon, 10 Feb 2020 11:10:43 +0000 (12:10 +0100)]
exactness_player: make use of PATH_MAX define instead of posix variant
The use of _POSIX_ has lead to a compile problem on old Ubuntu (16.04)
versions. We use PATH_MAX everywhere else so follow suit here.
Fixes T8613
Reviewed-by: Ali Alzyod <ali198724@gmail.com>
Differential Revision: https://phab.enlightenment.org/D11305
Alastair Poole [Sun, 9 Feb 2020 21:14:42 +0000 (21:14 +0000)]
elm_code_widget: Display cursor within widget.
Only render the cursor when its within the bounds of the
widget.
Lucas Cavalcante de Sousa [Fri, 7 Feb 2020 18:52:15 +0000 (13:52 -0500)]
efl: `eina_value_list_vinsert` assumes that the position is always valid
Summary:
The eina_value_list_vinsert at src/lib/eina_inline_value.x was not checking if the desired position was valid:
When inserting in an empty list in any out of bounds position it actually createded a list with the head being the desired value.
When inserting in a non-empty list in an out of bounds position caused a c error.
Now both cases return EINA_FALSE
Ref T8611
Test Plan:
Meson configured with -Dbindings=mono,cxx -Dmono-beta=true, and tests runned
with ninja test all.
Reviewers: felipealmeida, zmike
Reviewed By: zmike
Subscribers: cedric, #reviewers, #committers
Tags: #efl
Maniphest Tasks: T8611
Differential Revision: https://phab.enlightenment.org/D11301
Marcel Hollerbach [Fri, 7 Feb 2020 14:04:03 +0000 (15:04 +0100)]
exactness: fix osx build
edje depends on more than just edje_cc.
Ali Alzyod [Fri, 7 Feb 2020 13:15:20 +0000 (14:15 +0100)]
efl.ui.textbox: prevent longpress + right click from working on scrollbars
summary_: Longpress and right click one textbox scrollbars will not show up the menu.
Reviewers: woohyun, bu5hm4n
Subscribers: cedric, #reviewers, #committers
Tags: #efl
Maniphest Tasks: T8604
Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D11295
Stefan Schmidt [Fri, 31 Jan 2020 12:35:54 +0000 (13:35 +0100)]
exactness: cleanup some coding style isues
Just a few that I spotted when looking over the code.
Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com>
Differential Revision: https://phab.enlightenment.org/D11290
Stefan Schmidt [Fri, 31 Jan 2020 12:34:23 +0000 (13:34 +0100)]
exactness: fix init/shutdown handling of efl components in various binaries
The init/shutdown handling for efl libraries was a bit sloppy and
unbalanced in the exactness binaries. Switching over to use
ecore_eas_init/shutdown here instead of doing all libs individually.
Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com>
Differential Revision: https://phab.enlightenment.org/D11289
Stefan Schmidt [Fri, 31 Jan 2020 12:32:24 +0000 (13:32 +0100)]
exactness: remove unused code
Either commented out, blank lines or no needed includes.
Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com>
Differential Revision: https://phab.enlightenment.org/D11288
Stefan Schmidt [Fri, 31 Jan 2020 12:29:28 +0000 (13:29 +0100)]
exactness: remove no longer needed EO and BETA API defines
This has been in place during development and not updated for recent
changes and merge into efl master.
Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com>
Differential Revision: https://phab.enlightenment.org/D11287
Stefan Schmidt [Wed, 16 Jan 2019 14:41:50 +0000 (15:41 +0100)]
exactness: print out statistics at the end of the run
Given a better overview after the wall of text we are seeing from
starting and stopping jobs in parallel.
Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com>
Differential Revision: https://phab.enlightenment.org/D11286
Daniel Zaoui [Mon, 24 Dec 2018 20:51:52 +0000 (22:51 +0200)]
exactness: import code from external repo into efl.git
Exactness has been developed in a separate git repo for many years. This
finally moves it over into efl. Having it in tree allows us for easier
testing with our current main target elementary_test and integration
into our CI system (patches for this are work in progress already).
We are only importing the lib and binary for test execution, not the
full set of test data. This is would be over 500MB and thus it will stay
in a different repo and only made available during the actual testing.
[The original patch was made by Daniel Zaoui. Over the course of review
and testing it got extended with build fixes for API changes and mingw
compilation support from Stefan Schmidt and Michael Blumenkrantz]
Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com>
Differential Revision: https://phab.enlightenment.org/D11285
Stefan Schmidt [Tue, 4 Feb 2020 10:41:17 +0000 (11:41 +0100)]
elementary: build elementary_test additionally as shared object
For our exactness testing we are dlopen() the elementary_test
executable. Until glibc 2.30 this was posible with -pie, but it changed
in the glibc version. They no longer allow these executables to be
dlopen()'ed.
As a workaround for now we are building elementary_test also as a shared
object file which we load and use in the exactness testing process.
The code came from Marcel Hollerbach and I only tested it and fixed up a
small detail.
Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D11284
Daniel Kolesa [Thu, 6 Feb 2020 13:43:38 +0000 (14:43 +0100)]
eolian: fix missing stringshare_ref
This was properly applied to all cases except this one where it
was overlooked/forgotten. That resulted in the @c_name() feature
with enums being broken because of bad memory.
Fixes T8596.
abdulleh Ghujeh [Thu, 6 Feb 2020 06:36:30 +0000 (15:36 +0900)]
Evas Textblock: update obstacle test
Summary:
Update the test to check that the first character is rendered after the obstacle.
To ensure that the obstacle feature keeps working correctly.
Reviewers: ali.alzyod, woohyun
Reviewed By: ali.alzyod
Subscribers: segfaultxavi, cedric, #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D11033
Marcel Hollerbach [Wed, 5 Feb 2020 18:42:44 +0000 (19:42 +0100)]
eolian_cxx: fix a last few since errors
sinace has to be > 1.22 and the tests need to reflect that.
Xavi Artigas [Wed, 5 Feb 2020 17:53:43 +0000 (18:53 +0100)]
docs: Remove incorrect doc tag
This was breaking build with cxx bindings enabled.
ali [Wed, 5 Feb 2020 16:01:52 +0000 (16:01 +0000)]
evas_object_textbox: deal with <br> <tab> without closing '/'
This causes many issues because textbox functions deals with <br> <tab> differently depending that user write them with or without '/' at the end (for example <br> vs <br/>)
while most functionaliity are the same (like viewing <br> and <br/> are the same).
cursor dealing with these tags can be differently.
now we will assume <br> <tab> are already have there own closing tag, even if it is missing
Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D11293
Lucas Cavalcante de Sousa [Wed, 5 Feb 2020 14:58:17 +0000 (11:58 -0300)]
bindings-cxx: fix missing @since tags at stable c#/cpp tests
Story:
This revision got landed twice, I do not know what happend, but when i
wanted to land that initially with git phab, something reseted the state
of this revision to the initial version, and also changed the
authorship. This time, this commit contains all changes, and the correct
authorship, sorry for the noise.
Summary:
Add @since tags at stable (not tagged with @beta) `.eo` c#/c++ tests. Doing so,
eolian_enforce_since shouldn't accuse at stable tests.
Depends on D11264
Resolves T8600
Reviewers: zmike, segfaultxavi, woohyun, bu5hm4n
Subscribers: cedric, #reviewers, #committers
Tags: #efl
Maniphest Tasks: T8600
Differential Revision: https://phab.enlightenment.org/D11265
Marcel Hollerbach [Wed, 5 Feb 2020 15:23:51 +0000 (16:23 +0100)]
eot files: fixup all since tags
these tags have been wrong. eolian does not allow anything < 1.22
Marcel Hollerbach [Wed, 5 Feb 2020 15:22:59 +0000 (16:22 +0100)]
eolian: fix all remaining since errors
eolian checked for since tags that are invalid by its own rules. This is
now fixed.
Marcel Hollerbach [Tue, 4 Feb 2020 15:15:59 +0000 (12:15 -0300)]
bindings: fix missing @since tags & docs
summary_: Depends on D11264
Reviewers: zmike, segfaultxavi, woohyun
Subscribers: cedric, #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D11265
Mike Blumenkrantz [Wed, 29 Jan 2020 19:08:32 +0000 (14:08 -0500)]
efl/gesture: remove unused manager function
no functional changes
Reviewed-by: Xavi Artigas <xavierartigas@yahoo.es>
Differential Revision: https://phab.enlightenment.org/D11244
Mike Blumenkrantz [Wed, 29 Jan 2020 19:07:16 +0000 (14:07 -0500)]
evas/object: remove weird gesture manager method
these two objects have zero relation, and I don't know why this ever
existed
Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D11243
Mike Blumenkrantz [Wed, 29 Jan 2020 18:32:53 +0000 (13:32 -0500)]
efl/gesture: free object gesture list in destructor
leak--
Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D11242
Mike Blumenkrantz [Wed, 29 Jan 2020 18:19:14 +0000 (13:19 -0500)]
efl/gesture: use array instead of list for garbage gesture objects
this is always a full iteration so we don't actually need a list
Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D11241
Mike Blumenkrantz [Wed, 29 Jan 2020 18:09:36 +0000 (13:09 -0500)]
efl/gesture: fix gesture object cleanup when unregistering a recognizer
these objects need to be cleaned up immediately outside of event
processing to avoid having them automatically deleted later on and
triggering a double delete
Differential Revision: https://phab.enlightenment.org/D11240
Mike Blumenkrantz [Wed, 29 Jan 2020 16:50:57 +0000 (11:50 -0500)]
tests/gesture: add second custom recognizer test
ensure that this functionality works
Differential Revision: https://phab.enlightenment.org/D11239
Mike Blumenkrantz [Wed, 29 Jan 2020 16:07:23 +0000 (11:07 -0500)]
tests/elm: add explicit printf for event cb assert failure
efl explodes when this assert fails and presents bizarre errors which
obscure the actual test failure, so add a more explicit message as a
reminder to check this instead of trying to dive into insanity
Differential Revision: https://phab.enlightenment.org/D11238
Mike Blumenkrantz [Tue, 28 Jan 2020 19:23:27 +0000 (14:23 -0500)]
tests/gesture: add custom recognizer test
adds a very small implementation of a custom recognizer, registers it,
verifies events are processing as they should, then removes it
Differential Revision: https://phab.enlightenment.org/D11224
Mike Blumenkrantz [Tue, 28 Jan 2020 18:52:23 +0000 (13:52 -0500)]
efl/gesture: implement custom gesture recognizer framework
this adds support for creating custom (out of tree) gesture recognizers by
adding an abstract recognizer class that can be inherited and reused
docs TBA
Reviewed-by: woochan lee <wc0917.lee@samsung.com>
Differential Revision: https://phab.enlightenment.org/D11223
Mike Blumenkrantz [Tue, 28 Jan 2020 18:33:11 +0000 (13:33 -0500)]
efl/gesture: break out gesture processing code into separate function
this will get reused shortly
Reviewed-by: woochan lee <wc0917.lee@samsung.com>
Differential Revision: https://phab.enlightenment.org/D11222
Mike Blumenkrantz [Tue, 28 Jan 2020 17:56:41 +0000 (12:56 -0500)]
efl/gesture: remove manager's recognizer_get method
this exposes internal objects that we shouldn't be exposing
Reviewed-by: woochan lee <wc0917.lee@samsung.com>
Differential Revision: https://phab.enlightenment.org/D11221
Mike Blumenkrantz [Tue, 28 Jan 2020 14:45:42 +0000 (09:45 -0500)]
tests/gesture: change random momentum cb to be useful momentum cb
verify that we get 1 event here and also that callback removal works
Differential Revision: https://phab.enlightenment.org/D11220
Mike Blumenkrantz [Mon, 27 Jan 2020 18:40:52 +0000 (13:40 -0500)]
tests/gesture: add a 10-touch tap test
this tests the robustness of existing gesture recognizers
Differential Revision: https://phab.enlightenment.org/D11209
Mike Blumenkrantz [Mon, 27 Jan 2020 18:40:02 +0000 (13:40 -0500)]
efl/gesture: conditionally ignore multi-touch unpress events in zoom recognizer
if we have not begun to process a zoom gesture by this point, then we should
not be emitting a cancel result
Reviewed-by: woochan lee <wc0917.lee@samsung.com>
Differential Revision: https://phab.enlightenment.org/D11208
Mike Blumenkrantz [Mon, 27 Jan 2020 18:38:45 +0000 (13:38 -0500)]
efl/gesture: ignore successive press events in zoom recognizer when canceled
cancel is used to indicate that a gesture which has begun to trigger has been
canceled, so if we have already canceled then we should not re-cancel here
Reviewed-by: woochan lee <wc0917.lee@samsung.com>
Differential Revision: https://phab.enlightenment.org/D11207
Mike Blumenkrantz [Mon, 27 Jan 2020 18:37:25 +0000 (13:37 -0500)]
efl/gesture: ignore multi-touch end events in tap gesture recognizer
we only care about the last touch unpress event, so we should be ignoring
every other event instead of canceling, as we are also ignoring the
multi-touch press events
Reviewed-by: woochan lee <wc0917.lee@samsung.com>
Differential Revision: https://phab.enlightenment.org/D11206
Mike Blumenkrantz [Mon, 27 Jan 2020 14:27:41 +0000 (09:27 -0500)]
efl/gesture: avoid direct double access in structs
this is functionally wrong (the comparison fails) and also causes SIGBUS on
arm
Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Reviewed-by: woochan lee <wc0917.lee@samsung.com>
Differential Revision: https://phab.enlightenment.org/D11198
Mike Blumenkrantz [Mon, 27 Jan 2020 14:27:05 +0000 (09:27 -0500)]
efl/gesture: reorder gesture private structs
order based on descending member size
Reviewed-by: woochan lee <wc0917.lee@samsung.com>
Differential Revision: https://phab.enlightenment.org/D11197
Mike Blumenkrantz [Fri, 24 Jan 2020 19:12:46 +0000 (14:12 -0500)]
efl/gesture: remove 'gesture' member from recognizer data struct
this was never used
ref T8503
Reviewed-by: woochan lee <wc0917.lee@samsung.com>
Differential Revision: https://phab.enlightenment.org/D11178
Mike Blumenkrantz [Fri, 24 Jan 2020 19:11:01 +0000 (14:11 -0500)]
efl/gesture: remove 'manager' member from recognizer data struct
this can be fetched using efl_provider_find from the recognizer
ref T8503
Reviewed-by: woochan lee <wc0917.lee@samsung.com>
Differential Revision: https://phab.enlightenment.org/D11177
Mike Blumenkrantz [Fri, 24 Jan 2020 19:06:42 +0000 (14:06 -0500)]
efl/gesture: remove 'config' property from recognizers
this is a manager functionality, and recognizers are always child objects of
managers
ref T8503
Reviewed-by: woochan lee <wc0917.lee@samsung.com>
Differential Revision: https://phab.enlightenment.org/D11176
Mike Blumenkrantz [Fri, 24 Jan 2020 18:58:40 +0000 (13:58 -0500)]
efl/gesture: expose 'continues' as a public property for recognizers
we still access this using private data in the in-tree recognizers, but
now it's also accessible normally to custom recognizers
ref T8503
Reviewed-by: woochan lee <wc0917.lee@samsung.com>
Differential Revision: https://phab.enlightenment.org/D11175
Mike Blumenkrantz [Fri, 24 Jan 2020 18:38:49 +0000 (13:38 -0500)]
efl/gesture: add an event to notify when a config value has been changed
recognizers should be storing config values internally to avoid overhead
of multiple eo/hash lookups on every event
Reviewed-by: woochan lee <wc0917.lee@samsung.com>
Differential Revision: https://phab.enlightenment.org/D11174
Mike Blumenkrantz [Fri, 24 Jan 2020 18:34:45 +0000 (13:34 -0500)]
efl/gesture: move finger_size into private structs of recognizers
we must write these recognizers using publicly available apis to ensure
that we don't accidentally start using private struct data
this requires a lot of changes to properly manage the config value for finger
size and update the recognizer data when changed (which external recognizers
will need to monitor an event to achieve) and then also to change some
recognizers so they don't use memset and unset the finger_size value
ref TT8503
Reviewed-by: woochan lee <wc0917.lee@samsung.com>
Differential Revision: https://phab.enlightenment.org/D11173
Mike Blumenkrantz [Thu, 23 Jan 2020 20:21:22 +0000 (15:21 -0500)]
tests/gesture: add zoom and pinch tests
basic tests verifying completion of the gestures
Differential Revision: https://phab.enlightenment.org/D11170
Mike Blumenkrantz [Thu, 23 Jan 2020 20:20:12 +0000 (15:20 -0500)]
efl/gesture: discard non-press events for all touch points if unseen previously
not sure why this was restricted to first touch point previously, but the logic
applies to all points as we don't want to be tracking input for a touch we
haven't gotten a down for
Reviewed-by: woochan lee <wc0917.lee@samsung.com>
Differential Revision: https://phab.enlightenment.org/D11169
Mike Blumenkrantz [Thu, 23 Jan 2020 20:18:58 +0000 (15:18 -0500)]
efl/gesture: add 'pressed' state info for touch data
since we retain touch info for the duration of a touch sequence, including
after a touch point has been unpressed, it's necessary to track the current
state of each point and then use that to accurately determine the number of
touches active
Reviewed-by: woochan lee <wc0917.lee@samsung.com>
Differential Revision: https://phab.enlightenment.org/D11168
Mike Blumenkrantz [Thu, 23 Jan 2020 20:18:09 +0000 (15:18 -0500)]
efl/gesture: correctly reset zoom gesture recognizer data on >2 finger touch
canceling without a proper reset breaks the recognizer for successive events
Reviewed-by: woochan lee <wc0917.lee@samsung.com>
Differential Revision: https://phab.enlightenment.org/D11167
Mike Blumenkrantz [Wed, 22 Jan 2020 18:56:07 +0000 (13:56 -0500)]
tests/elm: add util function for doing pinch motions
simple function which takes start points and vectors for 2 touches
Differential Revision: https://phab.enlightenment.org/D11166
Mike Blumenkrantz [Wed, 22 Jan 2020 18:51:44 +0000 (13:51 -0500)]
efl/gesture: more accurately handle flick/momentum recognition with multi-touch
if we have one of these gestures where we are flicking/momentuming with multiple
touch points, we want to permit this so long as the touch points are moving vaguely
in the same direction, e.g., press -> swipe with 2 fingers
with this, we now cancel these gestures with multi-touch active if we detect motion
with subsequent touch events that doesn't match the vector of the "active" press
for which we are monitoring events
Reviewed-by: woochan lee <wc0917.lee@samsung.com>
Differential Revision: https://phab.enlightenment.org/D11165
Mike Blumenkrantz [Wed, 22 Jan 2020 18:51:02 +0000 (13:51 -0500)]
efl/gesture: break out _direction_get() into helper function
this is useful in multiple places, no functional changes
Reviewed-by: woochan lee <wc0917.lee@samsung.com>
Differential Revision: https://phab.enlightenment.org/D11164
Mike Blumenkrantz [Wed, 22 Jan 2020 18:48:39 +0000 (13:48 -0500)]
evas/events: fix multi-touch object targeting on touch down events
a touch press is both a touch and a move event, which means the canvas must
update its list of target objects on the first touch press to avoid using the
wrong objects when processing events
@fix
Reviewed-by: Cedric BAIL <cedric.bail@free.fr>
Differential Revision: https://phab.enlightenment.org/D11163