summaryrefslogtreecommitdiff
path: root/src/bin
Commit message (Collapse)AuthorAgeFilesLines
...
* 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
* Use __func__ C99 identifier instead of __FUNCTION__ compiler extensionVincent Torri2020-05-071-3/+3
| | | | | | | | | | | | | | 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
* exactness_player: do not test for delay_msMarcel Hollerbach2020-05-051-2/+2
| | | | | | | | | | that is a unsigned int, if its 0 the timer is called as fast as possible. Not doing that breaks exactness. @fix Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org> Differential Revision: https://phab.enlightenment.org/D11774
* ecore_evas_x - fix maximize/unmaximize request corner caseCarsten Haitzler (Rasterman)2020-05-031-2/+36
| | | | | | | state was not managed well.. this will fix that. also a test in elementary for it to you can test it out. @fix
* exactness: remove no longer needed _evas_new()Stefan Schmidt2020-04-301-2/+0
| | | | | | | | | | This has been unused since the move to a preloaded lib and now makes trouble durign compilation on Fedora32. @fix Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de> Differential Revision: https://phab.enlightenment.org/D11767
* edje: efl: add new EFL_VERSION_1_25 define for new cycleStefan Schmidt2020-04-291-1/+3
| | | | | Allow things to be marked for the upcoming 1.25 release version while we develop it.
* eolian_gen: mark empty setter value params as unusedDaniel Kolesa2020-04-281-1/+1
|
* eolian_gen: hack around function pointers generating unused paramsDaniel Kolesa2020-04-281-0/+7
|
* elm - combobox - pretty borken but make it a little lessCarsten Haitzler (Rasterman)2020-04-261-1/+39
| | | | | add some tests so i can see smaller comboboxes with padding and fix the scaling which was broken.
* exactness: check action for NULLStefan Schmidt2020-04-231-5/+11
| | | | | | | | | | If the given events list is NULL the data pointer would be as well. Make sure we check for NULL here before accessing. CID: 1419843 Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de> Differential Revision: https://phab.enlightenment.org/D11754
* exactness: use convinience ecore_file function to get dirStefan Schmidt2020-04-231-5/+5
| | | | | | | | | | Instead of doing our own parsing here we should use ecore_file_dir_get() which uses dirname() already to solve this. CID: 1422196 Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de> Differential Revision: https://phab.enlightenment.org/D11727
* exactness: do not access potential NULL pointerStefan Schmidt2020-04-231-2/+5
| | | | | | | | | | These to code lines should be in one block and not one exectued without the if. CID: 1422198 Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de> Differential Revision: https://phab.enlightenment.org/D11726
* exactness-inspect: do not use sprintfMarcel Hollerbach2020-04-201-2/+2
| | | | | | | | there is not yet a CID for this, but we should probebly not do that. Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org> Reviewed-by: Xavi Artigas <xavierartigas@yahoo.es> Differential Revision: https://phab.enlightenment.org/D11731
* exactness-inspect: remove abscure xor checkMarcel Hollerbach2020-04-201-7/+2
| | | | | | | | | | | this is a obscure check, if this was ever reached, then it would simply crash, because one pointer will be NULL, but strcmp will be called on it. CID 1419854 Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org> Differential Revision: https://phab.enlightenment.org/D11722
* exactness: make sure we handle a negative file descriptorStefan Schmidt2020-04-201-0/+1
| | | | | | | | | | | In an error case the fd could be negative here and we should check before feeding it into fdopen(). This is the same patch we used in recorder.c CID: 1422194 Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de> Differential Revision: https://phab.enlightenment.org/D11728
* exactness: make sure we handle a negative file descriptorStefan Schmidt2020-04-201-0/+1
| | | | | | | | | | In an error case the fd could be negative here and we should check before feeding it into fdopen(). CID: 1422197 Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de> Differential Revision: https://phab.enlightenment.org/D11725
* exactness: handle case where eet_data_image_write() failsStefan Schmidt2020-04-201-4/+11
| | | | | | | | | | We never checked how many bytes had been written. Check on return and propagate error upwards to caller. CID: 1419856 Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de> Differential Revision: https://phab.enlightenment.org/D11724
* exactness: remove unused codeStefan Schmidt2020-04-201-16/+6
| | | | | | | | | | | We no longer support the old .rec format and we can always expect the file to be exu. Coverity found this block to be always true so the else part could not be reached. CID: 1421996 Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de> Differential Revision: https://phab.enlightenment.org/D11723
* exactness-inspect: unit1 can never be NULLMarcel Hollerbach2020-04-171-5/+5
| | | | | | | | | | checkinf for it beeing NULL means that we would have to equip every usage of unit1 with a check, but that is useless. CID 1419859 Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org> Differential Revision: https://phab.enlightenment.org/D11721
* exactness-inspect: fix NULL accessMarcel Hollerbach2020-04-171-4/+8
| | | | | | | | | ii might be NULL so we should ensure it is not NULL to call item_select CID 1419865 Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org> Differential Revision: https://phab.enlightenment.org/D11720
* exactness-inspect: sanitize if clauseMarcel Hollerbach2020-04-171-30/+13
| | | | | | | | | | | | these if clause where a bit bottom up, and the xor operation here seemed totally wrong, with this code we are simply displaying both entiteis of the two structs when they are there. *or* we are replacing it with the fallback. CID1419875 Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org> Differential Revision: https://phab.enlightenment.org/D11719
* elm - genlist test - handl g_data being nullCarsten Haitzler (Rasterman)2020-04-171-13/+14
| | | | fix CID 1396944 1396946
* exactness: limit length for font_path to stay in bufferStefan Schmidt2020-04-171-1/+1
| | | | | | | | | | | We get fonts_dir from a getenv() without and length check. Make sure that we stay in the given buffer size when stitching the file path together. CID: 1422195 Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de> Differential Revision: https://phab.enlightenment.org/D11718
* exactness: fix typo in function nameStefan Schmidt2020-04-174-4/+4
| | | | | | | Simple spelling mistake. overlay instead of overloay. Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com> Differential Revision: https://phab.enlightenment.org/D11714
* exactness: remove no longer used code since move to preloadStefan Schmidt2020-04-172-88/+0
| | | | | | | | | | | This has been around for prg handling before we switched to preload. No need for it anymore. Found when looking for a Coverity issue, which also got fixed now by luck. :-) CID: 1421994 Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com> Differential Revision: https://phab.enlightenment.org/D11713
* efreet - disable strict as this makes things less functional...Carsten Haitzler (Rasterman)2020-04-161-1/+2
|
* efreet - look in /usr/local/share/pixmaps tooCarsten Haitzler (Rasterman)2020-04-162-0/+4
|
* Fix namespace clashes for Efl.Text.CursorXavi Artigas2020-04-141-7/+7
| | | | | | | | | | | | | | | | | | Three renames are applied here: Efl.Text.Cursor -> Efl.Text_Cursor.Object (class) Efl.Text.Cursor_Type -> Efl.Text_Cursor.Type (enum) Efl.Text.Cursor_Move_Type -> Efl.Text_Cursor.Move_Type (enum) Nothing changes for the enums on the C side. For the class... Well, the method names are a bit more verbose now. These renames are required to avoid clashing with the Efl.Text interface. This did not cause trouble to C# because interfaces are prefixed with "I", but it did cause trouble to Eolian when the EO files were installed and somebody tried to use them. Ref T8648 Differential Revision: https://phab.enlightenment.org/D11663
* tests: popupAlastair Poole2020-04-091-2/+2
| | | | | I missed this warning on this machine apologies for the unnecessary noise.
* tests: Add case to match ephoto behaviour.Alastair Poole2020-04-091-0/+76
|
* build: add version to exactness libsMarcel Hollerbach2020-04-061-1/+3
| | | | | | | | this patch was submitted through the bugtracker by Andrew Brouwers. Thank you! fixes T8644
* doxygen docs: some fixes in Evas_Legacy.hXavi Artigas2020-04-021-1/+1
| | | | | | | | | This brings into the docs hundreds of methods! due to the ingroup->defgroup mistake, they were out of any scope and therefore they were silently ignored by doxygen. Also, document lots of missing "obj" parameters. Not strictly necessary, but this further reduces the number of doxygen warnings.
* exactness docs: Add docs, fix docs, remove docsXavi Artigas2020-04-024-44/+96
| | | | | | | | | | | | | | | | | | | | Summary: The Exactness tool needed usage instructions... and quite some more fixes. There was copypasta all around. Depends on D11634 Test Plan: Build and Enjoy Reviewers: bu5hm4n, stefan_schmidt Reviewed By: stefan_schmidt Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D11637
* exactness: protect against forked callsMarcel Hollerbach2020-04-014-11/+47
| | | | | | | | | there came up a issue, where a excatness spawned processes were bringing up a efreetd instance, when the efreetd instance turned off itself, the files for exactness were written again, which is wrong. This ensures that forked instances do not take any actions. Differential Revision: https://phab.enlightenment.org/D11634
* exactness: error when _src_unit is NULLMarcel 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. Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org> 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. Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org> Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com> 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. Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org> Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com> 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. Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org> 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 Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org> Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com> 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 Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org> 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 Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org> Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com> 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. Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org> 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 Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org> 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 Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org> 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. Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org> Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com> Differential Revision: https://phab.enlightenment.org/D11623
* 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
* exactness: handbuild a new argv array instead of reassemling the new oneMarcel Hollerbach2020-03-252-24/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | this commit removes the code that was changing argv values, and replaces it with a new array. Which is absolutly fine, as the argv / argc values are never accessed later on. Only the copies that have been passed to efl_main or elm_main. This resolves several issues: 1. the for loop is useless, every single array element that gets initialized with it, is some offset from argv[0] this may also crash when argv[i] - argv[opt_args] is bigger strlen argv[0]. 2. The memcpy here is super dangerous, the dest array is not garanteed to have the same size as argv[0], this only works if the client application name is shorter than the name "exactness_recorder" 3. The memset here is absolutly wrong. There is again no garantee that the array has the expected size behind that, this was constantly overwriting the segment after the place where argv was stored, which was lukely enough on fedora always the environs, which deleted the couple first segments. (This was not causing any fuzz, since they have been sudo related env vars on the docker image). However, on arch this just crashed right away. On Ubuntu this overwrote DISPLAY, which resulted in the unability to launch the app. Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org> Differential Revision: https://phab.enlightenment.org/D11600
* canvas 3d: remove all canvas3d features across efl.Hermet Park2020-03-252-72/+0
| | | | | | | | | | Reviewers: jsuya, kimcinoo, bu5hm4n, raster Subscribers: cedric, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D11552
* edje_cc: free ecore_evasMarcel Hollerbach2020-03-241-0/+1
| | | | | | | | | | | we have initialized it, we should shutdown it. This was we are not getting random vtable allocation leak reports in the asan job anymore. Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com> Reviewed-by: Chris Michael <cp.michael@samsung.com> Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org> Differential Revision: https://phab.enlightenment.org/D11572