platform/upstream/efl.git
4 years agoRevert "ui scroller: fix broken scroller loop" 52/236652/2
Shinwoo Kim [Fri, 19 Jun 2020 05:08:23 +0000 (14:08 +0900)]
Revert "ui scroller: fix broken scroller loop"

This reverts commit 036423307c7eafec904d650a71ef64dac646151a.

The commit 76b89b8 "evas proxy: use part of source" was reverted.
So we need to revert "ui scroller: fix broken scroller loop" to
avoid build break.
New commit using evas_object_image_load_region_set will handle.

Change-Id: Ibed05025c028555dc490f7b0bdd707c104b17943

4 years agoRevert "evas proxy: use part of source" 32/236632/2
Shinwoo Kim [Fri, 19 Jun 2020 03:14:16 +0000 (12:14 +0900)]
Revert "evas proxy: use part of source"

This reverts commit 76b89b8e49f4b16ad490c04134a9bdddea27950f.

The commit was for fix an issue quickly.
Now we are able to rever the commit because
evas_object_image_load_region_set is available.
New commit using evas_object_image_load_region_set is comming.

Change-Id: I5f8cf04f92065db22342521b9102e6d26d5121d6

4 years agoeina_unicode: have explicit type conversions accepted/tizen/unified/20200622.134453 submit/tizen/20200621.214257
Boris Faure [Sat, 20 Jun 2020 10:37:56 +0000 (11:37 +0100)]
eina_unicode: have explicit type conversions

Summary:
Found by running terminology's tests with UBSAN:
include/eina-1/eina/eina_inline_unicode.x:
runtime error: implicit conversion from type 'char' of value
-62 (8-bit, signed) to type 'unsigned char' changed the value to 194
(8-bit, unsigned)

Reviewers: #reviewers, vtorri

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

4 years agoefreet caceh - fix coverity report
Carsten Haitzler (Rasterman) [Thu, 18 Jun 2020 23:56:08 +0000 (00:56 +0100)]
efreet caceh - fix coverity report

fix CID 1429935

4 years agomeson: add Solaris support
Vincent Torri [Thu, 18 Jun 2020 13:35:46 +0000 (14:35 +0100)]
meson: add Solaris support

Summary: Add Solaris support for meson

Test Plan: test on OpenIndiana

Reviewers: raster, bu5hm4n, stefan_schmidt

Reviewed By: raster, stefan_schmidt

Subscribers: alarcher, stefan_schmidt, bu5hm4n, cedric, #reviewers, #committers

Tags: #efl

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

4 years agotests: eina: fix ENOMEM message on Windows and solaris
Vincent Torri [Tue, 16 Jun 2020 21:47:43 +0000 (21:47 +0000)]
tests: eina: fix ENOMEM message on Windows and solaris

On Windows and solaris the string associated to ENOMEM is "Not enough space"

Reviewed-by: João Paulo Taylor Ienczak Zanette <joao.tiz@expertisesolutions.com.br>
Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org>
Differential Revision: https://phab.enlightenment.org/D11983

4 years agoefreet - mtime 0 fix build on windows
Carsten Haitzler (Rasterman) [Thu, 18 Jun 2020 10:35:44 +0000 (11:35 +0100)]
efreet - mtime 0 fix build on windows

4 years agoefreetd - cache - add more statinfo work around 0 mtime distros
Carsten Haitzler (Rasterman) [Thu, 18 Jun 2020 10:03:02 +0000 (11:03 +0100)]
efreetd - cache - add more statinfo work around 0 mtime distros

some distros 9notably in this case nixos) want to do reproducible
builds. to them this means going around setting mtime for all files to
0. this means efreetd not only thinks mtime is invalid/stupid (0 is
generally just that as midnight on jan 1 1970 is not exactly a
sensible dare for a modified timestamp of a file as no filesystem with
any sanity will have not been modified since that time), but it keeps
mtime at 0 even when things update. this totally breaks efreetd that
expects to find mtime increases over time as things change. it's
necessary because it has to perform a "are mu caches up to date" scan
of all file data it caches and it needs to know if it should rebuild
something based on this.

so this does a few things:

1. it makes mtime have to be an exact match to the cache, not cache
mtime >= file mtime. so any change forward or back is an inavlidation.
2. it now also uses ctime, mode, size, uid, gid, block count and if a
symlink, the sha1 of the symlink path in addition and any change to
these == invalid.

this adds a lot of code and changes how dirs get scanned a bit but it
means it can pick up changes on these 0 mtime distros.

interestingly the policy of mtime being 0 is to have a reprodcible fs
... but ctime still changes and is > 0, as does inode info, so it's
not actually possible to have it totally work... but they try still,
so this is a fix for that problem.

whilst i was doing thisi also noticed efreetd re-red dirs many times
due to icon theme inhritance. i also fixed that to do a LOT less
syscalls by only scanning a dir once as i was rejigging the scanning
code at the time anyway. this should optimize thr scan costs at
efreetd startup too.

@fix
@opt

4 years agoRevert "Revert "eldbus: change to idle_enterer from idler"" 49/227549/3
Artur Świgoń [Mon, 23 Mar 2020 11:33:52 +0000 (12:33 +0100)]
Revert "Revert "eldbus: change to idle_enterer from idler""

This reverts commit 2bcc7b48bf2f34a19b0712b030ab143d09dc30fb.

Handling DBus messages during idle carries the risk of starving the DBus
communication (and making the UI unresponsive if screen reader is used). This
is well-explained in the original (reverted) upstream commit: "If an
application does not give idle time, then the eldbus cannot have chance to
work."

