Mike Blumenkrantz [Sat, 6 Apr 2019 00:16:44 +0000 (20:16 -0400)]
add EFL_VERSION_1_22 defines
whoops
Mike Blumenkrantz [Sat, 6 Apr 2019 00:13:54 +0000 (20:13 -0400)]
back to dev mode
Mike Blumenkrantz [Fri, 5 Apr 2019 23:12:33 +0000 (19:12 -0400)]
release: Update NEWS and bump version for 1.22.0 release
Vitor Sousa [Fri, 5 Apr 2019 22:59:34 +0000 (19:59 -0300)]
efl-csharp: fix crash when events trigger after C# object `Dispose`
Summary:
Rework general event handling to check individually each event call, if the
object is not alive then the event will not be propagated.
WeakReferences (and lambdas capturing those WeakRefs) are used to ensure this.
Dispose methods in object now take care of checking if efl libraries are still
initialized and thread-safely unregister each event before performing an
efl_unref on the Eo object.
Event handling in C# is now centered around a single dictionary inside the
object: `EoEvents`.
C# event triggers now properly trigger events on C too.
Standardize C# event-triggering methods names (remove underscores).
Some diminished use of static memory due events no longer requiring static key
objects to be registered/unregistered.
Some fixing of white space generation for generated events.
Depends on D8431
Reviewers: lauromoura, felipealmeida, segfaultxavi
Reviewed By: lauromoura
Subscribers: cedric, #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D8564
Vitor Sousa [Fri, 5 Apr 2019 22:57:29 +0000 (19:57 -0300)]
efl-csharp: fix resource deallocation causing errors everywhere
Summary:
This commit mainly fixes errors caused by deallocating resources in the garbage
collector thread. Using `ecore_main_loop_thread_safe_call_async` to queue
resource deallocation in the main thread seems to solve it.
Also, some `efl_ref` calls are added in places they were missing, mainly
objects that unref in the destructor thus taking ownership if efl_ref is not
called.
Also fix improper resource deallocation in tests that were causing it to crash,
enabling it to call Efl.All.Shutdown again. This allocation and the deallocation
process was moved from the Eo class constructor to static class methods that are
called in the test 'set up' and 'tear down' methods.
Queuing resource deallocation in the main thread make it mandatory that tests
call `Efl.App.AppMain.Iterate()` if they want to check proper resource
deallocation (like TestFunctionPointers.set_callback_inherited_called_from_c).
Extras:
Remove duplicated declaration of 'eflcustomexportsmono' in meson in order to fix
some linking problems.
Remove some unused code around deallocation functions that had to be reworked.
Object allocation is now supplied with the call site information it expects
(file name and line for _efl_add_start).
Depends on D8550
Test Plan: meson test
Reviewers: felipealmeida, lauromoura, cedric, segfaultxavi
Reviewed By: lauromoura
Subscribers: segfaultxavi
Tags: #efl_language_bindings, #do_not_merge
Differential Revision: https://phab.enlightenment.org/D8431
Lauro Moura [Fri, 5 Apr 2019 22:53:37 +0000 (19:53 -0300)]
csharp: Make classes abstract and rework casting
Summary:
Abstract Eo classes are now proper C# abstract classes.
As a side effect, returning Eo instances from native code was reworked
to return instances of their actual Eo classes instead of previous
behavior of returning a generic Efl.Object and using static_cast.
Instead of `var window = Efl.Ui.Win.static_cast(widget.GetParent());`
Use `var window = widget.GetParent() as Efl.Ui.Win;`
Another side effect was that `efl_constructor` was removed from the list
of supported `Efl.Object` overrides. It is invoked inside
`efl_add_internal_start`, before the bindings makes the association of
the newly created EoId with the C# instance that created it, making the
managed delegate meaningless. C# users then can use regular C#
constructors to initialize fields.
Also changed to set the private data of C#-inherited classes before the
call to constructing methods (aka constructor parameters) so C# classes
can override them correctly.
Fixes T7778
Fixes T7757
Reviewers: vitor.sousa, felipealmeida, segfaultxavi
Reviewed By: vitor.sousa, segfaultxavi
Subscribers: cedric, #reviewers, #committers
Tags: #efl
Maniphest Tasks: T7778, T7757, T7702
Differential Revision: https://phab.enlightenment.org/D8550
Marcel Hollerbach [Fri, 5 Apr 2019 12:15:39 +0000 (08:15 -0400)]
meson: correctly use the correct dependency
Summary:
edje_cc calls epp, so we should not only add edje_cc to the depends on
target, but rather also ensure that epp is availble. Additionally, this
removes unneccessary depends on declarations when we do cross compile.
Depends on D8561
Reviewers: zmike, segfaultxavi, cedric
Reviewed By: zmike
Subscribers: #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D8562
Xavi Artigas [Thu, 4 Apr 2019 17:06:05 +0000 (19:06 +0200)]
docs: Exclude *.NativeStruct from DocFX pages
These are for internal use only.
Also, make filtering rules a bit more robust.
Vincent Torri [Thu, 4 Apr 2019 14:39:05 +0000 (10:39 -0400)]
dist rule: add files to EXTRA_DIST2 inconditionally
Summary: ecore_con and ethumb were adding files to EXTRE_DIST confitionally
Reviewers: cedric, zmike, raster
Reviewed By: zmike
Subscribers: #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D8558
Wonki Kim [Thu, 4 Apr 2019 13:14:08 +0000 (09:14 -0400)]
efreetd: remove a tentative global variable definition
Summary:
there is a global variable that is defined tentatively.
this patch modify it not to be tentitive explictly.
Reviewers: raster, cedric, zmike
Reviewed By: raster, zmike
Subscribers: #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D8551
Mike Blumenkrantz [Wed, 3 Apr 2019 19:53:24 +0000 (15:53 -0400)]
autotools: add another entry test file to dist
Reviewers: vtorri
Reviewed By: vtorri
Subscribers: cedric, #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D8549
Mike Blumenkrantz [Wed, 3 Apr 2019 18:19:04 +0000 (14:19 -0400)]
autotools: add entry test file to dist
Reviewers: bu5hm4n
Reviewed By: bu5hm4n
Subscribers: cedric, #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D8547
Christopher Michael [Wed, 3 Apr 2019 17:08:59 +0000 (13:08 -0400)]
efl_ui_test_callback: Removed defined but unused function
Hosang Kim [Wed, 3 Apr 2019 13:26:50 +0000 (09:26 -0400)]
evas_callbacks: fix emission of EFL_GFX_ENTITY_EVENT_VISIBILITY_CHANGED
Summary:
When I add "efl_event_callback_add(btn, EFL_GFX_ENTITY_EVENT_VISIBILITY_CHANGED, _cb, NULL)",
_cb is not called. Because of callback_mask is not set correctly.
Test Plan: unit test
Reviewers: zmike, cedric
Subscribers: #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D8528
Marcel Hollerbach [Tue, 2 Apr 2019 18:50:43 +0000 (14:50 -0400)]
elm_fileselector: disable this test for now
Summary:
this test fails on travis, for the sake of the release we continue
without this tests, after the release we can enable this again with or
without failing tests.
Reviewers: zmike
Reviewed By: zmike
Subscribers: cedric, #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D8538
Carsten Haitzler (Rasterman) [Tue, 2 Apr 2019 17:24:04 +0000 (18:24 +0100)]
evas image - fix proxy change propagation to fix e video wallpapers
this fixes propagati[on of changes from img sources to proxies which
broke e's video wallpapers. fixes T7685
Woochanlee [Tue, 2 Apr 2019 16:39:23 +0000 (18:39 +0200)]
C# bindings: Make efl_gesture_Manager.eo available to bindings
Summary:
It was missing.
The efl_gesture_manager has to using in efl_sharp.
Reviewers: zmike, Jaehyun_Cho
Reviewed By: Jaehyun_Cho
Subscribers: segfaultxavi, cedric, #reviewers, #committers
Tags: #efl
Maniphest Tasks: T7550
Differential Revision: https://phab.enlightenment.org/D7249
Jaehyun Cho [Tue, 2 Apr 2019 14:51:05 +0000 (16:51 +0200)]
mono: fix space and brace warnings of StyleCop
Summary:
The following warning rules of StyleCop are checked.
Space rules : SA1000, SA1003, SA1008, SA1009, SA1010, SA1011
Brace rules : SA1500, SA1501, SA1502, SA1503, SA1513
Indentation is also applied.
Reviewers: lauromoura, felipealmeida, vitor.sousa, woohyun
Reviewed By: vitor.sousa
Subscribers: cedric, #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D8485
Lauro Moura [Tue, 2 Apr 2019 14:47:36 +0000 (16:47 +0200)]
csharp: Raise exception when Array is null.
Reviewers: felipealmeida, vitor.sousa, segfaultxavi
Reviewed By: vitor.sousa
Subscribers: cedric, #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D8499
Vincent Torri [Tue, 2 Apr 2019 14:09:24 +0000 (10:09 -0400)]
remove useless check of ws2tcpip.h
Summary: it is useless to check for header files which necessarly exist
Test Plan: compilation
Reviewers: raster, cedric, zmike
Reviewed By: zmike
Subscribers: #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D8535
Carsten Haitzler (Rasterman) [Tue, 2 Apr 2019 12:57:58 +0000 (13:57 +0100)]
elm naviframe - remove duplicated func prototype in headers
one hade EAPI, one did not and there were 2 anyway, so remove one and
ensure the other has EAPI and the same docs. this should fix T7776
Xavi Artigas [Tue, 2 Apr 2019 12:57:35 +0000 (08:57 -0400)]
docs: Update refs to Efl.Ui.Theme.Apply_Error
Summary:
Proper refs to particular errors cannot be added until T7736 is resolved.
Ref T7717
Test Plan: Only docs updated
Reviewers: zmike
Reviewed By: zmike
Subscribers: cedric, #reviewers, #committers
Tags: #efl
Maniphest Tasks: T7717
Differential Revision: https://phab.enlightenment.org/D8495
Xavi Artigas [Tue, 2 Apr 2019 12:57:19 +0000 (08:57 -0400)]
docs: Format efl_file.h
Summary:
And polish a bit.
Ref T7717
Test Plan: Only docs updated
Reviewers: zmike
Reviewed By: zmike
Subscribers: bu5hm4n, cedric, #reviewers, #committers
Tags: #efl
Maniphest Tasks: T7717
Differential Revision: https://phab.enlightenment.org/D8498
Marcel Hollerbach [Tue, 2 Apr 2019 12:56:28 +0000 (08:56 -0400)]
efl_check: add API to expect a error
Summary:
sometimes it is not enough to just disable aborting on critical error
messages. Sometimes it is better to explicitly expect an error, and fail
the testcase if there is no error.
This is used in later commits here.
Depends on D8417
Reviewers: YOhoho, segfaultxavi, zmike, woohyun, Jaehyun_Cho
Reviewed By: zmike
Subscribers: cedric, #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D8519
Vincent Torri [Tue, 2 Apr 2019 12:52:17 +0000 (08:52 -0400)]
configure.ac : remove definition of HAVE_ECORE_AUDIO_WASAPI as it is never used
Summary: HAVE_WIN32 is used instead of HAVE_ECORE_AUDIO_WASAPI
Test Plan: compilation
Reviewers: raster, zmike, cedric
Reviewed By: zmike
Subscribers: #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D8532
Vincent Torri [Tue, 2 Apr 2019 12:52:11 +0000 (08:52 -0400)]
elm: allow config_embed to be run out of tree
Test Plan: compilation
Reviewers: raster, cedric, zmike
Reviewed By: zmike
Subscribers: #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D8533
Marcel Hollerbach [Tue, 2 Apr 2019 12:50:54 +0000 (08:50 -0400)]
ecore: correctly clean up in ecore_fork_reset
Summary:
after a fork does happen, the new process does not have any self created
threads at all. However, _thread_cb can contain suspend calls of
ecore_thread_main_loop_begin. _ecore_main_call_flush will then wait in
the suspend block for the thread to call ecore_thread_main_loop_end.
However, the thread is dead, the end function will never be called.
Hence we should ensure that we definitly kill every entry in _thread_cb
that has a susped flag on true.
This fixes deadlocks while running the testsuites with
EIO_MONITOR_POLL=1
Depends on D8526
Reviewers: cedric, segfaultxavi, zmike
Reviewed By: zmike
Subscribers: #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D8531
Marcel Hollerbach [Tue, 2 Apr 2019 12:50:48 +0000 (08:50 -0400)]
eio: ensure that the monitor starts when it is called to
Summary:
we need to to this here, otherwise we effectfily start the monitor
later, which means, we can miss a change in the filesystem. However,
this makes things a lot slower.
Reviewers: cedric
Reviewed By: cedric
Subscribers: cedric, #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D8526
Marcel Hollerbach [Tue, 2 Apr 2019 12:50:34 +0000 (08:50 -0400)]
efl_io_model: start early on monitoring
Summary:
Little introduction into what eio did before this commit:
Efl.Io.Model creation:
- direct ls of a directory in a thread (A)
- take all the contents of a directory and feed it slowly back into
the mainloop
- when all events have been feeded back to the mainloop and have been
processed: start monitoring (B)
However, any file created between (A) and (B) will not be in the model,
since not the listing nor the monitoring did caputure it. Hence we need
to start monitoring before we actaully start listing. In the callbacks
we then check if we already published something.
ref T7311
Reviewers: zmike, cedric
Reviewed By: cedric
Subscribers: cedric, #reviewers, #committers
Tags: #efl
Maniphest Tasks: T7311
Differential Revision: https://phab.enlightenment.org/D8525
Mike Blumenkrantz [Tue, 2 Apr 2019 12:34:14 +0000 (08:34 -0400)]
tests: unset eina log callback at end of each efl_app promise test
Summary: ensure that this doesn't trigger cascading errors
Reviewers: cedric
Reviewed By: cedric
Subscribers: cedric, #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D8514
Xavi Artigas [Tue, 2 Apr 2019 11:28:48 +0000 (13:28 +0200)]
docs: Fix common misspellings in H files
Fixed all appearances of words from this list in H files:
https://en.wikipedia.org/wiki/Wikipedia:Lists_of_common_misspellings/For_machines
Xavi Artigas [Tue, 2 Apr 2019 09:32:05 +0000 (11:32 +0200)]
docs: Fix common misspellings in EO files
Fixed all appearances of words from this list in EO files:
https://en.wikipedia.org/wiki/Wikipedia:Lists_of_common_misspellings/For_machines
Lauro Moura [Mon, 1 Apr 2019 22:06:20 +0000 (22:06 +0000)]
eo: Correctly name Realized classes.
The previous preprocessor rule was generating strings like
"\"Efl.App\"_Realized.
Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D8527
Jaehyun Cho [Tue, 2 Apr 2019 07:39:40 +0000 (16:39 +0900)]
Revert "efl_ui_pager: implement unpack_at function"
This reverts commit
913a5bff66dd7a58753ccae1474657ec33350e43.
The commit is reverted because the commit implements features during
feature freeze period before release.
913a5bff66dd7a58753ccae1474657ec33350e43 will be restored after release.
Jaehyun Cho [Tue, 2 Apr 2019 07:36:20 +0000 (16:36 +0900)]
Revert "efl_ui_pager: implement unpack_all/clear function"
This reverts commit
74c48f029632d2260816b4cb3de6dbb40a29f040.
The commit is reverted because the commit implements features during
feature freeze period before release.
74c48f029632d2260816b4cb3de6dbb40a29f040 will be restored after release.
Jaehyun Cho [Tue, 2 Apr 2019 07:35:12 +0000 (16:35 +0900)]
Revert "test/efl_ui_pager_scroll: fix demo"
This reverts commit
30bd541bf264d369e2a96d27fa3ff5139c360e26.
The commit is reverted to revert the following commits.
74c48f029632d2260816b4cb3de6dbb40a29f040
913a5bff66dd7a58753ccae1474657ec33350e43
The above commits will be restored after release and then
30bd541bf264d369e2a96d27fa3ff5139c360e26 will be also restored.
Jaehyun Cho [Tue, 2 Apr 2019 07:29:11 +0000 (16:29 +0900)]
Revert "test/efl_ui_pager: fix demo"
This reverts commit
d3bb1a7342b2725c585d90557926e0c433058e50.
The commit is reverted to revert the following commits.
74c48f029632d2260816b4cb3de6dbb40a29f040
913a5bff66dd7a58753ccae1474657ec33350e43
The above commits will be restored after release and then
d3bb1a7342b2725c585d90557926e0c433058e50 will be also restored.
Jaehyun Cho [Tue, 2 Apr 2019 07:27:35 +0000 (16:27 +0900)]
Revert "elm - fix harmless warning for clean build"
This reverts commit
e3d2a0cf1235beceb3b403e42d2dafed4b4e90e5.
The commit is reverted to revert the following commit.
74c48f029632d2260816b4cb3de6dbb40a29f040
The above commit will be restored after release and then
e3d2a0cf1235beceb3b403e42d2dafed4b4e90e5 will be also restored.
Hosang Kim [Fri, 5 Apr 2019 08:43:59 +0000 (17:43 +0900)]
ecore_wl2_window: check if callback exists before adding it.
Change-Id: I86f41a3a04c0fdde1a73e4469264530e63ee02f0
Wonki Kim [Wed, 3 Apr 2019 07:11:20 +0000 (16:11 +0900)]
meson: Add missed public headers
those header files are necessary files to be installed.
so that this patch added them
Change-Id: I833768e6dd565bb33e5084fda2db9a4c98b1dcd3
Wonki Kim [Wed, 3 Apr 2019 06:12:17 +0000 (15:12 +0900)]
spec: fix a build break
edje_cc uses embryo_cc internally which has been removed before.
so that this patch fixes it.
Change-Id: Ie55bd74ab925167209b73da75cae5ab9f2b8a702
Wonki Kim [Wed, 3 Apr 2019 05:23:11 +0000 (14:23 +0900)]
spec: fix build break by broken dependency
edje-devel had a depedency to edje-tools, but it has been removed.
so that this patch restore it to fix build break
Change-Id: I3bdfca02608739252864c50c68fb13dd062a384c
Hermet Park [Wed, 3 Apr 2019 05:20:40 +0000 (14:20 +0900)]
efl_ui_animation: don't access a null pointer.
Change-Id: Ica448b5b7dd97cfaa5ef5644f692caacf48aa986
Bowon Ryu [Tue, 2 Apr 2019 12:40:22 +0000 (21:40 +0900)]
elm_label: fix build break for legacy api.
this commit related with
b8d3be5aa02b846d42961f74490e02448ef30d1d
@tizen_fix
Change-Id: If2154fe566384ae8dac6c8662d1466700c55f6d9
Signed-off-by: Bowon Ryu <bowon.ryu@samsung.com>
JunsuChoi [Wed, 3 Apr 2019 04:35:01 +0000 (13:35 +0900)]
elm_entry: prevent from accessing null pointer
Summary: change ELM_ENTRY_DATA_GET to ELM_ENTRY_DATA_GET_OR_RETURN_VAL
Test Plan: elm_entry_file_set(NULL, ~~~);
Reviewers: YOhoho, Hermet
Subscribers: cedric, #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D8544
Change-Id: I728a485db981b8c54a03476d1daee1e0c8c246bb
Hermet Park [Wed, 27 Mar 2019 07:24:54 +0000 (16:24 +0900)]
vg json: code refactoring.
just rename internal variable.
Change-Id: I0771455090ad9cb52d657585b03be696f389fd14
Hermet Park [Wed, 27 Mar 2019 07:15:24 +0000 (16:15 +0900)]
vg json: set mask parent to current tree node, not the root.
+ clean up trivial code.
Change-Id: I87d60bbcce1c68cd4e9d85199ab10b051ded2fef
Hermet Park [Tue, 2 Apr 2019 08:28:03 +0000 (17:28 +0900)]
canvas vg: code refactoring.
Remove unnecessary argument of an internal function.
Change-Id: Ib3c216997ba2c2fdaaea6d05738fe4d9f09342aa
Hermet Park [Tue, 2 Apr 2019 06:17:24 +0000 (15:17 +0900)]
canvas vg: fix wrong key pass for caching surface.
There is a wrong case that fails caching ector surfaces.
Those vector objects manually constructing shapes,
passed wrong pointer as the cacahing key instead of root node pointer.
This fixes it.
Change-Id: I5f547e896d1d380b2b7fbad00587609f54ce6ec2
Wonki Kim [Wed, 6 Mar 2019 04:45:38 +0000 (13:45 +0900)]
spec: fix a dependencies for devel package properly
package descriptions for edje-devel, elementary-devel was not described properly
they need libraries itself. so that this patch fix it.
Change-Id: I8daa2490b91546ca1083a2adfa69e0e7a02a55e6
Signed-off-by: Wonki Kim <wonki_.kim@samsung.com>
Wonki Kim [Mon, 1 Apr 2019 07:56:05 +0000 (16:56 +0900)]
spec: exclude unused binaries.
this pathc excludes some binaries from the packiging list
Change-Id: Ic967ad3846ce60d0c53b3c85296b0261dbe71be3
Signed-off-by: Wonki Kim <wonki_.kim@samsung.com>
Wonki Kim [Tue, 2 Apr 2019 07:50:21 +0000 (16:50 +0900)]
meson: fix a meson option to install eo files
as before, eo files are installed as default,
however, after some commit, it doesn't for now.
this patch fixes a meson option to install those files.
Change-Id: I1060ba5476533f479a18b86df020a7e051792cdb
JunsuChoi [Tue, 2 Apr 2019 07:39:56 +0000 (16:39 +0900)]
migratino: remove beta tags
@tizen_fix
Change-Id: If2b152273cc9d3fa31a8eac59810e4db7f7cdad5
Jaehyun Cho [Tue, 2 Apr 2019 06:33:04 +0000 (15:33 +0900)]
elm_widget_item_eo.legacy: change @ingroup to Elm_General
The @ingroup of elm_object_item APIs is Elm_General in elm_object_item.h.
Since legacy prefix of Elm_Widget_Item is elm_object_item,
the prefix of Elm_Widget_Item APIs is elm_object_item.
Consequently, to display the legacy Elm_Widget_Item APIs with
elm_object_item APIs, the @ingroup is changed from Elm_Object_Item_Group
to Elm_General.
Change-Id: Ic023a57f15ee3968fb0a5cfa09d9ab33c4adf420
Christopher Michael [Mon, 1 Apr 2019 14:31:08 +0000 (10:31 -0400)]
ecore-drm2: Add missing @ingroup for some doxy
Small patch to add missing @ingroup for doxygen comments, and correct
one that was in the wrong group
@fix
Christopher Michael [Mon, 1 Apr 2019 14:02:57 +0000 (10:02 -0400)]
efl_ui_selection_manager: Don't leak malloc'd data
Summary:
Coverity reports that we potentially leak char *s here. If we do not
have 'data_ret', then the malloc'd 's' sould be freed as we are not
going to use it.
Fixes Coverity CID1396949
@fix
Reviewers: raster, cedric, bu5hm4n, zmike
Reviewed By: bu5hm4n
Subscribers: #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D8523
Christopher Michael [Mon, 1 Apr 2019 13:05:57 +0000 (09:05 -0400)]
evas-font-dir: Minor formatting fixes
NB: No functional changes
Carsten Haitzler (Rasterman) [Sat, 30 Mar 2019 16:54:19 +0000 (16:54 +0000)]
elm - fix harmless warning for clean build
Carsten Haitzler (Rasterman) [Sat, 30 Mar 2019 16:49:48 +0000 (16:49 +0000)]
elm - fix harmless warning for clean build
Carsten Haitzler (Rasterman) [Sat, 30 Mar 2019 16:49:21 +0000 (16:49 +0000)]
elm - fix harmless warning for clean build
Yeongjong Lee [Sat, 30 Mar 2019 08:49:57 +0000 (08:49 +0000)]
efl_ui_table_layout: calculate cell size with colspan, rowspan property
We should consider occupied cells by colspan, rowspan property.
ref T7753
Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D8484
Mike Blumenkrantz [Thu, 28 Mar 2019 15:25:21 +0000 (11:25 -0400)]
build: improve autotools generation of elm config
- don't generate and re-link on every make rule
- fix distcheck
Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Reviewed-by: YeongJong Lee <yj34.lee@samsung.com>
Differential Revision: https://phab.enlightenment.org/D8506
Boris Faure [Fri, 29 Mar 2019 19:21:55 +0000 (20:21 +0100)]
README.meson: fix typo + rewrap
Mike Blumenkrantz [Fri, 29 Mar 2019 17:58:14 +0000 (18:58 +0100)]
tests: abort on errors during genlist expand test, not warnings
Summary:
log level=2 is the warning level, which is not super useful since
there's currently billions of eo warnings occuring in every function
call
Reviewers: cedric, segfaultxavi
Reviewed By: cedric, segfaultxavi
Subscribers: segfaultxavi, cedric, #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D8515
Mike Blumenkrantz [Fri, 29 Mar 2019 17:50:30 +0000 (18:50 +0100)]
eina_log: reset logging callback to default when null is set as the callback
Summary:
passing null here causes any log message to crash the app and is probably not
the intended result
Reviewers: cedric, segfaultxavi
Reviewed By: cedric, segfaultxavi
Subscribers: cedric, #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D8513
Ali Alzyod [Fri, 29 Mar 2019 13:52:51 +0000 (09:52 -0400)]
elm_entry: Speedup finding new line, prevent readind invalid memory
Summary:
1- Speed up detecting new lines.
```
if (!strncmp(text, "<br", 3) || !strncmp(text, "<ps", 3))
```
This will cause 6 comparisons (if one of conditions did not meet), or at least 3 comparisons.
this is changed to
```
if (!strncmp(text, "<", 1))
```
2- Speedup detecting lines
If this condition is true, we should increment the string for next iteration 3 times, not just one
```
if (!strncmp(text, "<br", 3) || !strncmp(text, "<ps", 3))
```
if '<' founded then 'pr' or 'br', we will skip 3 characters for next iteration.
```
if (!strncmp(text, "<", 1))
{
text++;
len--;
if (!strncmp(text, "br", 2) || !strncmp(text, "ps", 2))
{
text += 2;
len -= 2;
```
3- Prevent reading invalid memory out of the string
```
if (text[3] == '>' || ((text[3] == '/') && (text[4] == '>')))
```
string could reach last char in string (original string ends with "<br")
but now we will check if remaining string length allow comparison :
```
if (text[0] == '>' || (len > 1 && ((text[0] == '/') && (text[1] == '>'))))
```
Test Plan:
```
static int
oldFunc(const char *text)
{
if (!text)
return 0;
while (*text)
{
if (!strncmp(text, "<br", 3) || !strncmp(text, "<ps", 3))
{
if (text[3] == '>' || ((text[3] == '/') && (text[4] == '>')))
{
return 1;
}
}
text++;
}
return 0;
}
static int
newFunc(const char *text)
{
if (!text)
return 0;
char *pTemp = (char *)text;
while (pTemp = strchr(pTemp, '<'))
{
pTemp++;
if (!strncmp(pTemp, "br", 2) || !strncmp(pTemp, "ps", 2))
{
pTemp += 2;
if (pTemp[0] != '\0' && (pTemp[0] == '>' || (pTemp[0] == '/' && pTemp[1] == '>')))
{
return 1;
}
}
}
return 0;
}
int main()
{
int counter = 1000;
srand(time(NULL));
char pStr[50001] = {0};
char AllChars[] = "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789<>";
int AllCharsLen = strlen(AllChars);
for (int i = 0; i < 50000; i++)
pStr[i] = AllChars[rand() % AllCharsLen];
clock_t start, end;
double total_Time1 = 0;
int i;
for (int j = 0; j < 3; j++)
{
if (j == 0)
{
printf("random String\n");
}
else if (j == 1)
{
printf("With Random <br/>\n");
int location = rand()%(5000 - 5);
pStr[location++] = '<';
pStr[location++] = 'b';
pStr[location++] = 'r';
pStr[location++] = '/';
pStr[location++] = '>';
}
else if (j == 2)
{
printf("With Random <ps>\n");
int location = rand()%(5000 - 4);
pStr[location++] = '<';
pStr[location++] = 'p';
pStr[location++] = 's';
pStr[location++] = '>';
}
start = clock();
for (i = 0; i < counter; i++)
oldFunc(pStr);
end = clock();
total_Time1 = ((double)(end - start)) / CLOCKS_PER_SEC;
printf("original = %f has new Line = %i\n", total_Time1, oldFunc(pStr));
start = clock();
for (i = 0; i < counter; i++)
newFunc(pStr);
end = clock();
total_Time1 = ((double)(end - start)) / CLOCKS_PER_SEC;
printf("modified = %f has new line = %i\n\n", total_Time1, newFunc(pStr));
}
}
```
output:
random String
original = 2.523000 has new Line = 0
modified = 0.090000 has new line = 0
With Random <br/>
original = 0.081000 has new Line = 1
modified = 0.003000 has new line = 1
With Random <ps>
original = 0.016000 has new Line = 1
modified = 0.001000 has new line = 1
Reviewers: zmike, woohyun, bowonryu
Reviewed By: zmike
Subscribers: bu5hm4n, segfaultxavi, zmike, cedric, #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D8497
Xavi Artigas [Fri, 29 Mar 2019 13:12:37 +0000 (14:12 +0100)]
docs: Fix build break because of missed EO import
Xavi Artigas [Fri, 29 Mar 2019 13:08:22 +0000 (14:08 +0100)]
docs: Use Eina.Value references en efl_ui_format docs
Marcel Hollerbach [Fri, 29 Mar 2019 12:25:54 +0000 (13:25 +0100)]
efl_ui_layout: ensure that resize_obj is present before emitting signals
otherwise there will be warnings because of calling api on NULL objects.
Differential Revision: https://phab.enlightenment.org/D8512
Marcel Hollerbach [Tue, 26 Mar 2019 09:15:15 +0000 (10:15 +0100)]
efl_ui_widget: move from elm_widget_top_get to provider_find
this resolves a lot of cases where focus_highlight API was called on a
object, which is not a efl_ui_win object. With this patch we ensure that
the object is always a window.
Reviewed-by: Xavi Artigas <xavierartigas@yahoo.es>
Differential Revision: https://phab.enlightenment.org/D8476
Marcel Hollerbach [Tue, 26 Mar 2019 08:59:16 +0000 (09:59 +0100)]
efl_ui_widget: add implementation for finding the window
the problem with the previous implementation (just redirect the calls to
the widget_parent then to the efl_parent is that after invalidate its
impossible to find the window where the widget is in. However, there are
cases where we want to have access to the window of the widget, for
example, to invalidate focus highlight etc..
The window of a widget is always constant, and cannot be changed (as the
evas object cannot hop accross different evas)
Reviewed-by: Jaehyun Cho <jae_hyun.cho@samsung.com>
Differential Revision: https://phab.enlightenment.org/D8475
Xavi Artigas [Thu, 28 Mar 2019 16:12:27 +0000 (16:12 +0000)]
docs: Efl.Ui.Layout_Base update theme docs
If docs are present at property and set/get levels only one is used.
Ref T7717
Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D8494
Xavi Artigas [Fri, 29 Mar 2019 09:30:44 +0000 (09:30 +0000)]
docs: Clarify Efl.Ui.Win exit_on_close methods
exit_on_close and exit_on_all_windows_closed deserve a bit of clarification
since they have very similar meanings.
Also, add proper Eina.Value doc references.
Ref T7717
Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D8511
Xavi Artigas [Fri, 29 Mar 2019 09:25:02 +0000 (09:25 +0000)]
docs: Add Eina.Value extern to eina_types.eot
Eina.Values are built-in eolian types, accessed through any_value and
any_value_ptr. However, these types cannot be used in doc references.
Adding a placeholder extern struct Eina.Value causes no harm, and will allow
referencing the type from EO docs later on.
In C#, Eina.Value is defined in the manual binding code so the doc reference
resolves to a valid type.
Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D8510
Mike Blumenkrantz [Thu, 28 Mar 2019 22:04:55 +0000 (18:04 -0400)]
ci: check correct test log for meson build
Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D8504
Mike Blumenkrantz [Thu, 28 Mar 2019 15:26:55 +0000 (11:26 -0400)]
ci: use meson test runner with ninja build and use dbus in tests
this fixes dbus usage in tests
Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D8503
Jaeun Choi [Fri, 29 Mar 2019 11:03:16 +0000 (20:03 +0900)]
test/efl_ui_pager: fix demo
- use unpack_at function than unpack function
- add CLEAR option
Jaeun Choi [Tue, 19 Feb 2019 09:42:31 +0000 (18:42 +0900)]
test/efl_ui_pager_scroll: fix demo
- use radio than check for loop mode
- use unpack_at function than unpack function
- add CLEAR option
Jaeun Choi [Fri, 29 Mar 2019 10:48:58 +0000 (19:48 +0900)]
efl_ui_pager: implement unpack_all/clear function
Jaeun Choi [Fri, 29 Mar 2019 10:23:20 +0000 (19:23 +0900)]
efl_ui_pager: return mouse_down function if cnt equals zero
Jaeun Choi [Thu, 28 Mar 2019 10:26:01 +0000 (19:26 +0900)]
efl_ui_pager: implement unpack_at function
Jaeun Choi [Thu, 28 Mar 2019 10:21:21 +0000 (19:21 +0900)]
efl_ui_pager: refactor unpack function
Jaeun Choi [Fri, 22 Mar 2019 08:51:17 +0000 (17:51 +0900)]
efl_ui_pager: add a missing condition
Jaeun Choi [Fri, 22 Mar 2019 02:30:34 +0000 (11:30 +0900)]
efl_ui_pager: fix pack function
Jaeun Choi [Tue, 19 Mar 2019 05:32:18 +0000 (14:32 +0900)]
elm_priv.h: remove unnecesary line
Jaeun Choi [Tue, 19 Feb 2019 09:44:26 +0000 (18:44 +0900)]
efl_ui_pager: disable loop if items are not enough after unpacking
Xavi Artigas [Fri, 29 Mar 2019 08:51:56 +0000 (09:51 +0100)]
mono-docs: Minor ammendments to Eina.Value
Marcel Hollerbach [Sun, 24 Mar 2019 17:58:50 +0000 (18:58 +0100)]
efl_ui_widget: reintroduce legacy behaviour
before the refactoring of the disabled property, there was no way to
enable a widget which has a disabled tree. This here however enables
this to work again like this. The user will be told with an error
message. The integraty of the property is maintained accross reparents.
Reviewed-by: Jaehyun Cho <jae_hyun.cho@samsung.com>
Differential Revision: https://phab.enlightenment.org/D8459
Marcel Hollerbach [Sun, 24 Mar 2019 17:22:15 +0000 (18:22 +0100)]
efl_ui_widget: add tests for parent and disalbed property
this just adds more coverage over the behaviour of efl_ui_widget
properties.
Reviewed-by: Jaehyun Cho <jae_hyun.cho@samsung.com>
Differential Revision: https://phab.enlightenment.org/D8458
Marcel Hollerbach [Sun, 24 Mar 2019 17:22:03 +0000 (18:22 +0100)]
efl_ui_test_widget: ensures tests do not error
Reviewed-by: Jaehyun Cho <jae_hyun.cho@samsung.com>
Differential Revision: https://phab.enlightenment.org/D8457
Carsten Haitzler (Rasterman) [Fri, 29 Mar 2019 00:16:22 +0000 (00:16 +0000)]
Revert "evas gl engines - avoid getting context if possible"
This reverts commit
e7771438a25382a2e4fabe33753456dcde8cab40.
This should fix T7764
zmike - you're right. thanks for narrowing down the commit... revert
time.
Vincent Torri [Thu, 28 Mar 2019 13:23:20 +0000 (09:23 -0400)]
remove the definition of HAVE_WASAPI as it is never used
Summary: HAVE_WASAPI is never used
Test Plan: compilation
Reviewers: zmike, cedric, raster
Reviewed By: zmike
Subscribers: #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D8493
Taehyub Kim [Thu, 28 Mar 2019 09:48:03 +0000 (18:48 +0900)]
efl_ui_alert_popup: add new feature for applying side button style of alert popup
Summary: This feature will apply side button style for each left and right button of alert popup
Reviewers: Jaehyun_Cho, cedric
Reviewed By: Jaehyun_Cho
Subscribers: #reviewers, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D8492
Mike Blumenkrantz [Wed, 27 Mar 2019 15:25:28 +0000 (11:25 -0400)]
elm_entry: CRI if efl_file methods are called directly on this object
eo methods should not be called on legacy objects
Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D8487
Xavi Artigas [Wed, 27 Mar 2019 15:10:57 +0000 (15:10 +0000)]
docs: Document event info calling convention
Both at the emitter (efl_event_callback_call) and the receiver
(info field in the Efl.Event structure).
The Events tutorial should repeat this.
Fixes T7760
Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D8486
Marcel Hollerbach [Sun, 24 Mar 2019 12:24:47 +0000 (13:24 +0100)]
efl_ui_slider: block scrolling when on slider
when a mouse cursor is over a slider, the mouse wheel should be used to
affect the state of the slider, not also the one of the slider.
ref T2529
Reviewed-by: Bowon Ryu <bowon.ryu@samsung.com>
Reviewed-by: Jaehyun Cho <jae_hyun.cho@samsung.com>
Differential Revision: https://phab.enlightenment.org/D8455
Cedric BAIL [Wed, 27 Mar 2019 22:19:32 +0000 (15:19 -0700)]
elementary: remove some asynchronous behavior from the fileselector.
Marcel notted that when using the LIST view of the fileselector on a
big directory, we end up having to wait for the entire genlist to be
populated to be able to switch to another directory. This is actually
a side effect of the populate code being triggered through an idler.
This idler was useful when the list was populated directly, but now
that we rely on Efl.Io.Model, we should be asynchronous enough that
it shouldn't be a problem to actually not be asynchronous here. By
removing the reliance on the idler, we are not queued after all the
idler and can properly short circuit all of that.
Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D8491
Cedric BAIL [Wed, 27 Mar 2019 21:24:57 +0000 (14:24 -0700)]
elementary: restore quick exit from wait loop in fileselector test.
The test was not expecting both callback to be set when the wait loop
was started. By moving them around, it fixes the test case to only have
one relevant callback set at a time.
Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D8490
Marcel Hollerbach [Wed, 27 Mar 2019 20:15:53 +0000 (21:15 +0100)]
elc_filselector: solve the mysterium of sometimes failing tests
so after a phone call, two days of debugging, tears, crying etc. etc. we
finally came to a point of enlightenment! *Someone* (bu5hm4n) moved
gengrid and genlist events from eo back to smart events, so we can work
arround legacy borks and event-name collisions, at this point he did not
knew that some widgets (fileselector) already relied on those *lovely*
events. Hence this broke theoretically the testsuite, however, the
fileselector testsuite is ultimatily buggy, and the wait function does
not return false when it timeouts, (i don't know why not). So this break
was never discovered.
Additionally there is a second issue. it appears, that when we
immidiatly quit the mainloop after we have got the selected callback,
that then genlist decides to forget about the sd->selected pointer, and
NULLs that one out. Which then results in the fact that
elm_fileselector_selected_get ends up returning invalid paths.
Reviewed-by: Cedric BAIL <cedric.bail@free.fr>
Differential Revision: https://phab.enlightenment.org/D8488
Cedric BAIL [Fri, 22 Mar 2019 18:20:51 +0000 (11:20 -0700)]
elementary: make sure that the model parent being used is always the fileselector.
Model provided by an item selection would have there parent being the current
model of the fileselector. Once that one is replaced by the item model, it would
automatically invalidate the model and break any further request. This lead to
a bug where you could only get into one directory before everything else
being empty.
Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D8450