platform/upstream/efl.git
4 years agoecore-evas/cocoa: fix cnp leak
Mike Blumenkrantz [Thu, 26 Mar 2020 14:46:20 +0000 (10:46 -0400)]
ecore-evas/cocoa: fix cnp leak

this pointer needs to be freed if it's being copied

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

4 years agoelm_entry: using keycode for main keyboard shortcuts
Ali [Fri, 27 Mar 2020 06:07:16 +0000 (06:07 +0000)]
elm_entry: using keycode for main keyboard shortcuts

Previously we use key strings to detect keyboard shortcuts (for example copy/paste/select_all, .. etc)
Now we will use key code alongside with these key strings (I do not remove the key string, because I am afraid something depends on them because some methods do not send keycode with keys)

This issue can be easily detected when the keyboard layout is not a Latin language, for example Hitting "C" will produce key string depends on related language not letter C, but Keycodes remain the same regardless of keyboard layout.

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

4 years agocode: Silence compiler warnings.
Alastair Poole [Fri, 3 Apr 2020 10:38:48 +0000 (11:38 +0100)]
code: Silence compiler warnings.

4 years agoedje_legacy: Remove EINA_UNUSED when obj is actually used
ProhtMeyhet [Fri, 6 Mar 2020 21:21:58 +0000 (21:21 +0000)]
edje_legacy: Remove EINA_UNUSED when obj is actually used

This removes EINA_UNUSED when obj is actually used
like in a call ed = _edje_fetch(obj);
Please note the macro GET_REAL_PART_ON_FAIL_RETURN
also uses obj.

No functional changes, just cleanup.

Reviewed-by: Christopher Michael <devilhorns@comcast.net>
Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org>
Differential Revision: https://phab.enlightenment.org/D11471

4 years agofree memory in clipboard notify event and fix clipboard clear event
Vincent Torri [Tue, 24 Mar 2020 14:05:19 +0000 (14:05 +0000)]
free memory in clipboard notify event and fix clipboard clear event

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

4 years agobuild: resolve migration build error (bump version for 1.24)
Jongmin Lee [Fri, 3 Apr 2020 02:08:01 +0000 (11:08 +0900)]
build: resolve migration build error (bump version for 1.24)

Change-Id: I5a03078bb805032cc5920416ef2724e3291dfbc0

4 years agoelementary - get/set resources for extension theme to load bundle of Elm_Color_Overla...
Seungwan Woo [Wed, 26 Feb 2020 06:00:57 +0000 (15:00 +0900)]
elementary - get/set resources for extension theme to load bundle of Elm_Color_Overlay / Elm_Font_Overlay

Change-Id: If248a48a03a9dd559475a82a8d55d5b7e4b41076
Signed-off-by: Seungwan Woo <s-w.woo@samsung.com>
4 years agoAdd debug information to ecore_event
HyukSoon Choi [Tue, 31 Mar 2020 04:57:31 +0000 (13:57 +0900)]
Add debug information to ecore_event

- add debug information which for analyzing of many issues related to touch event.
Sometimes we need more information regarding touch events of ecore layer.
(ex. position, timestamp, etc)

Change-Id: I76ee7f9faf6027c914237355be4f3c2eaaf74dcb
Signed-off-by: HyukSoon Choi <hs619.choi@samsung.com>
4 years agoevas_object_main: skip processing when there is no pointer data.
Hosang Kim [Tue, 31 Mar 2020 07:09:35 +0000 (16:09 +0900)]
evas_object_main: skip processing when there is no pointer data.

@tizen_only

Change-Id: Ide0f64bda6f2d11e50b51c9d764eafec9213c71e

4 years agoBack to top feature Contribution.
Godly T.Alias [Fri, 27 Mar 2020 09:06:07 +0000 (14:36 +0530)]
Back to top feature Contribution.

Change-Id: I06224bed8b8fb7cb9354d619a1338ee1802e9cbe
Signed-off-by: Suresh Kumar <suresh.ku1@samsung.com>
4 years ago[text marquee] Fix text marquee with textblock fit.
Subodh Kumar [Wed, 22 Jan 2020 13:30:58 +0000 (19:00 +0530)]
[text marquee] Fix text marquee with textblock fit.

test plan: Please test with fit option and marquee enabled.

@tizen_fix

Change-Id: I51b00c0297283706bfc172b4884cf6e20ccd4305

4 years agogenlist : clean del_fx items on item clear
SangHyeon Jade Lee [Wed, 25 Mar 2020 09:15:01 +0000 (18:15 +0900)]
genlist : clean del_fx items on item clear

del_fx items are already invalidated,
so sd->items walking is broken as efl_del won't able to call
item_free().

so clean del_fx items before sd->items walking.

Change-Id: I1b61f96d58b59a698a08bc0a5ebbd72cbd59df9d

4 years agogenlist : freeze the event before select item
SangHyeon Jade Lee [Mon, 23 Mar 2020 11:15:12 +0000 (20:15 +0900)]
genlist : freeze the event before select item

this is revert patch of commented freeze_events on genlist.