Change-Id: Iffde1be749f5dba1ac1e8642c4ce4e3436770b5a

4 years agobenchmarks: eina: make sure we do not divide by zero accepted/tizen/unified/20200618.130024 submit/tizen/20200617.214046
Stefan Schmidt [Tue, 9 Jun 2020 12:22:19 +0000 (14:22 +0200)]
benchmarks: eina: make sure we do not divide by zero

Make sure we do not divide by i if it is zero here.

CID: 1400768

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

4 years agotests: eio: make sure we check return value
Stefan Schmidt [Wed, 10 Jun 2020 10:30:48 +0000 (12:30 +0200)]
tests: eio: make sure we check return value

Make sure we fail the test if the call does not succeed.

CID: 1412364

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

4 years agoevas gl common - remove comment - not valid anymore
Carsten Haitzler (Rasterman) [Wed, 17 Jun 2020 11:30:23 +0000 (12:30 +0100)]
evas gl common - remove comment - not valid anymore

4 years agogl common - actually allow vertex limits
Carsten Haitzler (Rasterman) [Wed, 17 Jun 2020 11:17:13 +0000 (12:17 +0100)]
gl common - actually allow vertex limits

fix non-commented out return that should have been commented out

4 years agoefl_ui_widget: addition to the previous commit
Marcel Hollerbach [Wed, 17 Jun 2020 07:13:05 +0000 (09:13 +0200)]
efl_ui_widget: addition to the previous commit

in the previous commit we started to only receive the parent_obj from
the evas object when its not a widget. However, we still need to ensure
that the parent is equal to the current one in order to remove it when a
widget.

While this is one more call, the call to get the parent is not as
heavy as the data_get call, as this directly addresses a struct, and
does not read from a hash table.

This also fixes a test case failure.

4 years agoelementary/perf: reduce number of efl_isa() call. in elementary test application...
Subhransu Mohanty [Wed, 17 Jun 2020 06:28:44 +0000 (06:28 +0000)]
elementary/perf: reduce number of efl_isa() call. in elementary test application efl_isa() shows up as hotsport during profiling so refactored the code to reduce redundant efl_isa() calls.

in genlist testcase the number of calls reduced by 30000.

Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Reviewed-by: Hermet Park <<hermetpark@gmail.com>>
Reviewed-by: Youngbok Shin <youngb.shin@samsung.com>
Differential Revision: https://phab.enlightenment.org/D11984

4 years agoelementary/perf:remove unnecessary elm_widget_is() check
Subhransu Mohanty [Wed, 17 Jun 2020 06:36:02 +0000 (06:36 +0000)]
elementary/perf:remove unnecessary elm_widget_is() check

As we always make sure the parent is a widget class we don't have to check anymore.

Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Reviewed-by: Hermet Park <<hermetpark@gmail.com>>
Differential Revision: https://phab.enlightenment.org/D11985

4 years agoelm_image: remove the spaces and keep the indentation for elm_image_file_set() 00/236500/1
Taehyub Kim [Wed, 17 Jun 2020 12:17:23 +0000 (21:17 +0900)]
elm_image: remove the spaces and keep the indentation for elm_image_file_set()

Summary: there are wrong indentation in the elm_image_file_set, so removed the tabs

Reviewers: Jaehyun_Cho

Reviewed By: Jaehyun_Cho

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

Change-Id: I797d0861fff02a8b49aacc748cec9c7238926441

4 years agoelm_image: keep backword compatibility for elm_image_file_set API when setting url... 99/236499/1
Taehyub Kim [Wed, 17 Jun 2020 12:04:08 +0000 (21:04 +0900)]
elm_image: keep backword compatibility for elm_image_file_set API when setting url file set twice

Summary:
when trying to set file using url path twice, the second api call's return value is EINA_FALSE
since image obj has already has same file path. After applying Efl.File interface, the behavior is changed compared to before.
both of the return values should be EINA_TRUE.
@fix

ex)
Eina_Bool ret1, ret2;
ret1 = elm_image_file_set(image, "http://sameurl/image.jpg", NULL);
ret2 = elm_image_file_set(image, "http://sameurl/image.jpg", NULL);
ret1 and ret2 should be EINA_TURE

Test Plan:
1. call elm_image_file_set api with same url path
2. see the return value

Reviewers: Hermet, kimcinoo, jsuya

Reviewed By: Hermet

Subscribers: bu5hm4n, cedric, #reviewers, #committers

Tags: #efl

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

Change-Id: If64d8d58424b559ea07dba14efb36996484c2fd5

4 years agoefl.spec: enable WebP Animation Feature 70/236370/3
Taehyub Kim [Tue, 16 Jun 2020 12:16:40 +0000 (21:16 +0900)]
efl.spec: enable WebP Animation Feature

libwebp-1.1.0 library is included in the binary after tizen_unified_20200528.2
WebP Animation can be enabeld now.

Change-Id: I20b17049f7caca332a1af7ceb5c4e32e252bdaa5

4 years agomodules: eina: chained_pool: check for NULL before passing it to deref submit/tizen/20200616.213914
Stefan Schmidt [Tue, 16 Jun 2020 09:03:26 +0000 (11:03 +0200)]
modules: eina: chained_pool: check for NULL before passing it to deref

Make sure we check if pool->first_fill is NULL before passing it onwards
where it gets dereferenced. In case this happens return NULL instead of
garbage.

CID: 1293520

Reviewed-by: Christopher Michael <devilhorns@comcast.net>
Differential Revision: https://phab.enlightenment.org/D11980

