summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* ecore_wl2: Include window_v6.x in the dist even if not configure for waylanddevs/derekf/build_fixDerek Foreman2018-07-101-2/+4
| | | | | Release tarballs need this file regardless of whether --enable-wayland is specified or not.
* eio_model: Fix deleting files that don't have assigned types yetDerek Foreman2018-07-102-3/+40
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Summary: Trying to delete a file from a creation notification callback can fail. Sometimes the eio model test would sit forever in select() waiting for events that will never occur because of this. This happens since d84a268a71b09d585f1672b44ade698d7babe28d broke deleting of files that haven't yet been assigned a type. Before this commit a delete_me flag would be set before attempting to build a stat buf asynchronously, and then on completion the file would be deleted. I think this was changed because that could potentially race with other async calls and delete the file sooner than expected. So instead of reverting I've made a special delete path that shouldn't race with non-delete paths. Reviewers: devilhorns, zmike Reviewed By: zmike Subscribers: cedric, #committers, zmike Tags: #efl Differential Revision: https://phab.enlightenment.org/D6543
* Revert "edje: clean up internal objects during invalidate"Mike Blumenkrantz2018-07-101-5/+3
| | | | | | | | | This reverts commit 78cd61034185f07779d70163a38f5ffdffe5507b. this patch needed a bit more review+testing; it fixes some issues but creates others Differential Revision: https://phab.enlightenment.org/D6551
* evas gl-x11 engine - nvidia driver - fix performance dropCarsten Haitzler (Rasterman)2018-07-103-0/+11
| | | | | | | | | | | this has been going on for a while. on nvidia drivers in gles mode on x11 there is a massive perf drop to like a few fps with enough windows if we build for egl/gles instead of opengl. it was the re-creating of eglimages every frame. put a vendor specific workaround for this and avoid it. it's not needed there anyway. framerate back to 60fps smoothness afterwards. @fix
* ui scroll_manager: update doc.Hermet Park2018-07-101-2/+2
|
* Ui.Scroll_Manager: restrict to Efl.Ui.Pan type for 'pan' propertyDaniel Hirt2018-07-102-2/+2
| | | | | | | | | | | | | | | | | | Summary: The implementation calls efl_ui_pan_xxx on what was assumed to be any Efl.Canvas.Object. This changes the type to be Efl.Ui.Pan. Also, reordered headers due to the introduced dependency. Reviewers: devilhorns, Hermet Reviewed By: Hermet Subscribers: cedric, #committers, zmike Tags: #efl Differential Revision: https://phab.enlightenment.org/D6536
* edje: clean up internal objects during invalidateMike Blumenkrantz2018-07-101-3/+5
| | | | | | | | | | | | | | | | Summary: these objects are destroyed when invalidate is called as a result of the evas smart object class destroying the smart object at this time ref D6222 Reviewers: bu5hm4n, devilhorns Subscribers: cedric, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D6540
* tests/ecore-con: add a global timeout timer for all testsMike Blumenkrantz2018-07-101-0/+15
| | | | | | | | | | | | | | | | | | | | Summary: it's possible for many tests in this suite to hang indefinitely, so ensure that they terminate eventually instead of holding up the build ref T6838 Reviewers: stefan_schmidt, bu5hm4n, devilhorns, Hermet Reviewed By: Hermet Subscribers: cedric, #committers Tags: #efl Maniphest Tasks: T6838 Differential Revision: https://phab.enlightenment.org/D6542
* updated AUTHORSHermet Park2018-07-101-0/+5
|
* eina: Improve documentation for binbuf/strbuf/ustrbufBryce Harrington2018-07-103-496/+443
| | | | | | | | | | | | | | | | | | | | | | | | Summary: These three classes have essentially the same API, just different underlying data types. This patch seeks to improve the docs for all three while retaining or improving the consistency between them. Several functions had completely incorrect documentation (looks like cut-and-paste mistakes), others had missing or incorrectly named parameter documentation, typographical errors, or other similar issues. This also cleans up a lot of spelling and grammar errors, defines return values as part of @return, and reformats/revises doxygen code for consistency. There are no changes to code, except some whitespace cleanup. Reviewers: devilhorns Subscribers: cedric, #committers, zmike Tags: #efl Differential Revision: https://phab.enlightenment.org/D6544
* elm/ctxpopup: check for content object before removing callbacks during delMike Blumenkrantz2018-07-091-1/+1
| | | | | | | | | | | | | | | | Summary: ref T7030 Reviewers: devilhorns Reviewed By: devilhorns Subscribers: cedric, #committers Tags: #efl Maniphest Tasks: T7030 Differential Revision: https://phab.enlightenment.org/D6541
* tests: Add a password callback for eet_test_identiyDerek Foreman2018-07-091-1/+7
| | | | | | | | | | | | | | | | Summary: For whatever reason the test is stalling for me waiting for me to entire a password at make time. Provide one internally. Reviewers: devilhorns, zmike Reviewed By: zmike Subscribers: cedric, #committers, zmike Tags: #efl Differential Revision: https://phab.enlightenment.org/D6528
* elm_focus_legacy: remove wrong EINA_UNUSEDMarcel Hollerbach2018-07-091-5/+5
| | | | | | | | | | | | | | Summary: Depends on D6532 Reviewers: devilhorns, segfaultxavi Reviewed By: segfaultxavi Subscribers: cedric, #committers, zmike Tags: #efl Differential Revision: https://phab.enlightenment.org/D6533
* elm_focus_legacy: check if top is really a windowMarcel Hollerbach2018-07-091-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Summary: if not, things are going to fall apart, as manager_top then can be NULL or invalid. Top has to be a window element, if this is not the case, then the widget tree of the given widget is dangling somewhere in the void. Calculating the next object in there or even the active manager will result in errors, since the active manager is not really the active manager, but rather just a manager object somewhere in a danging widget subtree. Moving the focus into such a dangling widgettree might result in a stuck focus rect on this object, since the DFS of the focus manager implementation cannot backtrack anymore into the widgets that are still part of the widget graph. Depends on D6531 Reviewers: devilhorns, segfaultxavi, zmike Reviewed By: segfaultxavi Subscribers: cedric, #committers, zmike Tags: #efl Differential Revision: https://phab.enlightenment.org/D6532
* elm_focus_legacy: fix focus moving for legacyMarcel Hollerbach2018-07-091-1/+2
| | | | | | | | | | | | | | | | | | | | Summary: elm_object_focus_next was not working correctly for objects where obj is not the focused object. fix T5940 Reviewers: devilhorns, segfaultxavi, zmike, stefan_schmidt Reviewed By: segfaultxavi Subscribers: cedric, #committers, zmike Tags: #efl Maniphest Tasks: T5940 Differential Revision: https://phab.enlightenment.org/D6531
* elementary: Avoid passing NULL in some win focus methodsXavi Artigas2018-07-091-2/+1
| | | | | | | | | | | | | | | | | | | | | | Summary: This one is rather harmless, as the NULL returned by this fuction is handled correctly in all places, but this commit removes the runtime warning. ref T7030 Test Plan: After this commit the warning disappears. Got the warning with win_example. Reviewers: zmike, bu5hm4n, devilhorns Reviewed By: devilhorns Subscribers: netstar, cedric, #committers Tags: #efl Maniphest Tasks: T7030 Differential Revision: https://phab.enlightenment.org/D6538
* tests/eina: make inclusion for define explicit and more compatibleMike Blumenkrantz2018-07-091-2/+4
| | | | | | | | | | | | | | | | | Summary: signal.h is not included with all build options somehow, making this a bit unreliable. also check for define existence for random platforms which don't define this symbol Reviewers: devilhorns, ManMower Reviewed By: ManMower Subscribers: ManMower, cedric, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D6535
* tests/ecore: check uid on ecore-file testsMike Blumenkrantz2018-07-091-6/+21
| | | | | | | | | | | | | | | | | | | | Summary: these tests will fail if run with root permission, so avoid checking them when run as root ref T7094 Reviewers: devilhorns Reviewed By: devilhorns Subscribers: cedric, #committers Tags: #efl Maniphest Tasks: T7094 Differential Revision: https://phab.enlightenment.org/D6534
* eina: remove no longer used function _eina_thread_queue_msg_block_real_freeStefan Schmidt2018-07-091-17/+0
| | | | | | | | | | | | | | | | Summary: It seems this function is no longer used since some rework. Clean it up. lib/eina/eina_thread_queue.c:127:1: warning: ‘_eina_thread_queue_msg_block_real_free’ defined but not used [-Wunused-function] Reviewers: devilhorns Reviewed By: devilhorns Subscribers: cedric, #committers, zmike Tags: #efl Differential Revision: https://phab.enlightenment.org/D6537
* update poHermet Park2018-07-0924-1896/+1896
|
* C# bindings: Make efl_loop_timer.eo available to bindingsXavi Artigas2018-07-061-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Summary: This file contains both legacy and new API code, this makes it a little difficult to deploy. In Makefile_Ecore.am files can be marked either as legacy or public. If it is not in legacy, make distcheck fails because Ecore_Legacy.h includes efl_loop_timer.eo.h. If it is not in public, bindings are not generated for it. It cannot be in both places, or make install fails, refusing to install the same file twice. Fortunately, there is an ugly place to put files like this one, and it's already full of other outcasts, so... Fixes T7114 Test Plan: make distcheck still works, and C# bindings are generated for efl_loop_timer.eo Reviewers: zmike, bu5hm4n, devilhorns Reviewed By: zmike Subscribers: vitor.sousa, cedric, #committers Tags: #efl Maniphest Tasks: T7114 Differential Revision: https://phab.enlightenment.org/D6527
* popup: fix popup sizing when scroll enabled.Alastair Poole2018-07-061-0/+8
| | | | | | | | | | | | | | | | | | | | | Summary: Force immediate calculate on main_layout after sizing hints set. @fix T6886 Test Plan: Elementary_test: popup -> select scrollable -> use popup examples. Reviewers: #committers, zmike, stephenmhouston, bu5hm4n, devilhorns Reviewed By: #committers, zmike, stephenmhouston Subscribers: cedric Tags: #efl Maniphest Tasks: T6886 Differential Revision: https://phab.enlightenment.org/D6509
* Revert "edje_cc: temporarily disable aborting compile when namespace ↵Mike Blumenkrantz2018-07-061-15/+4
| | | | | | | | | | | | | | | | | | | | validation fails" Summary: This reverts commit 0dc492087ed048f153bc45f488f1504034fdcd30. there are no longer any errors here so this should fail the build any time errors are introduced Reviewers: devilhorns Reviewed By: devilhorns Subscribers: cedric, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D6525
* selection_manager: Silence silly ERR messagesDerek Foreman2018-07-061-5/+5
| | | | | | | | | | | | | | Summary: These should be DBG, they're not errors. Reviewers: devilhorns Reviewed By: devilhorns Subscribers: cedric, #committers, zmike Tags: #efl Differential Revision: https://phab.enlightenment.org/D6526
* selection_manager: Remove unused variableDerek Foreman2018-07-061-2/+0
| | | | | | | | | | | | | | | | | Summary: gcc somehow didn't notice this until an unrelated use of the variable later was removed. Now it's a warning. Depends on D6523 Reviewers: devilhorns Reviewed By: devilhorns Subscribers: cedric, #committers, zmike Tags: #efl Differential Revision: https://phab.enlightenment.org/D6524
* ecore_wl2: Remove ecore_wl2_display_window_findDerek Foreman2018-07-065-36/+7
| | | | | | | | | | | | | | | | Summary: This is now totally trivial and needs not exist. Depends on D6522 Reviewers: devilhorns Reviewed By: devilhorns Subscribers: cedric, #committers, zmike Tags: #efl Differential Revision: https://phab.enlightenment.org/D6523
* ecore_wl2: Replace window ids with pointersDerek Foreman2018-07-0612-214/+187
| | | | | | | | | | | | | | | | | | | | | | | | | | | Summary: There's no benefit to generating ids instead of just using the Ecore_Wl2_Window pointer in events. This has the added benefit of working around a really nasty hash collision bug when multiple ecore_evas engines are used at once. ref T7053 ref T6222 @beta_break Depends on D6521 Reviewers: devilhorns Reviewed By: devilhorns Subscribers: cedric, #committers, zmike Tags: #efl Maniphest Tasks: T7053, T6222 Differential Revision: https://phab.enlightenment.org/D6522
* ee_wayland: Remove extraneous unregisterDerek Foreman2018-07-061-1/+0
| | | | | | | | | | | | | | | | | | | Summary: ecore_event_window_unregister(ee->prop.window) is actually *exactly* the same as ecore_evas_input_event_unregister(ee) So this sequence just uselessly tries to remove something from an empty hash table. Reviewers: devilhorns Reviewed By: devilhorns Subscribers: cedric, #committers, zmike Tags: #efl Differential Revision: https://phab.enlightenment.org/D6521
* build: always check for fork() and clearenv()Mike Blumenkrantz2018-07-061-3/+1
| | | | | | | | | these are used in places other than quicklaunch and must be correctly detected in order for functionality to be enabled @fix Differential Revision: https://phab.enlightenment.org/D6517
* efreet: simplify and fix efreetd launchMike Blumenkrantz2018-07-062-9/+11
| | | | | | | | | | | | | | this previously used an entire eina prefix to determine where to find efreetd, when a simpler approach would have been to just pass the directory where it's being installed this also inhibited running the correct efreetd during in-tree builds and tests, as it was using the install prefix instead of the in-tree wrapper script @fix fix T6713 Differential Revision: https://phab.enlightenment.org/D6516
* ecore-con/proxy_helper: fix in-tree run pathMike Blumenkrantz2018-07-061-1/+1
| | | | | | | | | | | this is located in the bin/ecore_con directory, not bin/ecore_con/utils, so ensure that we pass the correct path in order to avoid not finding the file @fix ref T6713 Differential Revision: https://phab.enlightenment.org/D6515
* edje_cc: disable efreetd connectionMike Blumenkrantz2018-07-061-0/+1
| | | | | | | | | | edje_cc does not use any of this functionality, and it's problematic at build time since efreetd is not run in-tree and does not use in-tree libraries ref T6713 Differential Revision: https://phab.enlightenment.org/D6514
* release: Update NEWS and bump version for 1.21.0-alpha1 releasev1.21.0-alpha1Stefan Schmidt2018-07-062-3/+451
|
* Revert "elua: disable this for now"Stefan Schmidt2018-07-061-1/+1
| | | | | | | | This reverts commit 7b4ff708dc4063731829cc919e056fdc431d6de1. This had a side effect of breaking the distcheck build (which I need for the upcoming alpha today). I talked to Marcel and we agreed on reverting this for now and come back with a fully working patch afterwards.
* elementary textpath: code refactoring.Hermet Park2018-07-051-8/+8
| | | | | | Don't twist code unnecessarily. logically same, round() is enough. Differential Revision: https://phab.enlightenment.org/D6501
* theme: skip namespace validation for "e/modules/music-control/popup" groupMike Blumenkrantz2018-07-051-0/+3
| | | | | | | | | | | as this group is used by an external application, it can never be modified, and thus namespacing issues cannot ever be resolved. for this reason there is no other option except to skip validation of this group fix T7072 Depends on D6388 Differential Revision: https://phab.enlightenment.org/D6389
* edje_cc: add 'skip_namespace_validation' keywordMike Blumenkrantz2018-07-054-16/+52
| | | | | | | | | | | | | | | | | | | | this pairs with the newly-added -N option to denote groups in edc files which should not be checked for correct namespacing the option is useful for cases such as music_control.edc, where a group has been shipping for years with API signals like "btn,clicked" which cannot be changed since they are used in an external application and compatibility must be maintained the documentation for this option explicitly states that it must only be used inside an #ifdef SKIP_NAMESPACE_VALIDATION block, allowing this keyword to be easily removed at a later point ref T7072 @feature Differential Revision: https://phab.enlightenment.org/D6388
* ecore_evas: skip rendering evas and making animator tickHosang Kim2018-07-051-3/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Summary: Some ecore_evas such as ecore_evas_extn_plug doesn't have evas. ecore_evas_extn_plug seems to be Ecore_Evas, but actually it is Evas_Object_Image. ecore_evas_extn_plug makes new ecore evas, but it only exists to communicate with ecore_evas_extn_socket. newly ecore evas only open and close file(ecore_evas_extn_socket). so it doesn't have evas. ``` EAPI Evas_Object * ecore_evas_extn_plug_new_internal(Ecore_Evas *ee_target) { ... ee = calloc(1, sizeof(Ecore_Evas)); ... o = evas_object_image_filled_add(ee_target->evas); ... return o; } ``` Reviewers: zmike, Hermet, woohyun, raster, devilhorns Reviewed By: zmike Subscribers: cedric, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D6504
* evas/main: call correct destroy function in evas_freeMike Blumenkrantz2018-07-051-1/+4
| | | | | | | | | | | | | | | | Summary: an evas may or may not have a parent; this is legacy api and it's all confusing Reviewers: bu5hm4n, devilhorns Reviewed By: bu5hm4n Subscribers: cedric, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D6486
* tests/eo: add explicit test case for setting an object as its own parentMike Blumenkrantz2018-07-051-0/+3
| | | | | | | | | | | | | | Summary: Depends on D6489 Reviewers: bu5hm4n, devilhorns Reviewed By: bu5hm4n Subscribers: cedric, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D6490
* eo/base_class: explicitly refuse to set an object as its own parentMike Blumenkrantz2018-07-051-3/+7
| | | | | | | | somehow this has been possible until now. no. @fix Differential Revision: https://phab.enlightenment.org/D6489
* elm_popup: ensure focus is emitted on this objectMarcel Hollerbach2018-07-052-0/+43
| | | | | | | | | | | | | | | | | | | | | | Summary: This mirrors the focus property from the notify to the one from popup. This is needed to establish legacy behaviour, as earlier popup was able to get focus. ref T6707 Depends on D6510 Reviewers: zmike, devilhorns Reviewed By: zmike Subscribers: cedric, #committers Tags: #efl Maniphest Tasks: T6707 Differential Revision: https://phab.enlightenment.org/D6511
* efl_ui_focus_manager_root_focus: mirror the focus properties correctlyMarcel Hollerbach2018-07-051-1/+14
| | | | | | | | | | | | | | | | | | | | Summary: this object takes a internal dummy object, and thus the properties should be mirrored to the real root. The focus property on the root of the manager is required that the outer world really sees that the focus is on the root or not. Depends on D6506 Reviewers: zmike, devilhorns Reviewed By: zmike Subscribers: cedric, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D6510
* elua: disable this for nowMarcel Hollerbach2018-07-051-1/+1
| | | | | | | | | | | | | | | | | | Summary: the files that are generated are calling files that dont exist anymore. This cannot work and is only eating time when building, disable this for now. It can be reenabled late. The api missing was removed in fc88037977dcc39dfd6d817c522cce01f5bfa024. Reviewers: devilhorns, ManMower Reviewed By: ManMower Subscribers: ManMower, cedric, #committers, zmike Tags: #efl Differential Revision: https://phab.enlightenment.org/D6506
* elementary: Add missing EINA_UNUSED for unused function parameterChris Michael2018-07-041-1/+1
|
* ecore: fix public/private class separationXavi Artigas2018-07-041-7/+6
| | | | | | | | | | | | | | | | | | | | | Summary: A few classes were added in the wrong place in the makefile, rendering them neither public nor legacy, and unaccessible to bindings. Test Plan: The Efl.Model_Item class is now accessible to C#, and previously it wasn't. This class is used in the EO tutorials, which would have needed a rewrite. make check, examples & distcheck still work. Reviewers: bu5hm4n, zmike, devilhorns Reviewed By: zmike, devilhorns Subscribers: cedric, felipealmeida, lauromoura, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D6507
* eina: Improve eina_binbuf function documentationBryce Harrington2018-07-041-51/+44
| | | | | | | | | | | | | | Summary: Define return values as part of @return. Cleanup grammar. Reviewers: devilhorns Subscribers: cedric, #committers, zmike Tags: #efl Differential Revision: https://phab.enlightenment.org/D6503
* evas: add more error messages when passing invalid objects to evas callbacksMike Blumenkrantz2018-07-041-24/+57
| | | | | | | | | | | | | | | | | | | | | Summary: this should only happen if the user has made a mistake regarding the existence or type of an object, so ensure that an error message occurs to help debug any failures which result fix T6326 Reviewers: bu5hm4n, Hermet, woohyun, devilhorns Reviewed By: Hermet Subscribers: cedric, #committers Tags: #efl Maniphest Tasks: T6326 Differential Revision: https://phab.enlightenment.org/D6322
* eina: Improve eina_benchmark function documentationBryce Harrington2018-07-031-19/+13
| | | | | | | | | | | | | | | | | Summary: Define return values as part of @return. Clarify that the returned list of files are gnuplot filenames specifically. Cleanup grammar throughout. Reviewers: zmike, devilhorns Reviewed By: devilhorns Subscribers: cedric, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D6502
* efreet: use correct path when generating error message about stale efreetd ↵Mike Blumenkrantz2018-07-032-2/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | socket Summary: using runtime directory in all cases for this is wrong, as ecore-con has a number of fallback codepaths for the case where runtime directory is not set or not valid. by using the same ecore-con function which ecore-ipc uses to generate the socket string, the error message path should always be the same as the path which is used by efreetd extra linkage was required by efreet in order to use ecore-con functions, so the internal lib variable in the build system was modified to provide this @fix fix T7045 Reviewers: devilhorns Reviewed By: devilhorns Subscribers: cedric, #committers Tags: #efl Maniphest Tasks: T7045 Differential Revision: https://phab.enlightenment.org/D6425
View Lorry Controller Status