Change-Id: I7e6d2021c2eb37bbc3aa06feb53190c3b3ddede2

4 years agoscroll_interface : change animator object as pan to prventing event freezer
SangHyeon Jade Lee [Mon, 23 Mar 2020 10:55:33 +0000 (19:55 +0900)]
scroll_interface : change animator object as pan to prventing event freezer

Change-Id: I574d1afd2797d5213ff5c7d201dacc3c91df7746

4 years agogenlist : do not calc deferring on scroll in center align mode
SangHyeon Jade Lee [Fri, 20 Mar 2020 08:22:20 +0000 (17:22 +0900)]
genlist : do not calc deferring on scroll in center align mode

Change-Id: Ie98ce347c436b6f08fb5578043a3463affe504d3

4 years ago[efl] Update 'gototop' to 'backtotop'
Yukti Pandey [Mon, 16 Mar 2020 10:59:27 +0000 (16:29 +0530)]
[efl] Update 'gototop' to 'backtotop'

Updated all gototop code with backtotop

Change-Id: Ife763dd93719771fad66b9b316c019f464759590
Signed-off-by: Yukti Pandey <yukti.pandey@samsung.com>
4 years agoRemove remarks flag of elm_win_aux_msg APIs
Hosang Kim [Thu, 12 Mar 2020 08:43:35 +0000 (17:43 +0900)]
Remove remarks flag of elm_win_aux_msg APIs

- elm_win_aux_msg_key_get
- elm_win_aux_msg_val_get
- elm_win_aux_msg_options_get

@tizen_only

Change-Id: Ic9624dde297385c7a0c28ca62ca8b02f3d5e0286

4 years agoecore/evas: fix indentations and comments
Hosang Kim [Tue, 10 Mar 2020 04:59:45 +0000 (13:59 +0900)]
ecore/evas: fix indentations and comments

Change-Id: I05ea238884266f79fbb28ec8cf34390cecf9e013

4 years agoecore/evas: add brief docs for enum.
Hosang Kim [Mon, 9 Mar 2020 08:06:02 +0000 (17:06 +0900)]
ecore/evas: add brief docs for enum.

Change-Id: I82565c0e749da4467b1d861e94b74338ec25af82

4 years agoelm_gen : Remove unnecessary inehitance of WIDGET_ITEM_STATIC_FOCUS
SangHyeon Jade Lee [Fri, 6 Mar 2020 05:54:59 +0000 (14:54 +0900)]
elm_gen : Remove unnecessary inehitance of WIDGET_ITEM_STATIC_FOCUS

Elm_Widget_Item_Static_Focus Class is for supporing item focus of
Elm_Gen_Item, in new focus manager system.

As this class add/del realized/unrealized callbacks for every each
items, if genlist/gengrid have more than 10000 items, performance
of adding/deleting item is getting severely increased.

In legacy focus system, we do not need this class anymore,
so get rid of it in class definitions.

Change-Id: I1666fec95c297b88a70fa0646df36ece6e01fac0
Signed-off-by: SangHyeon Jade Lee <sh10233.lee@samsung.com>
4 years agoecore_evas: fix class name of ecore device.
Hosang Kim [Thu, 5 Mar 2020 11:07:56 +0000 (20:07 +0900)]
ecore_evas: fix class name of ecore device.

Change-Id: I3671126ca4470e15ee9e4c215131219cd3054d21

4 years agoRevert "win/widget: free array when object is deleted."
kim hosang [Tue, 31 Mar 2020 10:50:16 +0000 (10:50 +0000)]
Revert "win/widget: free array when object is deleted."

This reverts commit c0ef877a9c6d0628e6934e5dcc5b09e3021ee02b.

Change-Id: I01308360805237d9f61025a15e642f5f7ddb387d

4 years agowin/widget: free array when object is deleted.
Hosang Kim [Tue, 31 Mar 2020 06:43:24 +0000 (15:43 +0900)]
win/widget: free array when object is deleted.

 It makes a memory leak.

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

Change-Id: I7dfaf2e70017382f9e905e4f77488fd283b10a7e

4 years agodoxygen docs: some fixes in Evas_Legacy.h
Xavi Artigas [Wed, 1 Apr 2020 07:50:27 +0000 (09:50 +0200)]
doxygen docs: some fixes in Evas_Legacy.h

This brings into the docs hundreds of methods!
due to the ingroup->defgroup mistake, they were out of any scope
and therefore they were silently ignored by doxygen.

Also, document lots of missing "obj" parameters. Not strictly necessary, but
this further reduces the number of doxygen warnings.

4 years agodoxygen docs: Add _Eina_Version struct docs
Xavi Artigas [Fri, 27 Mar 2020 14:29:56 +0000 (15:29 +0100)]
doxygen docs: Add _Eina_Version struct docs

4 years agocomments: Typos in Efl_Config.h
Xavi Artigas [Fri, 27 Mar 2020 12:49:07 +0000 (13:49 +0100)]
comments: Typos in Efl_Config.h