4 years agoRevert "ci: travis: disable the webp loader on CI builds for now"
Stefan Schmidt [Tue, 16 Jun 2020 14:19:04 +0000 (16:19 +0200)]
Revert "ci: travis: disable the webp loader on CI builds for now"

This reverts commit 29c491b977543e9ef86147604ab3a297ece18327.

We went back to a much older version which should be available on all
our CI test systems. Safe to enable again.

4 years agoevas_object_table: fix not to use hint_fill on elm_table 65/236365/1
Jaehyun Cho [Tue, 16 Jun 2020 11:40:59 +0000 (20:40 +0900)]
evas_object_table: fix not to use hint_fill on elm_table

Summary:
The hint_fill logic in evas_object_table was designed to be used for
Efl.Ui.Table not for elm_table.
Since Efl.Ui.Table does not use evas_object_table logic, hint_fill logic
in evas_object_table is removed.

Reviewers: YOhoho

Reviewed By: YOhoho

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

Change-Id: Ide4fe092d9eeee4120596bae3dc3cb2eab2a1da5

4 years agoefl : Fix build warnings. 74/236174/4
Woochanlee [Mon, 15 Jun 2020 09:55:45 +0000 (18:55 +0900)]
efl : Fix build warnings.

Change-Id: I0777c22ad7784beee94646cf17420e41b0e798e8

4 years agoEfl.Canvas.Vg.Object: Fix wrong render area 81/236281/2
JunsuChoi [Tue, 16 Jun 2020 04:10:25 +0000 (13:10 +0900)]
Efl.Canvas.Vg.Object: Fix wrong render area

Summary:
When determining the size of the ector buffer,
it was used the smaller of the object's geometry or path_bounds.
However, because of that, path worked as absolute coordinates.
path_bounds should be relative to geometry of object.

Test Plan:
{F3900444}
original
{F3900445}

before
+ evas_object_geometry_set(vg, 100, 100, 200, 200);
{F3900447}

after
+ evas_object_geometry_set(vg, 100, 100, 200, 200);
{F3900448}

Reviewers: Hermet, kimcinoo, herb, smohanty

Reviewed By: Hermet

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

Change-Id: Id190aeb88735be0b9cbb5f60879c6a8a63eb33e7

4 years agodoxygen: remove unsupported links to preview images 83/236283/1
Hosang Kim [Mon, 15 Jun 2020 10:41:03 +0000 (19:41 +0900)]
doxygen: remove unsupported links to preview images

Change-Id: I936462469e6713268fb8a41e0d10f77b4c78a424

4 years agotests: edje: check result of evas_engine_info_set()
Stefan Schmidt [Mon, 15 Jun 2020 11:20:07 +0000 (13:20 +0200)]
tests: edje: check result of evas_engine_info_set()

Make sure we check the result of evas_engine_info_set and only continue
if it succeeds.

CID: 1401077

Reviewed-by: Christopher Michael <devilhorns@comcast.net>
Differential Revision: https://phab.enlightenment.org/D11978

4 years agomodules: gl_x11: ensure we check the result of evas_eglMakeCurrent()
Stefan Schmidt [Mon, 15 Jun 2020 10:48:37 +0000 (12:48 +0200)]
modules: gl_x11: ensure we check the result of  evas_eglMakeCurrent()

In case we fail make sure we error out to allow better debugging.
Also fix some surrounding indent.

CID: 1402693, 1402670

Reviewed-by: Christopher Michael <devilhorns@comcast.net>
Differential Revision: https://phab.enlightenment.org/D11976

4 years agoelm_atspi_bridge: call _elm_win_atspi only once 80/236080/2
Kamil Konieczny [Fri, 12 Jun 2020 13:37:50 +0000 (15:37 +0200)]
elm_atspi_bridge: call _elm_win_atspi only once

Change-Id: I843c37646f60dfad985db66ee49838a68c239c25

4 years agoefl_ui_win_socket: fix socket_proxy release 78/236078/3
Kamil Konieczny [Fri, 12 Jun 2020 13:18:54 +0000 (15:18 +0200)]
efl_ui_win_socket: fix socket_proxy release

socket_proxy was created with efl_add method, so use efl_del
instead of efl_unref for object release.

Change-Id: I81a470f1acc8d803aa1146dd0d6b8a1663c797ec

4 years agoelm_atspi_bridge: fix memory leak 79/236079/2
Kamil Konieczny [Fri, 12 Jun 2020 13:30:16 +0000 (15:30 +0200)]
elm_atspi_bridge: fix memory leak

Fix memory leak on error path.

Change-Id: I314f200ec20f2f0810d15ed1b6ae9fec1597e775

4 years agowebp - 0.5.0 is actually new enough submit/tizen/20200614.220342
Carsten Haitzler (Rasterman) [Sun, 14 Jun 2020 00:09:04 +0000 (01:09 +0100)]
webp - 0.5.0 is actually new enough

webp anim apis came out in 0.5.0 not 1.1.0 :)

4 years agoevas/engines/gl_generic: fix byte order after glReadPixels on BE
Daniel Kolesa [Fri, 12 Jun 2020 16:46:33 +0000 (18:46 +0200)]
evas/engines/gl_generic: fix byte order after glReadPixels on BE

This fixes the screenshot tool in Enlightenment on big endian
systems besides other things.

4 years agomeson_options: really fix the typo
Stefan Schmidt [Fri, 12 Jun 2020 14:03:48 +0000 (16:03 +0200)]
meson_options: really fix the typo

Sigh, check your patch, even if it appears to be tiny.

4 years agomeson_options: fix typo in option description
Stefan Schmidt [Fri, 12 Jun 2020 14:02:43 +0000 (16:02 +0200)]
meson_options: fix typo in option description

