summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* ci: travis: switch exactness build to Fedora32 imageStefan Schmidt2020-05-142-2/+2
|
* elm: refactor legacy _icon_signal_emitMike Blumenkrantz2020-05-136-86/+33
| | | | | | | | | | | | | | | | | Summary: this was duplicated for a number of widgets in slightly different ways, so it can be unified as a private layout function to reduce maintenance costs Reviewers: Hermet, YOhoho, Jaehyun_Cho, jsuya Reviewed By: jsuya Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D11818
* edje_calc: Add unload file for already loaded vector lottie file.JunsuChoi2020-05-131-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | Summary: Users can use multiple jsons per description in one part in edc. At that time, multiple files are loaded, if a file is already loaded, it must be unloaded. Test Plan: edje_cc -beta -id ./ json_edc.edc make ./test Push Change button -> shutdown application {F3882716} Reviewers: Hermet, herb, kimcinoo Reviewed By: Hermet Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D11820
* Revert "canvas render: don't apply mask if the objects in the map."Hermet Park2020-05-131-47/+26
| | | | | | This reverts commit eaafb9e4c53863cfd0ae4aaf24f04d333458ff2b. Found some regression bugs. revert it.
* Efl.Ui.Radio_Legacy: Prevent unnecessary signal emit for contentsJunsuChoi2020-05-121-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Summary: When call elm_radio_add, theme_apply of layout is called and _efl_ui_radio_legacy_efl_ui_widget_theme_apply is called. Then it calls the signal emit for the icon. this call is unnecessary. Test Plan: Evas_Object *bt; clock_t start, finish; double sum= 0 ; double avg = 0; double cnt = 1000; for(int i =0 ; i<(int)cnt; i++) { start = clock(); bt = elm_radio_add(win); finish = clock(); sum += (double)(finish-start)/CLOCKS_PER_SEC; } avg = sum / cnt ; printf("radio avg : %f\n",avg); [before] radio avg : 0.000232 [after] radio avg : 0.000197 Reviewers: akanad, Jaehyun_Cho, Hermet, YOhoho, zmike Reviewed By: zmike Subscribers: bu5hm4n, zmike, cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D11813
* Efl.Ui.Check_Legacy: Prevent unnecessary signal emit for contentsJunsuChoi2020-05-121-1/+1
| | | | | | | | | | | | | | | | | | | | Summary: When call elm_check_add, theme_apply of layout is called and _efl_ui_check_legacy_efl_ui_widget_theme_apply is called. Then it calls the signal emit for the icon. this call is unnecessary. Test Plan: N/A Reviewers: akanad, Jaehyun_Cho, Hermet, YOhoho, zmike Reviewed By: zmike Subscribers: zmike, cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D11812
* Efl.Ui.Progressbar_Legacy: Prevent unnecessary signal emit for contentsJunsuChoi2020-05-121-1/+1
| | | | | | | | | | | | | | | | | | | | Summary: When call elm_progressbar_add, theme_apply of layout is called and _efl_ui_progressbar_legacy_efl_ui_widget_theme_apply is called. Then it calls the signal emit for the icon. this call is unnecessary. Test Plan: N/A Reviewers: Hermet, bowonryu, Jaehyun_Cho, YOhoho, zmike Reviewed By: zmike Subscribers: zmike, cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D11811
* Efl.Ui.Button_Legacy: Prevent duplicate signal emit of contentsJunsuChoi2020-05-121-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Summary: When call elm_button_add, theme_apply of layout is called and _efl_ui_button_legacy_efl_ui_widget_theme_apply is called. Then it calls the signal emit for the icon. In the button theme, the default of the icon is hidden, so this call is unnecessary. Test Plan: Evas_Object *bt; clock_t start, finish; start = clock(); bt = elm_button_add(win); finish = clock(); printf("duration : %f\n", (double)(finish-start)/CLOCKS_PER_SEC); Evas_Object *bt; clock_t start, finish; double sum= 0 ; double avg = 0; double cnt = 1000; for(int i =0 ; i<(int)cnt; i++) { start = clock(); bt = elm_button_add(win); finish = clock(); sum += (double)(finish-start)/CLOCKS_PER_SEC; } avg = sum / cnt ; printf("button avg : %f %f\n",avg, sum); [before] button avg : 0.000215 [after] button avg : 0.000138 Reviewers: Hermet, YOhoho, Jaehyun_Cho, zmike Reviewed By: zmike Subscribers: zmike, cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D11810
* build: enable -Wpointer-arith for dev buildsMike Blumenkrantz2020-05-121-0/+1
| | | | | | | | | | | | Summary: this is a useful warning to avoid errors when calculating pointer offsets Depends on D11816 Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D11817
* build: fix void* use in pointer mathsMike Blumenkrantz2020-05-122-7/+7
| | | | | | | | | | | | | Summary: void* is an invalid type for calculating pointer offsets, so ensure that this is always cast to something else (e.g., char*) in the few cases where it's been misused Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D11816
* build: enable -Wfloat-compare by default for dev buildsMike Blumenkrantz2020-05-121-0/+1
| | | | | | | | | | Summary: Depends on D11799 Subscribers: segfaultxavi, cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D11800
* static/triangulator: resolve float compare warningsMike Blumenkrantz2020-05-121-2/+2
| | | | | | | | | | Summary: Depends on D11798 Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D11799
* edje: resolve float compare warningsMike Blumenkrantz2020-05-121-2/+2
| | | | | | | | | | Summary: Depends on D11797 Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D11798
* exactness: resolve float compare warningsMike Blumenkrantz2020-05-121-1/+1
| | | | | | | | | | Summary: Depends on D11796 Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D11797
* examples: resolve float compare warningsMike Blumenkrantz2020-05-1210-21/+21
| | | | | | | | | | Summary: Depends on D11795 Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D11796
* evas/gl: resolve float compare warningsMike Blumenkrantz2020-05-123-14/+13
| | | | | | | | | | Summary: Depends on D11794 Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D11795
* evas/filter: resolve float compare warningsMike Blumenkrantz2020-05-123-14/+17
| | | | | | | | | | Summary: Depends on D11793 Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D11794
* evas: resolve float compare warningsMike Blumenkrantz2020-05-127-30/+34
| | | | | | | | | | Summary: Depends on D11792 Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D11793
* Efl.Canvas.Object.Animation: Prevent crash when called .animation_pause_set.JunsuChoi2020-05-121-1/+1
| | | | | | | | | | | | | | | | | | | | Summary: When call efl_canvas_object_animation_animation_pause_set, pd->in is freed by calling efl_canvas_object_animation_stop in _animator_cb. So when set pause_state, crash occurred This patch add null check to prevent null access. Test Plan: N/A Reviewers: Hermet, kimcinoo, herb, bu5hm4n Reviewed By: Hermet, bu5hm4n Subscribers: bu5hm4n, cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D11815
* evas_object_textblock: reduce layout calculationsAli Alzyod2020-05-121-9/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Summary: This patch reduces calculations for layouting textblock when it is not needed. Exactly in **evas_object_textblock_render_pre**, layouting was done (if needed) regardless of object visibility. evas_object_render_pre will async called if object status is changed, for example show->hide In short words: **We do not layout textblock content if textblock t is hidden.** ``` // suppose textblock is shown // and user want to hide it and set content in it // to be visible later on if needed evas_object_hide(textblock); evas_object_textblock_text_markup_set(textblock, "Hello World"); //Layouting will be done on textblock regardless of its visiblity, becase render_pre //will be called and will make it relayout ``` Reviewers: woohyun, zmike, tasn, raster, bu5hm4n Reviewed By: zmike Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D11508
* build: ensure we set ENABLE_LUA_OLD when using a lua interpreterStefan Schmidt2020-05-121-0/+1
| | | | | | | | | We missed to actually set the ENABLE_LUA_OLD. Ross run into some problems with Lua 5.2 builds and we hope this fixes the issue for him. Found and fixed by Marcel Hollerbach (only put in the system by me). Fixes T8705
* evas_object_textblock: fix to free style properly by edje's style_user_popWooHyun Jung2020-05-121-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | Summary: When calling style_user_pop in edje, current style was not freed by style_user_pop in evas properly (since old_ts was not cared at all). Because of this, memory was leaked by none accessible textblock styles which should have been freed. @fix Test Plan: 1. call edje_object_part_text_style_user_pop 2. call edje_object_part_text_style_user_push with new style-text 3. repeat #1 and #2 several times and check the memory (ps -u) Reviewers: ali.alzyod, bowonryu, bu5hm4n, zmike Reviewed By: ali.alzyod Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D11801
* evas_vg_node: deprecate evas_vg_node_geometry_set.JunsuChoi2020-05-122-1/+3
| | | | | | | | | | | | | | | | | | | Summary: The geometry info of evas_vg_node is based on Efl.Path_Bounds. The size of vg_node does not change unless path data is changed. Therefore, this API does not work normally. Test Plan: N/A Reviewers: Hermet, kimcinoo, herb Reviewed By: Hermet Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D11809
* efl_canvas_vg_node: fix the implemenation function of the ↵Taehyub Kim2020-05-121-79/+28
| | | | | | | | | | | | | | | | | | | | efl_canvas_vg_node(evas_object_above_get, evas_object_below_get) Summary: There are wrong implemenation codes in the above and below get objects of efl_canvas_vg_node functin. and made it working correctly. Reviewers: Hermet, jsuya Reviewed By: jsuya Subscribers: singh.amitesh, cedric, #reviewers, #committers Tags: #efl Maniphest Tasks: T8700 Differential Revision: https://phab.enlightenment.org/D11781
* tests: ecore_con another cert update to extend the time its valid to 100 yearsStefan Schmidt2020-05-112-76/+76
| | | | | | | | | Instead of forcing someone to look at this again in a couple of years we can simply extend it to 100 years and expect this code does no longer exist. :-) For more details on the steps I used to generate and the problem in ecore_con please see the original commit 3b835b57ff70659c12a89158cd8d29e3187a42a9
* canvas render: don't apply mask if the objects in the map.Hermet Park2020-05-101-26/+47
| | | | | | | | | | | Current mask is shared across the objects hierarchy, children would try to apply this mask on the rendering when it's enabled by the parent. but if the map is enabled from one of parents, children should not apply the parents mask but leaves it to the parent's job. @fix
* evas + gl - fix callback-to-app-changes-state issue with pixel dataCarsten Haitzler (Rasterman)2020-05-092-2/+4
| | | | | | | | | | | | so if we call a callback that has e do something like set new pixel data in a pixel data provider and we do it from the is pixel inside call ebcause pixels were marked as dirty - pixels may no longer exist after that, so re-get to make sure and handle properly. this was found with tiling + e + shaped windows + moving them around in tiling. it's a valid efl bug though. @fix
* evas/events: fix conditional for emitting source mouse move eventsMike Blumenkrantz2020-05-091-1/+1
| | | | | | | | | | | | | | | | | Summary: this should match all other similar cases when determining whether to emit the event by checking whether at least one coord has changed, not both Depends on D11791 Reviewers: devilhorns Reviewed By: devilhorns Subscribers: raster, cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D11792
* elm: resolve float comparison warningsMike Blumenkrantz2020-05-0931-132/+134
| | | | | | | | | | Summary: Depends on D11790 Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D11791
* bin/elm: resolve float compare warningsMike Blumenkrantz2020-05-097-60/+60
| | | | | | | | | | Summary: Depends on D11789 Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D11790
* tests: resolve float comparison warningsMike Blumenkrantz2020-05-0921-154/+162
| | | | | | | | | | Summary: Depends on D11788 Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D11789
* edje_cc: consistently use eina macros for all float comparisonsMike Blumenkrantz2020-05-091-7/+7
| | | | | | | | | | Summary: Depends on D11787 Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D11788
* edje_cc: fix comparisons for possible non-float valuesMike Blumenkrantz2020-05-092-9/+9
| | | | | | | | | | | | | | Summary: when defined, BUILD_EDJE_FP causes some of these struct members to not be floating types and additionally defines special macros to be used for comparisons to handle this case Depends on D11786 Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D11787
* eina: resolve float comparison warningsMike Blumenkrantz2020-05-092-3/+5
| | | | | | | | Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D11786
* doxygen docs: Fix problems in Evas_CanvasXavi Artigas2020-05-083-13/+16
| | | | | | - Put some methods in a group so they are accessible and can be linked to. - Remove references to Efl_ methods, unavailable in Legacy. - Incorrect param names and typos
* doxygen docs: Put Eina XAttrs in their own groupXavi Artigas2020-05-081-1/+3
| | | | To stop polluting the Eina Tools group.
* doxygen docs: Complete docs for Eina ThreadXavi Artigas2020-05-081-14/+5
|
* doxygen docs: Fix grouping mess in Eina Simple XMLXavi Artigas2020-05-081-14/+1
| | | | Seriously, who came up with such convoluted doxygen code?
* doxygen docs: Complete docs for Eina ModuleXavi Artigas2020-05-082-36/+19
| | | | And fix yet another doxygen grouping mess
* doxygen docs: Complete docs for Eina LogXavi Artigas2020-05-081-2/+2
|
* doxygen docs: Complete docs for Eina CPU functionsXavi Artigas2020-05-081-11/+19
|
* doxygen docs: Complete docs for Eina_RectangleXavi Artigas2020-05-081-8/+28
|
* evas: use SEQUENTIAL for decoding png fileShinwoo Kim2020-05-081-6/+6
| | | | | | | | | | | | | | | | | | | | | | | Summary: There was a mistake caused by "2a0eeba evas: fix png regression issue". The flag is_for_data was close_file before this commit. _evas_image_load_file_internal_head_png closes file if it is called only for header. So the close_file does not mean is_for_data. This patch is changing is_for_data to is_for_head, and make _evas_image_load_file_internal_head_png use EINA_FILE_SEQUENTIAL for data decoding. Reviewers: Hermet, jsuya, herb, raster Reviewed By: raster Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D11783
* tests/elm: add callback testing to image memfile testMike Blumenkrantz2020-05-081-1/+11
| | | | | | | | | | | | | | | | | Summary: verify that ready is called (preload done) and error is not called ref D11758 Reviewers: Hermet, jsuya, kimcinoo Reviewed By: Hermet Subscribers: stefan_schmidt, cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D11760
* ci: travis: update Debian to 10.3 and Ubuntu to 20.04Stefan Schmidt2020-05-071-2/+2
| | | | | Now that the ecore_con tes suite failure are fixed with a renewed ssl certificate we can update these two distros as well.
* tests: ecore_con: renew test ssl certificate which expired in 2016Stefan Schmidt2020-05-072-25/+76
| | | | | | | | | | | | | | | | | | For our unit testing of ecore_con we have a sself-signed certificate for localhost to do some of the tests. It expired in 2016 and this was not really a problem so far. Starting from Debian 10 and Ubuntu 20.04 this resulted in failures thought. Either the newer openssl version is more strict or the config for Debian and Ubuntu is. Anyway, time to generate a new one. Its valied until Sep 23 12:33:13 2024 GMT and I used the follwoing commands to generate the cert/key and the second command to rmeove the key passphrase. openssl req -x509 -newkey rsa:4096 -keyout server.key -out server.pem -days 1600 -subj '/CN=127.0.0.1' openssl rsa -in server.key -out server2.key Someone has to do this again in 4 years. if we stillhave this code around.
* Use __func__ C99 identifier instead of __FUNCTION__ compiler extensionVincent Torri2020-05-0753-225/+225
| | | | | | | | | | | | | | Summary: see http://www.open-std.org/JTC1/SC22/wg14/www/docs/n1124.pdf section 6.4.2.2 page 52 Test Plan: compilation Reviewers: raster, devilhorns Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D11785
* cxx: Initialize _list and _node with nullptrJoão Paulo Taylor Ienczak Zanette2020-05-071-1/+1
| | | | | | | | Fixes CID 1409576 pointing that _node was unitialized. Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org> Reviewed-by: Felipe Magno de Almeida <felipe@expertisesolutions.com.br> Differential Revision: https://phab.enlightenment.org/D11770
* evas: fix png regression issueShinwoo Kim2020-05-071-0/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Summary: Accidentally commit "382c580 evas: add support for .9.png file to PNG loader." adding the 9 patch feature with small code refactoring made use of setjmp incorrectly. [Problem] evas_image_load_file_data_png calls _evas_image_load_file_internal_head_png, and _evas_image_load_file_internal_head_png calls setjmp and returns without problem. And png_read_row calls longjmp. This causes jumping into a function which was exited. Problematic png file will be attached. [Solution] Save calling environment i.e. call setjmp, after returning from _evas_image_load_file_internal_head_png. Test Plan: Problematic png file {F3876983} And example code. {F3876986} Reviewers: Hermet, jsuya, herb Reviewed By: Hermet Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D11782
* canvas map: draw direct image as possible.Hermet Park2020-05-071-7/+6
| | | | | | | Since we moved to floating calculation for msaa, this comparsion is not working for direct image drawing though it's doable. To resolve this, we can compare floatings regardless of msaa enable.