4 years agoexactness docs: Add docs, fix docs, remove docs
Xavi Artigas [Wed, 1 Apr 2020 16:40:40 +0000 (18:40 +0200)]
exactness docs: Add docs, fix docs, remove docs

Summary:
The Exactness tool needed usage instructions... and quite some more
fixes. There was copypasta all around.

Depends on D11634

Test Plan: Build and Enjoy

Reviewers: bu5hm4n, stefan_schmidt

Reviewed By: stefan_schmidt

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

4 years agoci: travis: switch exactness testing to normal test file
Stefan Schmidt [Wed, 1 Apr 2020 13:03:05 +0000 (15:03 +0200)]
ci: travis: switch exactness testing to normal test file

Since the changes from Marcel to run exactness as LD_PRELOADed library
we can switch back to the normal test.txt file and names.

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

4 years agoexactness: protect against forked calls
Marcel Hollerbach [Wed, 1 Apr 2020 12:04:19 +0000 (14:04 +0200)]
exactness: protect against forked calls

there came up a issue, where a excatness spawned processes were bringing
up a efreetd instance, when the efreetd instance turned off itself, the
files for exactness were written again, which is wrong. This ensures
that forked instances do not take any actions.

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

4 years agorelease: Update NEWS and bump version for 1.24.0-alpha1 release
Stefan Schmidt [Wed, 1 Apr 2020 10:51:51 +0000 (12:51 +0200)]
release: Update NEWS and bump version for 1.24.0-alpha1 release

4 years agoRevert "win/widget: free array when object is deleted."
kim hosang [Tue, 31 Mar 2020 10:50:16 +0000 (10:50 +0000)]
Revert "win/widget: free array when object is deleted."

This reverts commit c0ef877a9c6d0628e6934e5dcc5b09e3021ee02b.

Change-Id: I01308360805237d9f61025a15e642f5f7ddb387d

4 years agoexactness: error when _src_unit is NULL
Marcel Hollerbach [Tue, 31 Mar 2020 08:07:29 +0000 (10:07 +0200)]
exactness: error when _src_unit is NULL

if _src_unit is NULL, the write here would delete the actions, with this
commit we ensure that this is printing an error.

Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org>
Differential Revision: https://phab.enlightenment.org/D11627

4 years agoexactness: equip with ERR and CRIT
Marcel Hollerbach [Mon, 30 Mar 2020 14:49:05 +0000 (16:49 +0200)]
exactness: equip with ERR and CRIT

before a few commits, we had the situation that errors were overseen
because the log was simply so big, that errors did not get shown
properly.

With this commit, exactness will simply abort if there is a real issue
in the code.

Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org>
Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com>
Differential Revision: https://phab.enlightenment.org/D11624

4 years agoexactness: correctly call mkdir
Marcel Hollerbach [Fri, 27 Mar 2020 17:43:42 +0000 (18:43 +0100)]
exactness: correctly call mkdir

we should not error when mkdir returns < 0. EEXIST should not result in
the return here.

Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org>
Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com>
Differential Revision: https://phab.enlightenment.org/D11618

4 years agoexactness: do not efl_del the evas
Marcel Hollerbach [Fri, 27 Mar 2020 17:11:07 +0000 (18:11 +0100)]
exactness: do not efl_del the evas

there is no need to do that, more than that. This is super dangerous,
the display and connection ptr of x are passed from ecore_evas to evas,
if you delete evas before ecore_evas, the later ecore_evas deletion will
destroy the x connection which calls some functions in evas, which is
already freed, which leads to a crash.

Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org>
Differential Revision: https://phab.enlightenment.org/D11617

4 years agoexactness_play: introduce runner script
Marcel Hollerbach [Fri, 27 Mar 2020 15:54:09 +0000 (16:54 +0100)]
exactness_play: introduce runner script

this is just a little python script, so you can lunch exactness_play
without the need of handdefining LD_PRELOAD

Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org>
Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com>
Differential Revision: https://phab.enlightenment.org/D11616

4 years agoexactness_play: transform to LD_PRELOAD library
Marcel Hollerbach [Fri, 27 Mar 2020 14:44:33 +0000 (15:44 +0100)]
exactness_play: transform to LD_PRELOAD library

this can now be loaded as LD_PRELOAD library

Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org>
Differential Revision: https://phab.enlightenment.org/D11615

4 years agoexactness_player: refactor main method
Marcel Hollerbach [Fri, 27 Mar 2020 14:08:46 +0000 (15:08 +0100)]
exactness_player: refactor main method

this refactors everything that is not argc argv related out of the main
method. For later usage

Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org>
Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com>
Differential Revision: https://phab.enlightenment.org/D11614

4 years agoexactness: remove code for compiling
Marcel Hollerbach [Fri, 27 Mar 2020 13:03:07 +0000 (14:03 +0100)]
exactness: remove code for compiling

this seems to be made for compiling binaries before testing. That sounds
like a good idea, however, implementing a full buildtool in exactness is
a bit hard. Hence, using meson for that would be better.

Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org>
Differential Revision: https://phab.enlightenment.org/D11613