4 years agomodules: vnc: allow building for FreeBSD with a different header path
Stefan Schmidt [Fri, 12 Jun 2020 11:25:01 +0000 (13:25 +0200)]
modules: vnc: allow building for FreeBSD with a different header path

Original patch provided by phyb user CeDeROM.

Fixes T8713

4 years agobuild: decrease the minimal needed meson verion to 0.50
Stefan Schmidt [Fri, 12 Jun 2020 07:32:39 +0000 (09:32 +0200)]
build: decrease the minimal needed meson verion to 0.50

When we jumped from 0.47 to 0.52 to fix problems some users have been
seeing when building we might jumped a bit to far. We want a good
balance between stable and fast building but also mature enough to be in
distro releases and widely available.

We will try to settle for a middle-ground with 0.50 here. If that causes
problems for you building please let us know instantly.

4 years agoefl_ui_win: reuse bg which was created for version check 51/236051/1
WooHyun Jung [Fri, 12 Jun 2020 07:57:45 +0000 (16:57 +0900)]
efl_ui_win: reuse bg which was created for version check

Summary: This is just for not creating bg object twice.

Reviewers: bu5hm4n, zmike, YOhoho

Reviewed By: YOhoho

Subscribers: YOhoho, cedric, #reviewers, #committers

Tags: #efl

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

Change-Id: I548a3f12a728d90212d47c110b2d1904b3d472b7

4 years agoui image: fix a missing else case. 46/236046/1
Hermet Park [Fri, 12 Jun 2020 07:33:21 +0000 (16:33 +0900)]
ui image: fix a missing else case.

it sholud be turned on enabled status.

it's side issue by 0826b4916efb0bce20a353cece56e84d05fb406c

Change-Id: I84b06594fda8694f865c74db4bdabfda505665c3

4 years agoefl_ui_image: fixed elm_image_preload_disabled_set api is not working when it is... 44/236044/1
Taehyub Kim [Fri, 12 Jun 2020 07:14:56 +0000 (16:14 +0900)]
efl_ui_image: fixed elm_image_preload_disabled_set api is not working when it is called before file set

Summary:
when elm_image_preload_disabled_set(img, EINA_TRUE) is called before file set.
The image preload is not canceled, because the function will be returned since the image is not loaded.
so preload disabled api is fixed to work even if it is called before file set.
@fix

Reviewers: Hermet, kimcinoo, jsuya

Reviewed By: Hermet

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

Change-Id: I72e559314dbe494bbcf1dddf91f309ff8772780c

4 years agotests: elm: check return of efl_config_bool_set submit/tizen/20200611.221942
Stefan Schmidt [Thu, 11 Jun 2020 11:35:52 +0000 (13:35 +0200)]
tests: elm: check return of efl_config_bool_set

Make sure we fail the test, unless this settign succeed.

CID: 1403891

Reviewed-by: Christopher Michael <devilhorns@comcast.net>
Differential Revision: https://phab.enlightenment.org/D11968

4 years agomodules: only build gl_drm evas engine when using es-egl
Daniel Kolesa [Thu, 11 Jun 2020 13:14:03 +0000 (15:14 +0200)]
modules: only build gl_drm evas engine when using es-egl

This module does not work with desktop opengl contexts,
and needs EGL to work.

4 years agoedje_cc_handlers: update documentation of "fixed" to be clearer
Jaehyun Cho [Thu, 11 Jun 2020 09:57:30 +0000 (18:57 +0900)]
edje_cc_handlers: update documentation of "fixed" to be clearer

Summary:
Since "fixed" does not affect the group size by the part's min size, the
documentation is updated to be clearer.

Reviewers: woohyun, raster

Reviewed By: woohyun

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

4 years agoelm_calendar: Do not pass NULL to function.
Woochanlee [Thu, 11 Jun 2020 07:34:16 +0000 (16:34 +0900)]
elm_calendar: Do not pass NULL to function.

Summary:
T7076

legacy calendar inc/dec button has auto repeat feature.
for that if user click the button very quickly, we delete internal timer than it will pass to function.

to prevent this, need to check NULL pointer.

Reviewers: devilhorns, Hermet

Reviewed By: Hermet

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

4 years agoecore: use standard LC_ALL instead of __LC_ALL in systemd module
Daniel Kolesa [Thu, 11 Jun 2020 00:32:07 +0000 (02:32 +0200)]
ecore: use standard LC_ALL instead of __LC_ALL in systemd module

The former is a POSIX name, the latter is non-standard. I don't
know why the latter was used, considering glibc literally has
just #define LC_ALL __LC_ALL, but change it and unbreak build
on musl and other systems.

4 years agoeina: only enable EINA_LOG_BACKTRACE when backtrace API is present
Daniel Kolesa [Thu, 11 Jun 2020 00:18:50 +0000 (02:18 +0200)]
eina: only enable EINA_LOG_BACKTRACE when backtrace API is present

This prevents build breakage on platforms that either don't have
backtrace() or don't have the appropriate library for it installed.

4 years agoevas glengine: fix code sync after migration. 43/235943/1 accepted/tizen/unified/20200612.022321 submit/tizen/20200611.072014
Hermet Park [Thu, 11 Jun 2020 07:10:36 +0000 (16:10 +0900)]
evas glengine: fix code sync after migration.

Introduced a wrong result after upstream migration,
since we have tizen only code diff....

Change-Id: Ia4bac99a244a63a1d54c63dd93aea80c5a0bfce0

