summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* ecore-drm2: Refactor drm2 fb flipping code to avoid NULLsdevs/devilhorns/flipChristopher Michael2019-05-292-34/+33
| | | | | | | | | With certain drivers we end up in a case where output->current, next, and pending FBs all are getting set to NULL which causes freezes/crashes. This patch refactors our ecore_drm2_fb_flip code to avoid that case. @fix
* evas-drm: Check for valid fb before flippingChristopher Michael2019-05-291-5/+13
| | | | | | | If we do not have a valid fb to assign to a plane, then we should not be trying to assign it and doing a flip either. @fix
* ecore-drm2: Check for atomic usage before setting fb on planeChristopher Michael2019-05-291-0/+2
| | | | | | | | Without atomic support, trying to set an FB onto a plane is pointless, so add a quick check at the start of this function to test if we are using atomic. @fix
* efl_ui_relative_layout: prevent infinite loop in chain calculationYeongjong Lee2019-05-291-1/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Test Plan: test code ``` EAPI_MAIN int elm_main(int argc, char **argv) { Eo *win, *layout, *btn1, *btn2, *btn3; win = efl_add(EFL_UI_WIN_CLASS, efl_main_loop_get()); layout = efl_add(EFL_UI_RELATIVE_LAYOUT_CLASS, win, efl_content_set(win, efl_added)); btn1 = efl_add(EFL_UI_BUTTON_CLASS, layout); btn2 = efl_add(EFL_UI_BUTTON_CLASS, layout); efl_ui_relative_layout_relation_right_set(layout, btn1, btn2, 0.0); efl_ui_relative_layout_relation_right_set(layout, btn2, btn1, 0.0); efl_ui_relative_layout_relation_left_set(layout, btn2, btn1, 1.0); efl_ui_relative_layout_relation_left_set(layout, btn1, btn2, 1.0); elm_run(); return 0; } ELM_MAIN() ``` Reviewers: Jaehyun_Cho Reviewed By: Jaehyun_Cho Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D9032
* efl_ui_relative_layout: allow to respect the min size of its childYeongjong Lee2019-05-292-6/+100
| | | | | | | | | | | | | | | | Summary: It will have the same policy as the Efl.Ui.Box, Table. Test Plan: elementary_test -to 'efl.ui.relative_layout' Reviewers: Jaehyun_Cho, herb Reviewed By: Jaehyun_Cho Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D9029
* efl_ui: add a helper for not tollerating errors in a testsuiteMarcel Hollerbach2019-05-287-49/+28
| | | | | | | | | | | | | | | | | Summary: We now also fail in the focus tests Depends on D9021 Reviewers: zmike, cedric, segfaultxavi Reviewed By: zmike Subscribers: #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D9022
* efl_ui_suite: fix errors in focus testsMarcel Hollerbach2019-05-287-152/+145
| | | | | | | | | | | | | | | | | Summary: there have been a few cases where things were just wrong. Like missing regular nodes in a focus manager that received focus, and stuff like this. This now fixes all those cases. Reviewers: zmike, cedric, segfaultxavi Reviewed By: zmike Subscribers: #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D9021
* Evil: fix infinite loop with setlocaleVincent Torri2019-05-281-1/+21
| | | | | | | | | | | | | | | | | | Summary: evil_setlocale was defined as setlocale in evil_private.h *before* evil_setlocale, hence evil_setlocale was calling itself. The solution was to not call evil_private.h in evil_locale.c Test Plan: compialtion and elementary_test Reviewers: zmike, raster, cedric Reviewed By: zmike Subscribers: #reviewers, #committers Tags: #efl Maniphest Tasks: T7985 Differential Revision: https://phab.enlightenment.org/D9030
* eolian: allow parts named like methodsXavi Artigas2019-05-281-10/+14
| | | | | | | | | | | | | | | | | | | | | | | | Summary: The C# bindings turn parts into class properties, so part names cannot clash with method names. However, a "Part" prefix has been recently added, just like it was done for events, and therefore this eolian restriction can be lifted. With this patch part name clashes are only checked among parts, just like it is done for events. Relates to D8582 Test Plan: Everything still builds, because we have no part-method name clashes in the tree, but now they are possible. Reviewers: q66, SanghyeonLee Reviewed By: q66 Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D9031
* build: Fix on OpenBSD.Alastair Poole2019-05-283-3/+3
| | | | | | | | | This adds support for OpenBSD with meson build system. Also use better paths. This resolves further issues when building EFL.` Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de> Differential Revision: https://phab.enlightenment.org/D9028
* efl_ui_stack: fix not to keep final state of push/pop animationsJaehyun Cho2019-05-271-2/+0
| | | | | | | It is just required to show pushed content and hide popped content after the push/pop animations. Therefore, it is not required to keep the final state of the push/pop animations after finishing the push/pop animations.
* ecore_evas win32: fix warning reported by clangVincent Torri2019-05-271-1/+1
| | | | | | | | | | | | | | Test Plan: compilation Reviewers: zmike, raster, cedric, Hermet Reviewed By: Hermet Subscribers: #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D9025
* Evas gesture: do not use abs when dealing with double, use fabs insteadVincent Torri2019-05-271-2/+2
| | | | | | | | | | | | | | | | Summary: clang reports a warning here. We compare the result of abs (casting to an int, returning an int) to a double value. Hence we should use fabs instead Test Plan: compilation Reviewers: Jaehyun, Hermet Reviewed By: Hermet Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D9024
* efl_ui_textpath: redraw text properly.Hermet Park2019-05-271-0/+2
| | | | It needs redraw since text draw depends on the obj' size.
* eolian: add runtime API to get file format versionDaniel Kolesa2019-05-263-0/+15
| | | | | This is useful for FFI based bindings (like the Lua or Python ones) and so on.
* eolian: prevent parsing when eo file version is too newDaniel Kolesa2019-05-262-0/+5
|
* eolian: add test for file versioningDaniel Kolesa2019-05-262-0/+19
|
* elua: add version_get API for units in eolian bindingsDaniel Kolesa2019-05-261-0/+5
|
* eolian: add API to query unit versionDaniel Kolesa2019-05-262-0/+19
|
* eolian: initial versioning implementationDaniel Kolesa2019-05-265-6/+44
| | | | | | | | | | | | | | | | | | | This implements initial support for specifying unit versions. The default version is 1, specifying the basic feature level. If you want to specify another version, you need to specify something like `#version 2` at the beginning of the .eo or .eot file; the version number must be higher than 0 and lower than USHRT_MAX (typically 65536). The beginning of the file is now called the "header section"; other things may be added into the header section later. Version cannot be specified twice, and it cannot be specified once other contents (like types or class definition) appear. Comments do not count as other contents, so those are fine to appear before #version. @feature
* eolian: rename @warn_unused and its associated APIDaniel Kolesa2019-05-2630-75/+75
| | | | | | | | | | | | @warn_unused in syntax is now called @no_unused - this is because "warning about unused" is a C thing (or rather, an extension to C) and various languages might want to use stricter behavior for this. Its associated API does the reverse now - it lets you query whether being unused is allowed at all. This is to match future behavior of Eolian (once it supports versioning) that will likely reverse it. @feature
* eolian: remove param @nonullDaniel Kolesa2019-05-2633-107/+56
| | | | | | | | | | | This has been deprecated for a while and is not strictly necessary - as a part of an effort to stabilize Eolian, remove this. Eolian will eventually gain support for versioning and use a reversed behavior (i.e. no NULL by default), but the API it wlll use for that will be very different. Features can always be added, it's much harder to drop them. @feature
* Rename Efl.Ui.Direction -> Efl.Ui.Layout_OrientationXavi Artigas2019-05-2495-617/+525
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | Summary: This clarifies a bit the whole Orientation vs. Direction confusion, at the expense of longer names (Image_Orientation vs. Layout_Orientation). Also, the interfaces are now adjectives (Orientable) and the enums have long names (*_Orientation). Pretty big patch, but no functional changes. Relates to T7863 Test Plan: Everything builds and passes tests. Elementary_tests show same behavior, including the "inverted" widgets, which are the only parts which received a bit of code changes. Proof: https://travis-ci.org/Enlightenment/efl/builds/536277282 Reviewers: zmike, bu5hm4n, cedric Reviewed By: zmike Subscribers: #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D8946
* Fix last missing reference to old efl_orientation headersXavi Artigas2019-05-241-2/+2
| | | | | | This is now efl_gfx_orientable. We are not building the C++ examples, so I did not pick this up sooner. Travis does not build C++ bidings at all.
* efl_mono: fix autogen make check errorJaehyun Cho2019-05-241-17/+21
| | | | | | | | | Previously, autogen make check error happened because some .eo.h files were not generated but were included in libefl_mono_native_test.h. Now, all .eo.c and .eo.h files are generated. Also CLEANFILES are fixed to include all the generated files.
* csharp: Fix autotools buildLauro Moura2019-05-231-14/+37
| | | | | | | | | | | | | | Summary: After the tests split Reviewers: vitor.sousa Reviewed By: vitor.sousa Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D9015
* efl/gesture: include .h file in private header, not .c fileMike Blumenkrantz2019-05-232-1/+2
| | | | | | | | | | | | | | | | | | | | | Summary: a generated eolian c file cannot be included multiple times or else it will create multiple definitions of symbols defined in that file and break the build ref 54175998d538e0b2173fc023bb822f1e6536e58f fix T7960 Reviewers: devilhorns, vtorri Reviewed By: vtorri Subscribers: cedric, #reviewers, #committers Tags: #efl_build Maniphest Tasks: T7960 Differential Revision: https://phab.enlightenment.org/D8945
* elm_widget_item_static_focus: mark param as usedMike Blumenkrantz2019-05-231-1/+1
| | | | | | | | | | | | | | Summary: Depends on D8935 Reviewers: bu5hm4n, devilhorns Reviewed By: bu5hm4n, devilhorns Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D8936
* efl_ui_focus_manager_sub: handle manager change when tree is set to unfocusableMike Blumenkrantz2019-05-231-0/+2
| | | | | | | | | | | | | | | | | | Summary: in this case, the event is emitted and no manager exists because the objects are not focusable @fix Reviewers: bu5hm4n Reviewed By: bu5hm4n Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D8935
* elm_color_class: use legacy event for genlist selectionMike Blumenkrantz2019-05-231-7/+7
| | | | | | | | | | | | | | | | | Summary: this fixes the selection callback and makes the widget work again @fix Reviewers: bu5hm4n Reviewed By: bu5hm4n Subscribers: bu5hm4n, cedric, #reviewers, #committers Tags: #efl_widgets Differential Revision: https://phab.enlightenment.org/D8913
* evas/render: fix proxy clipping when source_clip is disabledMike Blumenkrantz2019-05-231-9/+10
| | | | | | | | | | | | | | | | | | | | Summary: when source_clip is disabled, the clipper used should be the one used by the proxy object. previously this would never be the case, meaning that disabling source_clip had no effect @fix Depends on D8879 Reviewers: Hermet Reviewed By: Hermet Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D8880
* Emotion gst1 module: fix "empty if statement" warningVincent Torri2019-05-231-6/+23
| | | | | | | | | | | | | | | | Summary: the macro GST_DEBUG_BIN_TO_DOT_FILE_WITH_TS can be undefined (see gst core doc), hence the if statement may have no body Test Plan: compilation Reviewers: zmike, raster, cedric, Hermet Reviewed By: Hermet Subscribers: #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D8939
* Emotion gst1 module: remove whitespacesVincent Torri2019-05-231-61/+60
| | | | | | | | | | | | Reviewers: zmike, cedric, raster, Hermet Reviewed By: Hermet Subscribers: #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D8938
* Efl.Ui.Textpath: fix a size problemShinwoo Kim2019-05-231-9/+33
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Summary: Textpath set its hint_size using its position and calulated path information. The path information is calculated using center postion, radius & start angle. For example, if textpath position is 0,0 and center positon of path is 100,100 and radius 50, then text will be located on the rigth side of textpath object. Moreover there is another problem. [Step to reproduce] 1. Launch elementary_test 2. open Efl.Ui.Textpath 3. see circle is not center aligned. 4. click Clockwise 1 ~ 2 times center position is changed repeatedly. This occurs because textpath size is calculated by position_set. It does not make sense that changing postion defines its size. So this patch is setting textpath hint_size using only given center position. The text will position on the middle of textpath object always. Reviewers: Hermet, jsuya Reviewed By: Hermet Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D8930
* csharp: Split test filesLauro Moura2019-05-228-210/+225
| | | | | | | | | | | | | | Summary: Make it easier to add new tests Reviewers: vitor.sousa Reviewed By: vitor.sousa Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D8937
* Rename Efl.Gfx.Orientation -> Efl.Gfx.Image_OrientationXavi Artigas2019-05-2218-115/+111
| | | | | | | | | | | | | | | | | | | | Summary: This will help disambiguate later on when we rename Efl.Ui.Direction to Efl.Ui.Layout_Orientation. Relates to T7919 Test Plan: Everything still builds and passes tests. Reviewers: zmike Reviewed By: zmike Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D8934
* benchmarks: Fix copy-paste errorChristopher Michael2019-05-221-1/+1
| | | | | | | | | Coverity reports this as a copy-paste error, and checking the code it certainly looks that way, so lets call the proper hash function here Fixes CID1401052 @fix
* ecore_ipc: close dialer io closer when dialer is deleted.Hosang Kim2019-05-221-0/+6
| | | | | | | | | | | | | | | | Summary: dialer uses copier class, so dialer's close on invalidate flag is EINA_FALSE. Therefore dialer io closer is not closed automatically. Reviewers: barbieri, raster, Hermet, zmike Reviewed By: zmike Subscribers: zmike, cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D8922
* include evil_private.h in last files, and disable symbolic links on Windows ↵Vincent Torri2019-05-228-1/+23
| | | | | | | | | | | | | | | | in a couple of files Test Plan: compilation Reviewers: zmike, raster, cedric Reviewed By: zmike Subscribers: #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D8933
* ecore-evas-drm: Check if vblank is supportedChristopher Michael2019-05-221-0/+11
| | | | | | | | Small patch to check if the video driver supports drmWaitVBlank. If not, we can fall back to timer based animators and avoid freezing (due to usage of vsync'd animators). @fix
* ecore-drm2: Add API to check if vblank is supportedChristopher Michael2019-05-222-0/+28
| | | | | | | | | | | This patch adds a small API that we can use to check if the current video driver supports the usage of drmWaitVBlank. This check is required for certain drivers (like vbox) which do not support drmWaitVBlank and thus are causing our animators in ecore_evas to freeze. We can now use this API from within Ecore_Evas to disable vsync'd animators and fall back to timer based ones. @feature
* evas/scale_sample: even more code deduplicationXavi Artigas2019-05-211-45/+15
| | | | | | | | | | | | | | | | | | Summary: how many times was this block of code copied around??? no functional changes Depends on D8848 Reviewers: segfaultxavi Reviewed By: segfaultxavi Subscribers: segfaultxavi, cedric, #reviewers, #committers Tags: #efl_rendering Differential Revision: https://phab.enlightenment.org/D8849
* evas/scale_sample: remove unnecessary clamping to mask geometryMike Blumenkrantz2019-05-211-10/+0
| | | | | | | | | | | | | | | | | | Summary: this is now handled entirely in the masking helper function no functional changes Depends on D8847 Reviewers: segfaultxavi Reviewed By: segfaultxavi Subscribers: segfaultxavi, cedric, #reviewers, #committers Tags: #efl_rendering Differential Revision: https://phab.enlightenment.org/D8848
* evas/scale_sample: further deduplicate masking codeMike Blumenkrantz2019-05-211-64/+66
| | | | | | | | | | | | | | | | | | | | Summary: by adding a couple extra params to existing functions, we can reuse existing code instead of copying it around and adding more bugs no functional changes Depends on D8846 Reviewers: segfaultxavi Reviewed By: segfaultxavi Subscribers: segfaultxavi, cedric, #reviewers, #committers Tags: #efl_rendering Differential Revision: https://phab.enlightenment.org/D8847
* evas/scale_sample: deduplicate masking codeMike Blumenkrantz2019-05-211-48/+11
| | | | | | | | | | | | | | | | | | Summary: these functions provide identical functionality to the inline blocks no functional changes Depends on D8841 Reviewers: Hermet Reviewed By: Hermet Subscribers: Hermet, cedric, #reviewers, #committers Tags: #efl_rendering Differential Revision: https://phab.enlightenment.org/D8846
* Eio: avoid symbolic link on WindowsVincent Torri2019-05-211-0/+8
| | | | | | | | | | | | | | | | Summary: On Windows, symbolic links do not exist Test Plan: compilation Reviewers: zmike, raster, cedric Reviewed By: zmike Subscribers: #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D8931
* eina: move builtin detection to eina_config.hMarcel Hollerbach2019-05-213-24/+18
| | | | | | | | | | | | | | Summary: this means we don't need to handle this in the buildtool at all. Reviewers: vtorri, zmike Reviewed By: zmike Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D8553
* eolian: remove @nullable keywordDaniel Kolesa2019-05-2129-165/+63
| | | | | | | | | | This was an experiment that never properly took off and was never used by any generator. Its use was highly variable, so it could not be relied upon. We will still want to reverse the current behavior eventually (no null by default), but that will be done with eo file versioning in the future. @feature
* docs: remove obsolete docgen from the treeDaniel Kolesa2019-05-2110-4668/+1
| | | | | This has long been replaced with tools/edocgen.git and most likely does not even work.
* elm_entry: prevents invalid cursor position updatesBowon Ryu2019-05-211-1/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | Summary: sd->cursor_pos is updated in _entry_cursor_changed_signal_cb. Generally, there is no problem. But in some cases, before the _entry_cursor_changed_signal_cb is called there is a situation in which cursor_pos is updated through _elm_entry_efl_ui_widget_theme_apply. In this case, before _entry_cursor_changed_signal_cb is called, in _elm_entry_efl_ui_widget_theme_apply () cursor_pos = sd->cursor_pos; The wrong cursor_pos is set here. Because it is the value before sd->cursor_pos is updated. This causes an invalid cursor position when entering a key into the entry. This patch prevents sd->cursor_pos from being updated with invalid values. Reviewers: zmike, woohyun Reviewed By: zmike Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D8923
View Lorry Controller Status