4 years agoexactness_record: introduce runner script
Marcel Hollerbach [Fri, 27 Mar 2020 10:25:22 +0000 (11:25 +0100)]
exactness_record: introduce runner script

this is just a little python script, so you can lunch exactness_record
without the need of handdefining LD_PRELOAD

Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org>
Differential Revision: https://phab.enlightenment.org/D11611

4 years agoexactness_recorder: move to preload
Marcel Hollerbach [Fri, 27 Mar 2020 08:14:45 +0000 (09:14 +0100)]
exactness_recorder: move to preload

this is now not a binary anymore, that dlopen's a binary, it is now a
library, that can be loaded using LD_PRELOAD. EXACTNESS_DEST is used for
the path of the .exu file. EXACTNESS_FONTS_DIR is used to get the fonts
directory

Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org>
Differential Revision: https://phab.enlightenment.org/D11610

4 years agoexactness_recorder: refactor main method
Marcel Hollerbach [Thu, 26 Mar 2020 15:41:21 +0000 (16:41 +0100)]
exactness_recorder: refactor main method

all calls taht are not related to env var checking, are moved out of the
main method. That is in preparation for later refactorings.

Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org>
Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com>
Differential Revision: https://phab.enlightenment.org/D11623

4 years agoelm_atspi_bridge: initialize variables 74/229274/1
Shinwoo Kim [Mon, 30 Mar 2020 05:19:19 +0000 (05:19 +0000)]
elm_atspi_bridge: initialize variables

The efl_access_text_attribute_get is resolved by elm_entry.
Please refer to _elm_entry_efl_access_text_attribute_get first.
Uninitialized variables are used for its parameters, and it is able to
return before setting these variables.

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

Change-Id: I14ef91d723a4bf0326c6e90e45226109857268c6

4 years agoelm_atspi_bridge: fix bugs reported by coverity 15/229115/2
Bartlomiej Grzelewski [Fri, 27 Mar 2020 11:33:06 +0000 (11:33 +0000)]
elm_atspi_bridge: fix bugs reported by coverity

Prevent usage of uninitilized values.

CID: 1421995, 1421997, 1422001

Signed-off-by: Bartlomiej Grzelewski <b.grzelewski@samsung.com>
Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org>
Differential Revision: https://phab.enlightenment.org/D11612

Change-Id: Iaf4b2df1826a31f6d2c022f00e76e2db01732c81

4 years agoelm_atspi_bridge: fix bugs reported by coverity 14/229114/2
Bartlomiej Grzelewski [Fri, 27 Mar 2020 11:24:42 +0000 (11:24 +0000)]
elm_atspi_bridge: fix bugs reported by coverity

Make sure we free allocated resources in the error path.
Prevent usage of uninitilized value.

Signed-off-by: Bartlomiej Grzelewski <b.grzelewski@samsung.com>
Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org>
Differential Revision: https://phab.enlightenment.org/D11607

Change-Id: I1d86a397a9d983d8b733615ffeaf318d4f0c9437

4 years ago[access] Add missing widget name setter
Prasoon Singh [Fri, 27 Mar 2020 13:28:02 +0000 (18:58 +0530)]
[access] Add missing widget name setter

Implementation file has _efl_ui_widget_efl_access_object_i18n_name_set
but same is missing from eo file. Due to which issue in access
name set/get API.

Change-Id: Iff96512d3b985bda06afe0ea65d99784eda559e9
Signed-off-by: Prasoon Singh <prasoon.16@samsung.com>
4 years agoscroller: fix a focus logic to move next/prior.
Wonki Kim [Mon, 30 Mar 2020 09:38:24 +0000 (18:38 +0900)]
scroller: fix a focus logic to move next/prior.

efl_ui_scrollable interface doesn't have a getter related with page number
this patch fixes a focus logic to move next/prior by pan size.

Change-Id: I0ad41886a1157e00a460c85705571de164b4510b

4 years agoecore_evas_wayland: Do not pass negative parameter to close function
Christopher Michael [Mon, 30 Mar 2020 18:34:41 +0000 (14:34 -0400)]
ecore_evas_wayland: Do not pass negative parameter to close function

Coverity reports that 'fd' here is negative, and close() cannot accept
a negative parameter, so add a check to make sure 'fd' is not negative
before passing to close function.

Fixes CID1420318

4 years agoecore_evas_wayland: Declare variables at top of function
Christopher Michael [Mon, 30 Mar 2020 18:33:23 +0000 (14:33 -0400)]
ecore_evas_wayland: Declare variables at top of function

4 years agoefl_ui_textbox: pass in correct seat
Marcel Hollerbach [Mon, 30 Mar 2020 14:22:37 +0000 (16:22 +0200)]
efl_ui_textbox: pass in correct seat

0 was the wrong seat, and only worked on a small amount of systems

fixes T8639

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

4 years agocanvas vg: null initialization for safety.
Hermet Park [Mon, 30 Mar 2020 12:30:08 +0000 (21:30 +0900)]
canvas vg: null initialization for safety.