4 years agoefl - build again - test suite break in using undefined ERR macro submit/tizen/20200610.214513
Carsten Haitzler (Rasterman) [Wed, 10 Jun 2020 18:30:14 +0000 (19:30 +0100)]
efl - build again - test suite break in using undefined ERR macro

4 years agotests: eina: make sure we always check on eina_matrixsparse_data_idx_get return
Stefan Schmidt [Tue, 9 Jun 2020 13:21:28 +0000 (15:21 +0200)]
tests: eina: make sure we always check on eina_matrixsparse_data_idx_get return

It has been checked in all other places, but has been forgotten in these
two.

CID: 1401081

Reviewed-by: Christopher Michael <devilhorns@comcast.net>
Differential Revision: https://phab.enlightenment.org/D11960

4 years agotests: eo: handle error return of eina_value_int_convert()
Stefan Schmidt [Tue, 9 Jun 2020 13:09:08 +0000 (15:09 +0200)]
tests: eo: handle error return of eina_value_int_convert()

CID: 1400962

Reviewed-by: Christopher Michael <devilhorns@comcast.net>
Differential Revision: https://phab.enlightenment.org/D11958

4 years agotests: eina: also handle failure cases when cleaning up created files/folders
Stefan Schmidt [Tue, 9 Jun 2020 12:04:31 +0000 (14:04 +0200)]
tests: eina: also handle failure cases when cleaning up created files/folders

Somethign fishy is going on if we can not delete the files and folders
we created.

CID: 1400986

Reviewed-by: Christopher Michael <devilhorns@comcast.net>
Differential Revision: https://phab.enlightenment.org/D11955

4 years agoecore_drm: check return of drmModeSetCrtc()
Stefan Schmidt [Wed, 10 Jun 2020 12:12:59 +0000 (14:12 +0200)]
ecore_drm: check return of drmModeSetCrtc()

Report if we fail to reset the Crtc during output free. Only print an
error and keep going with the rest of the output free.

CID: 1402668

Reviewed-by: Christopher Michael <devilhorns@comcast.net>
Differential Revision: https://phab.enlightenment.org/D11965

4 years agotests/ecore_wl2: Fix unchecked return value
Christopher Michael [Wed, 10 Jun 2020 12:38:02 +0000 (08:38 -0400)]
tests/ecore_wl2: Fix unchecked return value

Coverity reports that we do not check the return of eglInitialize
here.

Fixes Coverity CID1412362

4 years agotests/ecore_wl2: Fix unchecked return value
Christopher Michael [Wed, 10 Jun 2020 12:32:40 +0000 (08:32 -0400)]
tests/ecore_wl2: Fix unchecked return value

Coverity reports eglChooseConfig is called without checking return
value here. This patch fixes the issue by checking the return value
and by also making _init_egl return a bool so that we can fail the
test if eglChooseConfig results in failure.

Fixes Coverity CID1412365

4 years agoecore_wl2_tests: Fix formatting
Christopher Michael [Wed, 10 Jun 2020 12:26:29 +0000 (08:26 -0400)]
ecore_wl2_tests: Fix formatting

4 years agotests: eina: check return of eina_module_load and unload
Stefan Schmidt [Tue, 9 Jun 2020 13:17:36 +0000 (15:17 +0200)]
tests: eina: check return of eina_module_load and unload

These could fail as well. Check in testsuite.

CID: 1400961

Reviewed-by: Christopher Michael <devilhorns@comcast.net>
Differential Revision: https://phab.enlightenment.org/D11959

4 years agotests: evas: check fd being non -1 before using close()
Stefan Schmidt [Tue, 9 Jun 2020 12:29:35 +0000 (14:29 +0200)]
tests: evas: check fd being non -1 before using close()

eina_file_mkstemp would return -1 in an error case. Make sure we check
the return when clsoing here.

CID: 1400790

Reviewed-by: Christopher Michael <devilhorns@comcast.net>
Differential Revision: https://phab.enlightenment.org/D11957

4 years agoUse extern after EAPI
Vincent Torri [Wed, 10 Jun 2020 05:07:47 +0000 (05:07 +0000)]
Use extern after EAPI

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

4 years agoevas stack: ++safe code. 71/235871/1
Hermet Park [Wed, 10 Jun 2020 11:30:19 +0000 (20:30 +0900)]
evas stack: ++safe code.

null check to prevent any crashes (just in case)

Change-Id: Ic77a2c785feba9be32eb1460439f4c3315087e07

4 years agoui textpath: updated doc. 38/235838/1
Hermet Park [Wed, 10 Jun 2020 06:49:49 +0000 (15:49 +0900)]
ui textpath: updated doc.

Change-Id: I562734a64b5ad8f371c1952a996a80f54a05e0c7

4 years agoefl_ui_textpath: add the efl_ui_textpath_text_user_style_set API for applying the... 37/235837/1
Taehyub Kim [Wed, 10 Jun 2020 06:39:03 +0000 (15:39 +0900)]
efl_ui_textpath: add the efl_ui_textpath_text_user_style_set API for applying the user text style

Summary:
For the text style customizing, added the new API to apply user text style.
The following example is the user style sample.

DEFAULT='font_size=16 color=#F00 underline=on underline_color=#00ffff'";

{F3899541}

Test Plan:
1. run elementary_text (textpath)
2. toggle user style check box
3. see the result

Reviewers: Hermet, kimcinoo, jsuya, bu5hm4n

Reviewed By: Hermet

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

Change-Id: Ia3a182aebadcad24383c3021e358d4e40e5e9515

4 years agomono: update window construction submit/tizen/20200609.215559
Marcel Hollerbach [Tue, 9 Jun 2020 10:09:25 +0000 (12:09 +0200)]
mono: update window construction

