summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* exactness: error when _src_unit is NULLdevs/stefan/exactness-preloadMarcel Hollerbach2020-03-311-6/+5
| | | | | | | if _src_unit is NULL, the write here would delete the actions, with this commit we ensure that this is printing an error. Differential Revision: https://phab.enlightenment.org/D11627
* exactness: equip with ERR and CRITMarcel Hollerbach2020-03-311-5/+15
| | | | | | | | | | | before a few commits, we had the situation that errors were overseen because the log was simply so big, that errors did not get shown properly. With this commit, exactness will simply abort if there is a real issue in the code. Differential Revision: https://phab.enlightenment.org/D11624
* exactness: correctly call mkdirMarcel Hollerbach2020-03-311-11/+20
| | | | | | | we should not error when mkdir returns < 0. EEXIST should not result in the return here. Differential Revision: https://phab.enlightenment.org/D11618
* exactness: do not efl_del the evasMarcel Hollerbach2020-03-311-1/+0
| | | | | | | | | | there is no need to do that, more than that. This is super dangerous, the display and connection ptr of x are passed from ecore_evas to evas, if you delete evas before ecore_evas, the later ecore_evas deletion will destroy the x connection which calls some functions in evas, which is already freed, which leads to a crash. Differential Revision: https://phab.enlightenment.org/D11617
* exactness_play: introduce runner scriptMarcel Hollerbach2020-03-312-0/+104
| | | | | | | this is just a little python script, so you can lunch exactness_play without the need of handdefining LD_PRELOAD Differential Revision: https://phab.enlightenment.org/D11616
* exactness_play: transform to LD_PRELOAD libraryMarcel Hollerbach2020-03-312-142/+101
| | | | | | this can now be loaded as LD_PRELOAD library Differential Revision: https://phab.enlightenment.org/D11615
* exactness_player: refactor main methodMarcel Hollerbach2020-03-311-98/+145
| | | | | | | this refactors everything that is not argc argv related out of the main method. For later usage Differential Revision: https://phab.enlightenment.org/D11614
* exactness: remove code for compilingMarcel Hollerbach2020-03-313-54/+2
| | | | | | | | this seems to be made for compiling binaries before testing. That sounds like a good idea, however, implementing a full buildtool in exactness is a bit hard. Hence, using meson for that would be better. Differential Revision: https://phab.enlightenment.org/D11613
* exactness_record: introduce runner scriptMarcel Hollerbach2020-03-312-1/+72
| | | | | | | this is just a little python script, so you can lunch exactness_record without the need of handdefining LD_PRELOAD Differential Revision: https://phab.enlightenment.org/D11611
* exactness_recorder: move to preloadMarcel Hollerbach2020-03-312-126/+82
| | | | | | | | | this is now not a binary anymore, that dlopen's a binary, it is now a library, that can be loaded using LD_PRELOAD. EXACTNESS_DEST is used for the path of the .exu file. EXACTNESS_FONTS_DIR is used to get the fonts directory Differential Revision: https://phab.enlightenment.org/D11610
* exactness_recorder: refactor main methodMarcel Hollerbach2020-03-313-35/+67
| | | | | | | all calls taht are not related to env var checking, are moved out of the main method. That is in preparation for later refactorings. Differential Revision: https://phab.enlightenment.org/D11623
* Testing onlyStefan Schmidt2020-03-311-12/+0
|
* ecore_evas_wayland: Do not pass negative parameter to close functionChristopher Michael2020-03-301-1/+1
| | | | | | | | Coverity reports that 'fd' here is negative, and close() cannot accept a negative parameter, so add a check to make sure 'fd' is not negative before passing to close function. Fixes CID1420318
* ecore_evas_wayland: Declare variables at top of functionChristopher Michael2020-03-301-1/+3
|
* efl_ui_textbox: pass in correct seatMarcel Hollerbach2020-03-301-2/+2
| | | | | | | | | 0 was the wrong seat, and only worked on a small amount of systems fixes T8639 Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com> Differential Revision: https://phab.enlightenment.org/D11622
* canvas vg: null initialization for safety.Hermet Park2020-03-301-1/+1
|
* exactness_recorder: cleanup code a bitMarcel Hollerbach2020-03-301-4/+1
| | | | | | | | _test_name was not used globally, so lets move it to the used scope. Verbose is not used at all either. Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org> Differential Revision: https://phab.enlightenment.org/D11609
* elm_atspi_bridge: initialize variablesShinwoo Kim2020-03-301-4/+4
| | | | | | | | | | The efl_access_text_attribute_get is resolved by elm_entry. Please refer to _elm_entry_efl_access_text_attribute_get first. Uninitialized variables are used for its parameters, and it is able to return before setting these variables. Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de> Differential Revision: https://phab.enlightenment.org/D11619
* ci: enable ecore_con IPv6 testsStefan Schmidt2020-03-271-0/+1
| | | | | | | | | | | We have them behind an env variable as this is system specific. Enable them on CI to fully test these paths as well. While this works out of the box for the native build on Travis it fails for our docker builds, as the dockerd does not have ipv6 enabled by default. Only enable for Codecov for now. Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com> Differential Revision: https://phab.enlightenment.org/D11608
* ci: travis: enable s390x build on daily cronStefan Schmidt2020-03-273-1/+15
| | | | | | | | As there is no luajit package for s390x in Ubuntu Bionic we need to adapt our installed packages as well as config options for it. Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de> Differential Revision: https://phab.enlightenment.org/D11601
* travis: enable more archs during cron buildStefan Schmidt2020-03-272-4/+18
| | | | | | | | | | Enable builds on ppc64le on our daily build. s390x will follow at some later point (once all quirks are resolved). As we also use the Linux native build from Travis for Codecov we start to handle meson options based on builds here as well. Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de> Differential Revision: https://phab.enlightenment.org/D11586
* ci: install needed packages for ppc64le buildsStefan Schmidt2020-03-271-1/+1
| | | | | | | | It seems on ppc64le and s390x these do not get pulled in automatically. We need them for regex.h, make sure they are installed before building. Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de> Differential Revision: https://phab.enlightenment.org/D11585
* Eina: Remove eina_prime_tableXavi Artigas2020-03-272-9/+0
| | | | | | | | | This has been present since ancient times and no current uses have been detected. It's an array containing SOME prime numbers without a terminator, so the caller needs to know the array length in advance. This does not look very useful (or usable) and therefore it has been decided in public IRC session to send this thing to hell. Witnesses: raster stefan_schmidt bu5hm4n
* doxygen docs: Add missing docs for core Eina typesXavi Artigas2020-03-272-10/+39
| | | | Things like EINA_LIKELY or EAPI were undocumented...
* elm_atspi_bridge: fix bugs reported by coverityBartlomiej Grzelewski2020-03-271-2/+6
| | | | | | | | | | Prevent usage of uninitilized values. CID: 1421995, 1421997, 1422001 Signed-off-by: Bartlomiej Grzelewski <b.grzelewski@samsung.com> Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org> Differential Revision: https://phab.enlightenment.org/D11612
* elm_atspi_bridge: fix bugs reported by coverityBartlomiej Grzelewski2020-03-271-5/+8
| | | | | | | | | Make sure we free allocated resources in the error path. Prevent usage of uninitilized value. Signed-off-by: Bartlomiej Grzelewski <b.grzelewski@samsung.com> Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org> Differential Revision: https://phab.enlightenment.org/D11607
* docs: Fix Eina Core GroupMyoungwoon Roy, Kim2020-03-274-6/+19
| | | | | | | | | | | | | | | | Summary: I had fixed unlinked Eina API group(eina_main, eina_types, hamster) from Eina nodes. Those APIs included into Core group before. Test Plan: API reference documentation modification only Reviewers: segfaultxavi Reviewed By: segfaultxavi Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D11605
* copying: remove obsolete items from copying that dont exist anymoreCarsten Haitzler (Rasterman)2020-03-261-2/+0
|
* eina: prefetch for _eo_obj_pointer_getMarcel Hollerbach2020-03-261-3/+6
| | | | | | | | | | Reviewers: raster, stefan_schmidt Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D11603
* fix segfault when there is no text in clipboardVincent Torri2020-03-261-12/+18
| | | | | | | | | | | | | | Test Plan: cnp a clipboard content without text Reviewers: zmike Reviewed By: zmike Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D11565
* doxygen docs: Fix warnings in elm_config.hXavi Artigas2020-03-262-9/+11
|
* doxygen docs: Fix warnings in Ecore_Common.hXavi Artigas2020-03-263-7/+11
|
* doxygen docs: Fix several warnings in Eina filesXavi Artigas2020-03-2611-26/+26
|
* eo: move all call assignmentMarcel Hollerbach2020-03-261-3/+2
| | | | | | | | this improves the cache performance a lot. Caches are only invalidated once, and not multiple times. Reviewed-by: Carsten Haitzler (Rasterman) <rasterman.com> Differential Revision: https://phab.enlightenment.org/D11592
* eo: there is no reason to equip for calls to classesMarcel Hollerbach2020-03-261-49/+15
| | | | | | | | | | | calls to classes are not needed anymore, since class inheritance is not a thing anymore. After removing is_obj from the function the compile can optimize the code better, since assignments to fields are not conditionally anymore. Reviewed-by: Carsten Haitzler (Rasterman) <rasterman.com> Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org> Differential Revision: https://phab.enlightenment.org/D11591
* evas_textblock: prevent updating cursor unless they are ready during markup_setAli2020-03-262-6/+23
| | | | | | | | | | | | | | | | | | | Summary: During Markup_set at text block level, we will not update the cursors, unless their status is updated and ready. This can cause serious issues, especially if a cursor also depends on another cursor for some calculations, (like the segfault happening in TextBox T8637) Reviewers: woohyun, bu5hm4n, zmike Reviewed By: woohyun Subscribers: cedric, #reviewers, #committers Tags: #efl Maniphest Tasks: T8637 Differential Revision: https://phab.enlightenment.org/D11598
* efl_ui_spin_button: add tests for direct input featureMarcel Hollerbach2020-03-251-0/+55
| | | | | | | | this simply passes in some key sequences and checks if the validation is working. Additionally the output value is checked. Depends on D11009 Differential Revision: https://phab.enlightenment.org/D11010
* suite_helpers: add helper for key down eventsMarcel Hollerbach2020-03-252-0/+16
| | | | usefull for emulating key events.
* suite_helper: do not use stacked values hereMarcel Hollerbach2020-03-251-1/+1
| | | | | | | this is ending up beeing garbage pointers when not properly overwritten. Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com> Differential Revision: https://phab.enlightenment.org/D11597
* evas_textblock_tests: do not free a text cursor hereMarcel Hollerbach2020-03-251-1/+1
| | | | | | | | | the cur_obj would have been always dead at this point, as the textblock was freed. This now moved the textcursor object to a previous point. However, we could also remove the explicit cursor deletion... Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com> Differential Revision: https://phab.enlightenment.org/D11596
* efl_check: unset the callback hereMarcel Hollerbach2020-03-251-0/+1
| | | | | | | | | | this is needed in order to not accidently overwrite stack values. This never showed up while running the test suite, as erroring would have aborted anyways. However, when we are running with CK_FORK=no, this may leak into another test. Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com> Differential Revision: https://phab.enlightenment.org/D11595
* evas: free font lists correctlyMarcel Hollerbach2020-03-251-0/+2
| | | | | | | | | sometimes a canvas object is zombying, and does not get freed correctly. If this is right now happening in our test suite, we are going to free the list, but do not clean up the pointer in there. Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com> Differential Revision: https://phab.enlightenment.org/D11594
* edje/multisense: track and destroy multisense inputs on shutdownMike Blumenkrantz2020-03-251-0/+6
| | | | | | | leaking these means leaking file references, and there's no reason for it Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de> Differential Revision: https://phab.enlightenment.org/D11533
* edje/multisense: free multisense output on shutdownMike Blumenkrantz2020-03-251-2/+1
| | | | | | | this comment has now been addressed Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de> Differential Revision: https://phab.enlightenment.org/D11532
* ecore/audio: rewrite pulseaudio output to not use global variablesMike Blumenkrantz2020-03-251-49/+76
| | | | | | | | | this breaks down immediately when calling init/shutdown in quick succession due to the async nature of pulseaudio. we have object-based private data, so we can just use that instead Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de> Differential Revision: https://phab.enlightenment.org/D11531
* ecore/audio: correctly handle sndfile error when setting new vioMike Blumenkrantz2020-03-251-0/+4
| | | | | | | need to immediately call the free here to avoid leaking the callback data Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de> Differential Revision: https://phab.enlightenment.org/D11530
* ecore/audio: remove duplicated vio code from sndfile inputMike Blumenkrantz2020-03-251-25/+5
| | | | | | | this is already done in the base implementation Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de> Differential Revision: https://phab.enlightenment.org/D11529
* ecore/audio: add a couple more functions to the pulse callback structMike Blumenkrantz2020-03-252-0/+4
| | | | | | | to be used later Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de> Differential Revision: https://phab.enlightenment.org/D11528
* ecore/audio: free vio on in/out object destructionMike Blumenkrantz2020-03-252-2/+10
| | | | | | | | ensure we don't leak these since that means we also leak the user's free data and callback Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de> Differential Revision: https://phab.enlightenment.org/D11527
* ecore/audio: add ERR messages when trying to overwrite existing vioMike Blumenkrantz2020-03-252-2/+8
| | | | | | | this probably causes a crash or a leak at some point Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de> Differential Revision: https://phab.enlightenment.org/D11526
View Lorry Controller Status