4 years agoexactness_recorder: cleanup code a bit
Marcel Hollerbach [Thu, 26 Mar 2020 15:44:55 +0000 (16:44 +0100)]
exactness_recorder: cleanup code a bit

_test_name was not used globally, so lets move it to the used scope.
Verbose is not used at all either.

Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org>
Differential Revision: https://phab.enlightenment.org/D11609

4 years agoecore_wl2: Introduce ecore_wl2_window_pointer/keyboard get APIs.
Woochanlee [Thu, 26 Mar 2020 03:44:15 +0000 (12:44 +0900)]
ecore_wl2: Introduce ecore_wl2_window_pointer/keyboard get APIs.

To support multi seat, It should be able to get the input connected to the focused window.

Change-Id: I45a1c3a049fd46a97471bc8485599f8f28c75f59

4 years agoelm_atspi_bridge: add missing line
Shinwoo Kim [Fri, 27 Mar 2020 01:48:49 +0000 (10:48 +0900)]
elm_atspi_bridge: add missing line

This patch adds missing line of
https://review.tizen.org/gerrit/#/c/platform/upstream/elementary/+/153836/

Especially,
this patch makes atspi_bridge send a signal for INPUT_METHOD_WINDOW.
Without this patch, the virtual keyboard does not work correctly
for accessibility.

@tizen_fix

Change-Id: I95d5a478c629de904e38bc2ff01d6d055de8a0ca

4 years agoecore_wl2: Support keygrab privilge to make it public.
Woochanlee [Wed, 25 Mar 2020 08:55:04 +0000 (17:55 +0900)]
ecore_wl2: Support keygrab privilge to make it public.

Among the keygrab APIs, other modes except TOPMOST can only call processes with platform level privilege.

These APIs will be used by 3rd-party without privilege inevitably.(TOPMOST mode only.)

@tizen_only

Change-Id: I556169a5bd74898e186596fefc4548064365a544

4 years agoci: enable ecore_con IPv6 tests
Stefan Schmidt [Fri, 27 Mar 2020 08:54:43 +0000 (09:54 +0100)]
ci: enable ecore_con IPv6 tests

We have them behind an env variable as this is system specific. Enable
them on CI to fully test these paths as well.
While this works out of the box for the native build on Travis it fails
for our docker builds, as the dockerd does not have ipv6 enabled by
default. Only enable for Codecov for now.

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

4 years agoci: travis: enable s390x build on daily cron
Stefan Schmidt [Wed, 25 Mar 2020 12:37:13 +0000 (13:37 +0100)]
ci: travis: enable s390x build on daily cron

As there is no luajit package for s390x in Ubuntu Bionic we need to
adapt our installed packages as well as config options for it.

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

4 years agotravis: enable more archs during cron build
Stefan Schmidt [Tue, 24 Mar 2020 12:11:31 +0000 (13:11 +0100)]
travis: enable more archs during cron build

Enable builds on ppc64le on our daily build. s390x will follow at some
later point (once all quirks are resolved).
As we also use the Linux native build from Travis for Codecov we start
to handle meson options based on builds here as well.

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

4 years agoci: install needed packages for ppc64le builds
Stefan Schmidt [Tue, 24 Mar 2020 12:58:21 +0000 (13:58 +0100)]
ci: install needed packages for ppc64le builds

It seems on ppc64le and s390x these do not get pulled in automatically.
We need them for regex.h, make sure they are installed before building.

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

4 years agoEina: Remove eina_prime_table
Xavi Artigas [Fri, 27 Mar 2020 10:46:24 +0000 (11:46 +0100)]
Eina: Remove eina_prime_table

This has been present since ancient times and no current uses have been detected.
It's an array containing SOME prime numbers without a terminator, so the caller
needs to know the array length in advance.
This does not look very useful (or usable) and therefore it has been decided in
public IRC session to send this thing to hell.
Witnesses: raster stefan_schmidt bu5hm4n

4 years agodoxygen docs: Add missing docs for core Eina types
Xavi Artigas [Fri, 27 Mar 2020 10:40:57 +0000 (11:40 +0100)]
doxygen docs: Add missing docs for core Eina types

Things like EINA_LIKELY or EAPI were undocumented...

4 years agodocs: Fix Eina Core Group
Myoungwoon Roy, Kim [Fri, 27 Mar 2020 09:11:12 +0000 (10:11 +0100)]
docs: Fix Eina Core Group

Summary: I had fixed unlinked Eina API group(eina_main, eina_types, hamster) from Eina nodes. Those APIs included into Core group before.

Test Plan: API reference documentation modification only

Reviewers: segfaultxavi

Reviewed By: segfaultxavi

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

4 years agoatspi : Add check for chain end object with attribute
JunsuChoi [Thu, 26 Mar 2020 05:04:44 +0000 (14:04 +0900)]
atspi : Add check for chain end object with attribute

   When searching for a neighbor object,
   check whether there is an "relation_chain_end" attribute.
   If it does, it returns the current object.

This patch is missed when migrating from 4.0 to 5.0.
https://review.tizen.org/gerrit/#/c/platform/upstream/elementary/+/154970/