they simply missed parameters here

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

4 years agoci: travis: disable the webp loader on CI builds for now
Stefan Schmidt [Tue, 9 Jun 2020 07:11:52 +0000 (09:11 +0200)]
ci: travis: disable the webp loader on CI builds for now

With the bump of the needed libwebp version to 1.1.0 we have have our
Debian and Ubuntu builds failing with a to old version. Disable by
default in CI for now and check back later.

4 years agoefreet - handle runtime relocation right with default XDG_DATA_DIRS
Carsten Haitzler (Rasterman) [Tue, 9 Jun 2020 09:13:04 +0000 (10:13 +0100)]
efreet - handle runtime relocation right with default XDG_DATA_DIRS

XDG_DATA_DIRS was only set up to a default including where efl was
installed prefix-wise as the compiled-=in prefix, not runtime
determined prefix. it shouldn't actually affect most people except
those making use of this.

@fix

4 years agoeina - prefix - use smaller buffers for building strings
Carsten Haitzler (Rasterman) [Sun, 7 Jun 2020 00:33:43 +0000 (01:33 +0100)]
eina - prefix - use smaller buffers for building strings

use less stack space but no features as buffers are big enough for all
content used or alloca'd now.

4 years agoeina - abstratc content - smaller stack buffer for limited size string
Carsten Haitzler (Rasterman) [Sun, 7 Jun 2020 00:28:18 +0000 (01:28 +0100)]
eina - abstratc content - smaller stack buffer for limited size string

no need for a 4k buffer when 128 bytes will be plenty - short string.

4 years agoefl_ui_win: declare constructors not optional
Marcel Hollerbach [Sat, 18 Apr 2020 17:43:11 +0000 (19:43 +0200)]
efl_ui_win: declare constructors not optional

they cannot be optional, or in later languages types will collide.

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

4 years agocxx: Fix uses of intrinsic eolian binbuf type
Felipe Magno de Almeida [Mon, 8 Jun 2020 21:05:49 +0000 (18:05 -0300)]
cxx: Fix uses of intrinsic eolian binbuf type

Add special binbuf and Eina_Strbuf conversions

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

4 years agoeo-files: convert all Eina.Strbuf & Eina.Binbuf
Marcel Hollerbach [Sun, 19 Apr 2020 13:14:12 +0000 (15:14 +0200)]
eo-files: convert all Eina.Strbuf & Eina.Binbuf

these structs should not be used, but rather the builtin types that
exist for it.

Reviewed-by: Felipe Magno de Almeida <felipe@expertisesolutions.com.br>
Differential Revision: https://phab.enlightenment.org/D11742

4 years agodotnet: Ignore Efl.Object.parent as constructor
Felipe Magno de Almeida [Mon, 8 Jun 2020 21:32:22 +0000 (18:32 -0300)]
dotnet: Ignore Efl.Object.parent as constructor

In C# we already have Efl.Object.parent as an implicit constructor. Ignore it if it is marked as a constructor.

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

4 years agoeo: make the parent a optional constructor
Marcel Hollerbach [Sun, 19 Apr 2020 19:34:17 +0000 (21:34 +0200)]
eo: make the parent a optional constructor

this is needed in order to sanely bind parent calls to some objejcts,
but not all.

However, from the canvas structure up we *need* a parent, so we can
fetch the evas from it. So declare it there a none optional

Reviewed-by: Felipe Magno de Almeida <felipe@expertisesolutions.com.br>
Differential Revision: https://phab.enlightenment.org/D11734

4 years agoRevert "ci: disable wepb loader until we have a change to update the mingw cross...
Stefan Schmidt [Tue, 9 Jun 2020 07:43:57 +0000 (09:43 +0200)]
Revert "ci: disable wepb loader until we have a change to update the mingw cross env"

This reverts commit 63b1334e2c98f0c0f08801ce77d66f372e7abb30.

No longer needed as we have updated the mingw image to have 1.1.0
available.

4 years agoevas_vg_cache: Don't caching vg file if value_provider is applied.
JunsuChoi [Tue, 9 Jun 2020 01:57:20 +0000 (10:57 +0900)]
evas_vg_cache: Don't caching vg file if value_provider is applied.

Summary:
value_provider can change the value of the property received from VG file.
When a file is cached, the changed properties are applied to all other objects using the same file.
So. If value provider is applied, evas_vg_cache is not caching vg file.

Test Plan: N/A

Reviewers: Hermet, herb, kimcinoo

Reviewed By: Hermet

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

4 years agoevas_common_font: release reallocated glyphs bitmaps data
Ali Alzyod [Tue, 9 Jun 2020 01:25:10 +0000 (10:25 +0900)]
evas_common_font: release reallocated glyphs bitmaps data

Reviewers: woohyun, smohanty

Reviewed By: woohyun

Subscribers: cedric, #reviewers, #committers

Tags: #efl

Maniphest Tasks: T8743

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

4 years agodoxygen docs: Fix API reference group to Ecore_Eo and eina_vector 03/235703/1
Myoungwoon Roy, Kim [Tue, 9 Jun 2020 05:08:11 +0000 (14:08 +0900)]
doxygen docs: Fix API reference group to Ecore_Eo and eina_vector

Change-Id: Ic20831f0db719f935dbf4e7421c70f7c4cdbd5cb

4 years agoelementary/example: Fix memory leak in gengrid test accepted/tizen/unified/20200610.015600 submit/tizen/20200608.220424
Subhransu Mohanty [Mon, 8 Jun 2020 11:28:14 +0000 (20:28 +0900)]
elementary/example: Fix memory leak in gengrid test

