summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* csharp: Hide more Eina stuff.devs/lauromoura/eina-cleanupLauro Moura2019-10-308-25/+26
| | | | | Ref T8165 Ref T8293
* csharp: Remove some uneeded variables.Lauro Moura2019-10-301-4/+0
| | | | Left behind from a previous version of the method.
* csharp: Hide concrete classes.Lauro Moura2019-10-301-1/+1
| | | | | | | | They are not supposed to be public anymore as the event argument wrapper classes are top level citizens (alongside the friendly approach). ref T8165
* csharp: Rename Efl.Eo.NativeClass to NativeMehtodsLauro Moura2019-10-305-20/+45
| | | | | | | | This should avoid confusion with `IWrapper.NativeClass`, which is a property to access the native EO Class of a given instance. Also added some comments and moved EflOpDescription to a proper namespace.
* elm/widget: error on null params for tree_unfocusable functionsMike Blumenkrantz2019-10-301-2/+2
| | | | | | | | | | | | | | Summary: these should error so the user can detect that they screwed up Reviewers: devilhorns Reviewed By: devilhorns Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D10563
* edje_calc: Remove unused flag.Woochanlee2019-10-302-8/+1
| | | | | | | | | | | | | | Summary: This has marked remove me. Reviewers: raster, Hermet, zmike, devilhorns Reviewed By: devilhorns Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D10561
* csharp: FunctionWrapper doc fixesLauro Moura2019-10-302-37/+64
| | | | | | | | | | | | Reviewers: felipealmeida, brunobelo, segfaultxavi, woohyun Reviewed By: segfaultxavi Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D10430
* eina: add comparison macros for Eina_Size2D and Eina_Position2DMike Blumenkrantz2019-10-292-4/+25
| | | | | | | | | | | | | | | | | | | Summary: I'm tired of typing all this out. it's exhausting. also add a couple usages internally to verify that this works as expected @feature Reviewers: cedric, bu5hm4n, devilhorns Reviewed By: devilhorns Subscribers: devilhorns, bu5hm4n, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D10557
* interfaces: replace doubles with Efl.Gfx.Align where appropriateMike Blumenkrantz2019-10-293-27/+19
| | | | | | | | | | | | | | Summary: this makes the types more explicit Depends on D10554 Reviewers: segfaultxavi Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D10555
* efl/gfx: add Efl.Gfx.Align typeMike Blumenkrantz2019-10-291-0/+11
| | | | | | | | | | | | | | | Summary: this can be used to more explicitly specify that a double is intended to be a value of 0.0 to 1.0 for the purpose of aligning objects. it also avoids the need to copy and paste the same docs around everywhere Reviewers: segfaultxavi Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D10554
* meson: add embryo as a dep of edjeMarcel Hollerbach2019-10-291-1/+1
| | | | | | if you want to build a edje theme without building all of efl, you can do that now. Before embryo was missing and no scripts could be compiled. Now this is possible and works.
* meson: redo evas buildingMarcel Hollerbach2019-10-2913-303/+210
| | | | | | | | | | | | | | | | | | | | | | | | | | | Summary: before recent times we had to support static and shared building based on the options of the user, which forced us to complicate our build with the evas_goal hack. the evas_goal hack more or less was the idea of "faking" the evas build in the evas directory, finish all the .eo generation there, then build the modules and make all the static files ready. Then build everything in evas_goal. Now, that we just build everything the same always, we can simply build it in the evas way (removing the evas_goal hack FINALLY), as the same modules are build statically and shared. This also gives us the possibility to build the shared image loaders *again* the the modules directory, which unbreaks peoples build scripts who packaged loader files seperatly. Reviewers: zmike, raster, cedric, stefan_schmidt Reviewed By: zmike Subscribers: #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D10548
* ci: remove existing base_dir ccache config on osx build before adding new oneMike Blumenkrantz2019-10-291-0/+1
| | | | | | | | | | | | | | Summary: ensure that this file does not balloon to infinite size Reviewers: bu5hm4n Reviewed By: bu5hm4n Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D10538
* mono: encapsulate internal workaround membersLauro Moura2019-10-291-15/+15
| | | | | | | | | | | | | | | | Summary: Depends on D10345 Test Plan: meson setup -Dbindings=mono,cxx -Dmono-beta=true Reviewers: segfaultxavi, Jaehyun_Cho, YOhoho Reviewed By: YOhoho Subscribers: cedric, #reviewers, woohyun, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D10346
* mono: encapsulate internal iwrapper membersLauro Moura2019-10-292-52/+52
| | | | | | | | | | | | | | | | Summary: Depends on D10339 Test Plan: meson setup -Dbindings=mono,cxx -Dmono-beta=true Reviewers: segfaultxavi, Jaehyun_Cho, YOhoho Reviewed By: YOhoho Subscribers: cedric, #reviewers, #committers, woohyun Tags: #efl Differential Revision: https://phab.enlightenment.org/D10345
* mono: encapsulate internal delegatesLauro Moura2019-10-292-3/+3
| | | | | | | | | | | | | | | | Summary: Depends on D10337 Test Plan: meson setup -Dbindings=mono,cxx -Dmono-beta=true Reviewers: segfaultxavi, Jaehyun_Cho, YOhoho, brunobelo Reviewed By: YOhoho Subscribers: cedric, #reviewers, #committers, woohyun Tags: #efl Differential Revision: https://phab.enlightenment.org/D10339
* edje: selectively inhibit content,changed events when changing swallowed partsMike Blumenkrantz2019-10-293-1/+11
| | | | | | | ref T8321 Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de> Differential Revision: https://phab.enlightenment.org/D10508
* tests/genlist: add behavior testing for nearly all genlist eventsMike Blumenkrantz2019-10-291-0/+187
| | | | | | | | | | | | | | | | this simulates a number of user interactions to ensure that various functionalities of genlist actually work, including but not limited to: * selection * highlighting * item focus * scrolling * filtering * dragging * item reordering ref T8433 Differential Revision: https://phab.enlightenment.org/D10528
* tests/loop_timer: add port of ecore_timer behavior tests for loop_timerMike Blumenkrantz2019-10-291-0/+81
| | | | | | | ensure that the behavior here is also working exactly as expected Reviewed-by: Cedric BAIL <cedric.bail@free.fr> Differential Revision: https://phab.enlightenment.org/D10531
* tests/ecore: make timer behavior test even more strictMike Blumenkrantz2019-10-291-1/+23
| | | | | | | | | | | | we need to also verify that timers will process out of order solely based on their timestamps and ignoring whether they are "recently-added" additionally verify the behavior of timer interval changing and re-instantiating ref T8434 Reviewed-by: Cedric BAIL <cedric.bail@free.fr> Differential Revision: https://phab.enlightenment.org/D10530
* elm/genlist: fix item focus unregister on item moveMike Blumenkrantz2019-10-291-0/+1
| | | | | | | | | | if the block is realized, its items have been registered into the focus manager and must be unregistered to avoid double-registering @fix Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de> Differential Revision: https://phab.enlightenment.org/D10544
* elm/genlist: hide cached item contents and mark content unfocusable during calcMike Blumenkrantz2019-10-292-9/+21
| | | | | | | | | | | | cached item contents should already be hidden by the edje clipper, so this simply changes their visible state to break them out of the focus calcs contents must also be explicitly marked as unfocusable during calc-only realize operations in order to avoid triggering a full focus recalc which will error due to missing focus adapter in the item block Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de> Differential Revision: https://phab.enlightenment.org/D10543
* elm/genlist: set pan need recalc during item move on item position changeMike Blumenkrantz2019-10-291-5/+12
| | | | | | | | | | item move operations require pan recalc in order to process the item block positioning updates @fix Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de> Differential Revision: https://phab.enlightenment.org/D10533
* elm/genlist: fix "drag" smart callbackMike Blumenkrantz2019-10-291-2/+1
| | | | | | | | | this is only a smart callback and not an eo callback @fix Reviewed-by: Cedric BAIL <cedric.bail@free.fr> Differential Revision: https://phab.enlightenment.org/D10527
* elm/genlist: trigger filter,done callback immediately if no filter queue existsMike Blumenkrantz2019-10-291-3/+5
| | | | | | | | | | the documentation says this should trigger when filtering is done, and if no filtering is pending then it is done @fix Reviewed-by: Cedric BAIL <cedric.bail@free.fr> Differential Revision: https://phab.enlightenment.org/D10526
* elm/genlist: slightly simplify use of _item_filtered_get()Mike Blumenkrantz2019-10-291-16/+14
| | | | | | | | | passing the smart data here (which we always have) makes the function a little clearer to read. similarly, we can check whether the filter_data pointer is set inside the function to avoid having to check it everywhere else Reviewed-by: Cedric BAIL <cedric.bail@free.fr> Differential Revision: https://phab.enlightenment.org/D10525
* elm/genlist: defer recalc when applying a name filterMike Blumenkrantz2019-10-292-2/+8
| | | | | | | | this may be called successively during the same mainloop iteration, so it's important to defer this as much as possible Reviewed-by: Cedric BAIL <cedric.bail@free.fr> Differential Revision: https://phab.enlightenment.org/D10524
* elm/genlist: remove calc jobsMike Blumenkrantz2019-10-292-40/+22
| | | | | | | | | | | | | | | | | | | now that we have a sane and consistent sizing calc mechanism, we no longer need to be randomly creating jobs to do additional side calcs in addition to our other multiple bespoke calculation methods instead, we can now call the calc function directly during the group calc to perform all the calcs at once and avoid the overhead of constantly triggering calc jobs a possible future improvement here could be to remove the _calc_job() call in the internal pan object's group_calculate, as this is likely a duplicated op but it will require additional testing to verify unit test performance (e.g., tree) increases roughly 50-80% after this patch Reviewed-by: Cedric BAIL <cedric.bail@free.fr> Differential Revision: https://phab.enlightenment.org/D10523
* elm/genlist: don't process entire item queue on each item addMike Blumenkrantz2019-10-291-1/+2
| | | | | | | | | | this is a colossal waste of time. it ends up realizing every single item immediately even if it won't be visible, which defeats the purpose of all the batching and viewport calculations that genlist explicitly does to avoid exactly this type of behavior Reviewed-by: Cedric BAIL <cedric.bail@free.fr> Differential Revision: https://phab.enlightenment.org/D10522
* elm/genlist: freeze/thaw canvas when processing item queueMike Blumenkrantz2019-10-291-1/+4
| | | | | Reviewed-by: Cedric BAIL <cedric.bail@free.fr> Differential Revision: https://phab.enlightenment.org/D10521
* elm/genlist: only add calc job on theme apply if already finalizedMike Blumenkrantz2019-10-291-1/+1
| | | | | | | not a required operation otherwise Reviewed-by: Cedric BAIL <cedric.bail@free.fr> Differential Revision: https://phab.enlightenment.org/D10519
* elm/genlist: remove misleading "changed" signalMike Blumenkrantz2019-10-291-1/+0
| | | | | | | | this was never implemented or documented anywhere and serves only to be misleading for anyone reading the code Reviewed-by: Cedric BAIL <cedric.bail@free.fr> Differential Revision: https://phab.enlightenment.org/D10518
* elm/genlist: fix item highlight/unhighlight eventing on clicksMike Blumenkrantz2019-10-291-1/+8
| | | | | | | | | | | | | | | | * highlight should only be triggered on mouse down, as that is the point of interaction with an item which indicates it is "in use"; a mouse-up event cannot occur on an item which has not previously received a mouse-down event, so toggling this on mouse-up will be wrong/duplicated 100% of the time * unhighlight should only be triggered during mouse-up events if the list is in no-select mode, as it will otherwise be implicitly during selection if necessary this should ensure that these events are emitted exactly one time and correctly for each click event Reviewed-by: Cedric BAIL <cedric.bail@free.fr> Differential Revision: https://phab.enlightenment.org/D10517
* tests/ecore: add more grueling timer throughput testMike Blumenkrantz2019-10-291-0/+25
| | | | | | | | | | | | | this verifies that: * newly-created timers are not triggered in the next loop iteration * newly-created timers can be triggered the second loop after created * multiple timers can be triggered in a single loop iteration * timers are effectively added to the pending timer list ref T8434 Reviewed-by: Cedric BAIL <cedric.bail@free.fr> Differential Revision: https://phab.enlightenment.org/D10516
* efl/timer: correctly handle recursion for timer processingMike Blumenkrantz2019-10-291-4/+18
| | | | | | | | | | | | | | | | | | | | if the currently-processed timer is recursively deleted (efl_del) while it is inside the timer tick event callback, we must correctly handle this case: * in the place where a timer's inlist is de-linked, we must check to see if the timer is the current timer and then update that pointer with the next timer in the list * in the post-tick part of timer processing, we must NOT update the current timer pointer if we detect that it has been updated recursively this fixes processing of timers in the mainloop to trigger more than one legacy timer per mainloop iteration and likely has some (positive) impact on mainloop throughput @fix Reviewed-by: Cedric BAIL <cedric.bail@free.fr> Differential Revision: https://phab.enlightenment.org/D10515
* efl/timer: don't attempt instantiating timers until they're finalizedMike Blumenkrantz2019-10-291-1/+2
| | | | | | | this will fail anyway so don't bother Reviewed-by: Cedric BAIL <cedric.bail@free.fr> Differential Revision: https://phab.enlightenment.org/D10514
* tests/genlist: fix tree test sizingMike Blumenkrantz2019-10-291-2/+5
| | | | | | | | for some reason I used 10px as the base height of a genlist item when clearly they are more like 30px Reviewed-by: Cedric BAIL <cedric.bail@free.fr> Differential Revision: https://phab.enlightenment.org/D10513
* tests/elm: add utility function to trigger a mouse dragMike Blumenkrantz2019-10-292-0/+27
| | | | | | | | | | a drag may require a lot of internal calculating to successfully trigger the intended test behavior, so this has a flag which allows it to perform loop iterations and canvas calcs in order to be more universal, along with a global #define for determining exactly how many mouse moves were triggered Reviewed-by: Cedric BAIL <cedric.bail@free.fr> Differential Revision: https://phab.enlightenment.org/D10512
* tests/elm: make argv a static char so it never goes out of scope and crashesMike Blumenkrantz2019-10-291-3/+2
| | | | | Reviewed-by: Cedric BAIL <cedric.bail@free.fr> Differential Revision: https://phab.enlightenment.org/D10511
* tests/elm: add util functions for clicking objects/parts with event flagsMike Blumenkrantz2019-10-292-5/+29
| | | | | | | this is useful for synthesizing e.g., double click events Reviewed-by: Cedric BAIL <cedric.bail@free.fr> Differential Revision: https://phab.enlightenment.org/D10510
* tests/elm: add util callback for incrementing an int dataMike Blumenkrantz2019-10-292-0/+9
| | | | | | | this is a common functionality Reviewed-by: Cedric BAIL <cedric.bail@free.fr> Differential Revision: https://phab.enlightenment.org/D10509
* efl_ui_win: Delete useless geometry set for legacy.edjeWoochanlee2019-10-291-1/+0
| | | | | | | | | | | | | | | | | | Summary: It will be resized on window_resize_job. when the window got specific size. I don't think this is needed here. It cause doing a calc in vain. Test Plan: Launching Test applications. Reviewers: raster, Hermet, zmike Reviewed By: zmike Subscribers: zmike, cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D10550
* edje_calc: Delete duplicated conditional statement.Woochanlee2019-10-291-6/+1
| | | | | | | | | | | | | | | Summary: this is identical to the conditional above and can be merged to improve code clarity Reviewers: raster, Hermet, zmike Reviewed By: zmike Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D10549
* Replace tabs with spacesXavi Artigas2019-10-291-2/+2
| | | | As our coding conventions mandate.
* vg_common_svg: Free node tree for memory leak after eet write.JunsuChoi2019-10-291-0/+2
| | | | | | | | | | | | | | | | | | Summary: The node tree created from vg_common_svg_create_svg_node is not used after eet_data_write(). Therefore, to prevent memory leaks, free the node tree. Test Plan: N/A Reviewers: Hermet, smohanty, kimcinoo Reviewed By: Hermet Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D10443
* eolian_cxx: Add key/value to function_defLauro Moura2019-10-283-43/+210
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Summary: This commit adds explicit eolian key/value/return info to function_def. The original parameter/return types are kept for compatibility. parameter/return -> Mimics the behavior of the generated C API (i.e. single-valued getters have their lone value translated to be their return type) explicit_return -> The actual return value for that block in the EO file. keys/values -> The values for respectively key and values blocks. This should help working with properties in the generators instead of fumbling with paremeter/return directly Fixes T8431 Test Plan: extra tests in the diff Reviewers: felipealmeida, brunobelo, segfaultxavi, YOhoho Reviewed By: brunobelo Subscribers: cedric, #reviewers, #committers, woohyun Tags: #efl Maniphest Tasks: T8431 Differential Revision: https://phab.enlightenment.org/D10494
* csharp: Add comparables operator to eina_error.Bruno da Silva Belo2019-10-281-13/+94
| | | | | | | | | | | | | | | | Summary: ref T8394 Reviewers: lauromoura, felipealmeida, segfaultxavi, YOhoho Reviewed By: YOhoho Subscribers: cedric, #reviewers, #committers Tags: #efl Maniphest Tasks: T8394 Differential Revision: https://phab.enlightenment.org/D10456
* csharp: Add comparables operators to eina_value.Bruno da Silva Belo2019-10-282-68/+69
| | | | | | | | | | | | | | | | Summary: ref T8394 Reviewers: lauromoura, felipealmeida, segfaultxavi, YOhoho, bu5hm4n Reviewed By: YOhoho Subscribers: bu5hm4n, cedric, #reviewers, #committers Tags: #efl Maniphest Tasks: T8394 Differential Revision: https://phab.enlightenment.org/D10458
* eina_mono: remove exceptions in unexpected locationsYeongjong Lee2019-10-282-101/+24
| | | | | | | | | | | | | | | | | | | | | | | | | Summary: Unexpected locations are listed in https://docs.microsoft.com/en-us/visualstudio/code-quality/ca1065 fix CA1065 ref T8402 Test Plan: meson setup -Dbindings=mono,cxx -Dmono-beta=true ninja test Reviewers: lauromoura, felipealmeida, brunobelo Reviewed By: brunobelo Subscribers: cedric, #reviewers, #committers Tags: #efl Maniphest Tasks: T8402 Differential Revision: https://phab.enlightenment.org/D10461
* mono: encapsulate internal NativeModuleLauro Moura2019-10-283-8/+8
| | | | | | | | | | | | | | | | | | Summary: Depends on D10342 Depends on D10338 Test Plan: meson setup -Dbindings=mono,cxx -Dmono-beta=true Reviewers: lauromoura, segfaultxavi, Jaehyun_Cho Reviewed By: lauromoura Subscribers: cedric, #reviewers, #committers, woohyun Tags: #efl Differential Revision: https://phab.enlightenment.org/D10343
View Lorry Controller Status