Change-Id: I61e5126b805391bcaa9ef6fa6448bb078b797001

4 years agocopying: remove obsolete items from copying that dont exist anymore
Carsten Haitzler (Rasterman) [Thu, 26 Mar 2020 17:07:29 +0000 (17:07 +0000)]
copying: remove obsolete items from copying that dont exist anymore

4 years agoeina: prefetch for _eo_obj_pointer_get
Marcel Hollerbach [Thu, 26 Mar 2020 16:42:46 +0000 (16:42 +0000)]
eina: prefetch for _eo_obj_pointer_get

Reviewers: raster, stefan_schmidt

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

4 years agofix segfault when there is no text in clipboard
Vincent Torri [Thu, 26 Mar 2020 14:47:34 +0000 (10:47 -0400)]
fix segfault when there is no text in clipboard

Test Plan: cnp a clipboard content without text

Reviewers: zmike

Reviewed By: zmike

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

4 years agodoxygen docs: Fix warnings in elm_config.h
Xavi Artigas [Thu, 26 Mar 2020 13:26:48 +0000 (14:26 +0100)]
doxygen docs: Fix warnings in elm_config.h

4 years agodoxygen docs: Fix warnings in Ecore_Common.h
Xavi Artigas [Thu, 26 Mar 2020 12:46:46 +0000 (13:46 +0100)]
doxygen docs: Fix warnings in Ecore_Common.h

4 years agodoxygen docs: Fix several warnings in Eina files
Xavi Artigas [Tue, 24 Mar 2020 15:49:04 +0000 (16:49 +0100)]
doxygen docs: Fix several warnings in Eina files

4 years agoeo: move all call assignment
Marcel Hollerbach [Tue, 24 Mar 2020 19:08:46 +0000 (20:08 +0100)]
eo: move all call assignment

this improves the cache performance a lot. Caches are only invalidated
once, and not multiple times.

Reviewed-by: Carsten Haitzler (Rasterman) <rasterman.com>
Differential Revision: https://phab.enlightenment.org/D11592

4 years agoeo: there is no reason to equip for calls to classes
Marcel Hollerbach [Tue, 24 Mar 2020 18:54:56 +0000 (19:54 +0100)]
eo: there is no reason to equip for calls to classes

calls to classes are not needed anymore, since class inheritance is not
a thing anymore. After removing is_obj from the function the compile can
optimize the code better, since assignments to fields are not
conditionally anymore.

Reviewed-by: Carsten Haitzler (Rasterman) <rasterman.com>
Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org>
Differential Revision: https://phab.enlightenment.org/D11591

4 years agoevas_textblock: prevent updating cursor unless they are ready during markup_set
Ali [Thu, 26 Mar 2020 05:49:02 +0000 (14:49 +0900)]
evas_textblock: prevent updating cursor unless they are ready during markup_set

Summary:
During Markup_set at text block level, we will not update the cursors, unless their status is updated and ready.

This can cause serious issues, especially if a cursor also depends on another cursor for some calculations, (like the segfault happening in TextBox T8637)

Reviewers: woohyun, bu5hm4n, zmike

Reviewed By: woohyun

Subscribers: cedric, #reviewers, #committers

Tags: #efl

Maniphest Tasks: T8637

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

4 years agoefl_ui_spin_button: add tests for direct input feature
Marcel Hollerbach [Wed, 25 Mar 2020 21:01:16 +0000 (22:01 +0100)]
efl_ui_spin_button: add tests for direct input feature

this simply passes in some key sequences and checks if the validation is
working. Additionally the output value is checked.
Depends on D11009

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

4 years agosuite_helpers: add helper for key down events
Marcel Hollerbach [Wed, 25 Mar 2020 21:01:06 +0000 (22:01 +0100)]
suite_helpers: add helper for key down events

usefull for emulating key events.

4 years agosuite_helper: do not use stacked values here
Marcel Hollerbach [Wed, 25 Mar 2020 09:12:54 +0000 (10:12 +0100)]
suite_helper: do not use stacked values here

this is ending up beeing garbage pointers when not properly overwritten.

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

4 years agoevas_textblock_tests: do not free a text cursor here
Marcel Hollerbach [Mon, 23 Mar 2020 20:53:54 +0000 (21:53 +0100)]
evas_textblock_tests: do not free a text cursor here

the cur_obj would have been always dead at this point, as the textblock
was freed. This now moved the textcursor object to a previous point.
However, we could also remove the explicit cursor deletion...

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

4 years agoefl_check: unset the callback here
Marcel Hollerbach [Mon, 23 Mar 2020 20:52:05 +0000 (21:52 +0100)]
efl_check: unset the callback here

this is needed in order to not accidently overwrite stack values.
This never showed up while running the test suite, as erroring would
have aborted anyways. However, when we are running with CK_FORK=no, this
may leak into another test.

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

4 years agoevas: free font lists correctly
Marcel Hollerbach [Mon, 23 Mar 2020 20:48:25 +0000 (21:48 +0100)]
evas: free font lists correctly