Reviewers: Hermet, SanghyeonLee

Reviewed By: Hermet, SanghyeonLee

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

4 years agoecore_ipc: Fix memory leak
Subhransu Mohanty [Mon, 8 Jun 2020 11:27:42 +0000 (20:27 +0900)]
ecore_ipc: Fix memory leak

Summary:
As delete_me flag was not updated in the delete callback it dosen't
call ecore_ipc_client_del() on the client hence the leak.

Reviewers: Hermet, raster, bu5hm4n, CHAN

Reviewed By: Hermet, CHAN

Subscribers: CHAN, cedric, #reviewers, #committers

Tags: #efl

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

4 years agocanvas gl: remove out-dated compiler keywords
Hermet Park [Mon, 8 Jun 2020 10:46:31 +0000 (19:46 +0900)]
canvas gl: remove out-dated compiler keywords

4 years agoexactness: record and print test time on success
Stefan Schmidt [Thu, 4 Jun 2020 10:52:07 +0000 (12:52 +0200)]
exactness: record and print test time on success

Having the runtime of a test is interesting for debugging and
optimization. Printing them only on success for now.

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

4 years agoci: travis: add more fold and timing markers in some forgotten places
Stefan Schmidt [Tue, 2 Jun 2020 14:03:00 +0000 (16:03 +0200)]
ci: travis: add more fold and timing markers in some forgotten places

In teh initial patch we forgot some interesting places we also want to
fold and get timing info on. Dependency installation, cross native build
step, coverity tools download and codecov run.

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

4 years agoci: simplify ccache setup handling
Stefan Schmidt [Thu, 4 Jun 2020 13:19:47 +0000 (15:19 +0200)]
ci: simplify ccache setup handling

We no longer need the special case for release ready. This was needed
for the way autotools setup directories, which we no longer use. (We are
not having a config.log anymore)

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

4 years agoci: disable wepb loader until we have a change to update the mingw cross env
Stefan Schmidt [Mon, 8 Jun 2020 07:10:51 +0000 (09:10 +0200)]
ci: disable wepb loader until we have a change to update the mingw cross env

We now need webp 1.1.0 and our current mingw cross env only has 1.0.2.
Disable to keep CI working while we update.

4 years agowebp: add version check in configuration time
WooHyun Jung [Mon, 8 Jun 2020 02:03:07 +0000 (11:03 +0900)]
webp: add version check in configuration time

Reviewers: Hermet, herb, bu5hm4n, stefan_schmidt

Reviewed By: stefan_schmidt

Subscribers: stefan_schmidt, cedric, #reviewers, #committers

Tags: #efl

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

4 years agoeldbus: fix the data size issue.
Hermet Park [Mon, 8 Jun 2020 02:00:26 +0000 (11:00 +0900)]
eldbus: fix the data size issue.

Summary:
dbus message requires the void* value for the message data,
the returned value of dbus_message_iter_get_basic() can be mismatched by to the architure,
This could bring the returned va_argues memory corrupted among the series of values.

We can use the defined type for exceptions.

Reviewers: herb, kimcinoo, jsuya, raster

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

4 years agoevas gl - tune gl engine to use a bit less cpu overhead 57/235657/2
Carsten Haitzler (Rasterman) [Tue, 2 Jun 2020 10:39:06 +0000 (11:39 +0100)]
evas gl - tune gl engine to use a bit less cpu overhead

we have used a fair bit of cpu to avoid gpu overhead and this tunes
the balance back a bit to throwing more at the gpu and less at the cpu
by reducing the number of pipes and max vertex counts per pipe by
default and only resetting pipe state vars if adding the first pipe
member.

@opt

Change-Id: Id102255c532ec7d116ca9a4d45f18ad69cde3829

4 years agoui scroller: fix broken scroller loop 08/235608/2
Hermet Park [Mon, 8 Jun 2020 05:00:04 +0000 (14:00 +0900)]
ui scroller: fix broken scroller loop

image is streched since proxy fill/region size is mismatched

this is a regression bug introduced by 76b89b8e49f4b16ad490c04134a9bdddea27950f

Change-Id: I1eb4de61e6a5e080a2f350de30cb76d1db575429

4 years agoci: drop lua from default binding set as elua is off by default accepted/tizen/unified/20200608.214808 submit/tizen/20200608.010949
Daniel Kolesa [Sat, 6 Jun 2020 21:29:23 +0000 (23:29 +0200)]
ci: drop lua from default binding set as elua is off by default

4 years agobuild: disable elua by default, plus nicer detection
Daniel Kolesa [Sat, 6 Jun 2020 17:09:21 +0000 (19:09 +0200)]
build: disable elua by default, plus nicer detection

Elua is now disabled by default. There are some other changes:

1) Elua scripts are only installed if Elua is enabled
2) Lua bindings are only installed if Elua is enabled
3) Elua with interpreter is clearly experimental and will message

4 years agoemotion: remove all left overs from removed backends
Stefan Schmidt [Fri, 24 Apr 2020 07:35:31 +0000 (09:35 +0200)]
emotion: remove all left overs from removed backends

We removed them in the last release. Make sure we cleanup all the
references and dead code still belonging to it.

Extra note on the Emotion_Module enum removal. While it was exposed in
the public Emotion.h header it is never used anywhere in the code and
has no purpose, thus it can safely be removed.

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

4 years agobuild: do not publish surface .eo files
Marcel Hollerbach [Thu, 4 Jun 2020 08:23:29 +0000 (10:23 +0200)]
build: do not publish surface .eo files