sometimes a canvas object is zombying, and does not get freed correctly.
If this is right now happening in our test suite, we are going to free
the list, but do not clean up the pointer in there.

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

4 years agoedje/multisense: track and destroy multisense inputs on shutdown
Mike Blumenkrantz [Tue, 17 Mar 2020 16:07:49 +0000 (12:07 -0400)]
edje/multisense: track and destroy multisense inputs on shutdown

leaking these means leaking file references, and there's no reason for it

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

4 years agoedje/multisense: free multisense output on shutdown
Mike Blumenkrantz [Tue, 17 Mar 2020 16:07:28 +0000 (12:07 -0400)]
edje/multisense: free multisense output on shutdown

this comment has now been addressed

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

4 years agoecore/audio: rewrite pulseaudio output to not use global variables
Mike Blumenkrantz [Tue, 17 Mar 2020 16:06:13 +0000 (12:06 -0400)]
ecore/audio: rewrite pulseaudio output to not use global variables

this breaks down immediately when calling init/shutdown in quick succession
due to the async nature of pulseaudio. we have object-based private data, so
we can just use that instead

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

4 years agoecore/audio: correctly handle sndfile error when setting new vio
Mike Blumenkrantz [Tue, 17 Mar 2020 16:05:01 +0000 (12:05 -0400)]
ecore/audio: correctly handle sndfile error when setting new vio

need to immediately call the free here to avoid leaking the callback data

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

4 years agoecore/audio: remove duplicated vio code from sndfile input
Mike Blumenkrantz [Tue, 17 Mar 2020 16:04:38 +0000 (12:04 -0400)]
ecore/audio: remove duplicated vio code from sndfile input

this is already done in the base implementation

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

4 years agoecore/audio: add a couple more functions to the pulse callback struct
Mike Blumenkrantz [Tue, 17 Mar 2020 16:03:52 +0000 (12:03 -0400)]
ecore/audio: add a couple more functions to the pulse callback struct

to be used later

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

4 years agoecore/audio: free vio on in/out object destruction
Mike Blumenkrantz [Tue, 17 Mar 2020 16:03:03 +0000 (12:03 -0400)]
ecore/audio: free vio on in/out object destruction

ensure we don't leak these since that means we also leak the user's free
data and callback

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

4 years agoecore/audio: add ERR messages when trying to overwrite existing vio
Mike Blumenkrantz [Tue, 17 Mar 2020 16:02:23 +0000 (12:02 -0400)]
ecore/audio: add ERR messages when trying to overwrite existing vio

this probably causes a crash or a leak at some point

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

4 years agoexactness: handbuild a new argv array instead of reassemling the new one
Marcel Hollerbach [Wed, 25 Mar 2020 14:24:09 +0000 (15:24 +0100)]
exactness: handbuild a new argv array instead of reassemling the new one

this commit removes the code that was changing argv values, and replaces
it with a new array. Which is absolutly fine, as the argv / argc values
are never accessed later on. Only the copies that have been passed to
efl_main or elm_main.

This resolves several issues:
1. the for loop is useless, every single array element that gets
   initialized with it, is some offset from argv[0] this may also crash
   when argv[i] - argv[opt_args] is bigger strlen argv[0].
2. The memcpy here is super dangerous, the dest array is not garanteed
   to have the same size as argv[0], this only works if the client
   application name is shorter than the name "exactness_recorder"
3. The memset here is absolutly wrong. There is again no garantee that
   the array has the expected size behind that, this was constantly
   overwriting the segment after the place where argv was stored, which
   was lukely enough on fedora always the environs, which deleted the
   couple first segments. (This was not causing any fuzz, since they
   have been sudo related env vars on the docker image). However, on
   arch this just crashed right away. On Ubuntu this overwrote DISPLAY,
   which resulted in the unability to launch the app.

Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org>
Differential Revision: https://phab.enlightenment.org/D11600

4 years agocanvas3d: remove all feature implementation in edje/edje_cc
Hermet Park [Fri, 20 Mar 2020 09:00:24 +0000 (09:00 +0000)]
canvas3d: remove all feature implementation in edje/edje_cc

Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Reviewed-by: Shinwoo Kim <cinoo.kim@samsung.com>
Differential Revision: https://phab.enlightenment.org/D11547

Change-Id: Ib9dc06e5c3197a6bdfc45edac2d7a5db1cb64219

4 years agoecore_evas_tbm: add surface_get and stride_get symbols.
Hosang Kim [Mon, 23 Mar 2020 08:17:08 +0000 (17:17 +0900)]
ecore_evas_tbm: add surface_get and stride_get symbols.

Change-Id: If833a6fef1789541be88e5292992aec534ddabc4
(cherry picked from commit 7a8201697f3d41fe3c50482a6befcda6f903f683)

4 years agoemotion - webcam - fix segv on webcam plug/unplug and clean
Carsten Haitzler (Rasterman) [Wed, 25 Mar 2020 16:46:27 +0000 (16:46 +0000)]
emotion - webcam - fix segv on webcam plug/unplug and clean

well hunting was fun... custom webcams i just cant see being used. no
api to add them - have to hand craft a config file .. and udev/eeze
provide info on webcam devices anyway at runtime with plug/unplug etc.
... so this should be the only ay (for now) and it keesp the code
simpler and less bug-prone

now issue was some nasty skipping unref as opposed to destroy. in
chasing i simplified the code to help me narrow it down and not chase
the same logic in multiple places. shorter cleaere, simpler and minux
one bug.

@fix

4 years agoevas/render: minor cleanups for norender functions
Mike Blumenkrantz [Wed, 25 Mar 2020 13:27:16 +0000 (09:27 -0400)]
evas/render: minor cleanups for norender functions

Summary:
evas_norender cannot produce update regions, so there's nothing to free

remove commented lines

no functional changes

Reviewers: bu5hm4n

Reviewed By: bu5hm4n

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

4 years agoecore_cocoa: remove left over from evas3d removal
Stefan Schmidt [Wed, 25 Mar 2020 11:54:39 +0000 (12:54 +0100)]
ecore_cocoa: remove left over from evas3d removal

These directories do not exist any longer. Failing build on CI.

4 years agocanvas 3d: remove all canvas3d features across efl.
Hermet Park [Wed, 25 Mar 2020 10:14:02 +0000 (19:14 +0900)]
canvas 3d: remove all canvas3d features across efl.

Reviewers: jsuya, kimcinoo, bu5hm4n, raster

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

4 years agoefl_ui_image: keep efl_ui_image size for internal image
Shinwoo Kim [Wed, 25 Mar 2020 10:12:16 +0000 (19:12 +0900)]
efl_ui_image: keep efl_ui_image size for internal image

Summary:
The size of internal image could be bigger than the size of efl_ui_image
with following code.

   image = elm_image_add
   elm_image_fill_outside_set(image, EINA_TRUE);

If the internal image object is 300x300, and efl_ui_image is 360x77, then
the internal image object will resize to 360x360 which is bigger than the
size of efl_ui_image.

This is a compatibility issue. This patch will make efl_ui_image work as
before commit 8cb6c3e Elm_image: implement 'scale_type' and 'scalable'...

Test Plan:
This is old example to reproduce the problem.
{F3859361}

This is newly added example to check if this patch breaks compatibility or not.
{F3859390}

You can use the example with following image.
{F3859391}

This is result before applying this patch.
{F3859388}

This is result after applying this patch.
{F3859389}

The translucent rectangle is the size of the efl_ui_image.
As you might be noticed, only FIT_WIDTH (the 4th one of each row),
and EXPAND (the 6th one of each row) are different.

One more difference the 1st one of 2nd row; NONE.

F.Y.I. and for quick understanding of example.
The 1st row efl_ui_image is bigger than internal image.
The 2nd row efl_ui_image is smaller than internal image.
From the left the scale type is NONE, FILL, FIT, FIT_WIDTH,
FIT_HEIGHT, EXPAND, and TILE.

Reviewers: Hermet, jsuya, herb

Reviewed By: Hermet

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

4 years agoATSPI bridge refactoring 71/228671/2
Bartlomiej Grzelewski [Tue, 24 Mar 2020 11:08:03 +0000 (12:08 +0100)]
ATSPI bridge refactoring

Add support for Text Interactive interface.

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

Change-Id: I9b862608cacbcbc3c29af0e515bf6364de05f829

4 years agoelm_entry: use legacy focus function for legacy widget
Yeongjong Lee [Tue, 29 Oct 2019 01:24:31 +0000 (10:24 +0900)]
elm_entry: use legacy focus function for legacy widget

Since focus manager is not used in tizen, we should use elm_widget_focus_get
instead of efl_ui_focus_object_focus_get.

@tizen_fix

Change-Id: I9aa7da4ec33551eb653d90eb16e0a62e629d719b

4 years ago[Elm AT-SPI Bridge] Replaced service name validation with sanitization.
Maria Bialota [Thu, 19 Mar 2020 18:04:50 +0000 (19:04 +0100)]
[Elm AT-SPI Bridge] Replaced service name validation with sanitization.
This is to enable proper handling of services with names containing non-standard characters such as @.

Change-Id: Iff2ee845648c6e40fd9c950e7fb02ea331cff818

4 years agoefl/mailmap: Move unused addresses to the bottom of list
Christopher Michael [Tue, 24 Mar 2020 20:57:35 +0000 (16:57 -0400)]
efl/mailmap: Move unused addresses to the bottom of list

4 years agoRevert "efl/mailmap: Remove old unused addresses"
Christopher Michael [Tue, 24 Mar 2020 20:52:29 +0000 (16:52 -0400)]
Revert "efl/mailmap: Remove old unused addresses"

Revert these in favor of reorder so we can keep history

This reverts commit 94a80803558a682e78b53e190b23c32b54200c65.

4 years agoefl/mailmap: Remove old unused addresses
Christopher Michael [Tue, 24 Mar 2020 20:17:52 +0000 (16:17 -0400)]
efl/mailmap: Remove old unused addresses