they are privat, they should never be installed.

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

4 years agoevas: correctly declare filter_internal as public API
Marcel Hollerbach [Thu, 4 Jun 2020 08:03:44 +0000 (10:03 +0200)]
evas: correctly declare filter_internal as public API

I know this is internal. However it is public used in public .eo files.
Even if it is internal, and beta, we still need to expose the API in
order to have it correctly in the .so.

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

4 years agoedje: markup flag should be TRUE in legacy edje_object_part_text_set
Bowon Ryu [Fri, 5 Jun 2020 05:26:22 +0000 (14:26 +0900)]
edje: markup flag should be TRUE in legacy edje_object_part_text_set

Summary:
Through some APIs (elm_entry_editable_set, elm_entry_single_line_set, etc.)
located after edje_object_part_text_set(legacy)

If efl_ui_widget_theme_apply is called,
In edje_object_part_text_raw_generic_set, the legacy flag becomes FALSE.
And in this case, the logic works in the unintended direction
because the set_markup flag is FALSE.

Test Plan:
/*
gcc -o entry_example entry.c `pkg-config --cflags --libs elementary`
 */
#include <Elementary.h>

EAPI_MAIN int
elm_main(int argc EINA_UNUSED, char **argv EINA_UNUSED)
{
   Evas_Object *win, *en;

   elm_policy_set(ELM_POLICY_QUIT, ELM_POLICY_QUIT_LAST_WINDOW_CLOSED);

   win = elm_win_util_standard_add("entry-example", "test");
   elm_win_autodel_set(win, EINA_TRUE);
   en = elm_entry_add(win);
   evas_object_size_hint_weight_set(en, EVAS_HINT_EXPAND, EVAS_HINT_EXPAND);
   elm_object_part_text_set(en, "elm.guide", "<font_size=32 color=#FFF>TEST</font_size>");
   elm_entry_editable_set(en, EINA_FALSE);

   evas_object_show(en);

   elm_object_content_set(win, en);
   evas_object_resize(win, 300, 200);
   evas_object_show(win);

   elm_run();

   return 0;
}
ELM_MAIN()

Reviewers: woohyun, ali.alzyod

Subscribers: zmike, cedric, #reviewers, #committers

Tags: #efl

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

4 years agoevas - genetic update regions smart merge - merge v adjacent regions
Carsten Haitzler (Rasterman) [Thu, 4 Jun 2020 19:25:46 +0000 (20:25 +0100)]
evas - genetic update regions smart merge - merge v adjacent regions

regions sometimes produce box sets like:
// +---+
// |   |
// +---+    +-------+
// |   |    |       |
// +---+    +-------+---------+
//          |                 |
//          |                 |
//          +-----------------+

so the upper-left 2 boxes can be merged into 1 and they have the same
x coords and are flush-aligned one above the other. that is what
this does - find these and merge them to have fewer rects

@fix

4 years agoevas render - add debug to draw boxes around update regions
Carsten Haitzler (Rasterman) [Thu, 4 Jun 2020 19:23:13 +0000 (20:23 +0100)]
evas render - add debug to draw boxes around update regions

helps debug update regions to see if there is too much overdraw or too
many regions. i smelled too manhy and this showed it. i was right. fix
for too many regions coming next

4 years agotiler - add maxreg for newtiler - not used atm but matches current
Carsten Haitzler (Rasterman) [Thu, 4 Jun 2020 19:22:03 +0000 (20:22 +0100)]
tiler - add maxreg for newtiler - not used atm but matches current

4 years agoatspi: return proxy object 52/233952/3
Bartlomiej Grzelewski [Wed, 20 May 2020 09:12:38 +0000 (11:12 +0200)]
atspi: return proxy object

Proxy objects were never returned by function because
they do not have x,y,w,h coordinates.

Change-Id: I237e63b6da3bd345f74e92c9e135bb9ff8d17599

4 years agoelm_poup_eo.legacy.h: modify Elm_Popup_Group to Elm_Popup 30/235530/1
Taehyub Kim [Fri, 5 Jun 2020 08:14:13 +0000 (17:14 +0900)]
elm_poup_eo.legacy.h: modify Elm_Popup_Group to Elm_Popup

Change-Id: I4a49df80c7a92ba388683cb7f96ecbb6666fdf24

4 years agoneed to upgrade meson. it looks like older ones have trouble
Carsten Haitzler (Rasterman) [Thu, 4 Jun 2020 11:00:52 +0000 (12:00 +0100)]
need to upgrade meson. it looks like older ones have trouble

like adding endless: -lm -ldl -lm -ldl ....

4 years agoelm test - animated icon - big win to have a bg and not garbage
Carsten Haitzler (Rasterman) [Tue, 2 Jun 2020 10:37:37 +0000 (11:37 +0100)]
elm test - animated icon - big win to have a bg and not garbage

4 years agobuild: move static out of none_static
Marcel Hollerbach [Thu, 4 Jun 2020 01:29:44 +0000 (10:29 +0900)]
build: move static out of none_static

Summary: these are static libs, they shall not be in the static list.

Reviewers: stefan_schmidt, jsuya

Reviewed By: stefan_schmidt, jsuya

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

4 years agoEfl.Canvas.Vg.Object: Fix backup size
JunsuChoi [Thu, 4 Jun 2020 01:28:21 +0000 (10:28 +0900)]
Efl.Canvas.Vg.Object: Fix backup size

Summary:
Make the size to be backed up and the size to be compared equal.
and minor fix

Test Plan: N/A

Reviewers: Hermet, herb, kimcinoo

Reviewed By: Hermet

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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