summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* tests: making infra to cover all focus testsdevs/woohyun/focus_testsWooHyun Jung2017-08-041-21/+45
|
* Merge branch 'devs/woohyun/focus_tests' of ↵WooHyun Jung2017-08-010-0/+0
|\ | | | | | | ssh://git.enlightenment.org/core/efl into devs/woohyun/focus_tests
| * tests: add elm_test_focus_legacyWooHyun Jung2017-07-315-86/+92
| | | | | | | | | | In this test, you can add any test case which is based on legacy focus APIs with elementary widgets.
| * elm_test_focus: add a test case for focus movingWooHyun Jung2017-07-311-0/+85
| |
* | tests: add elm_test_focus_legacyWooHyun Jung2017-08-015-86/+92
| | | | | | | | | | In this test, you can add any test case which is based on legacy focus APIs with elementary widgets.
* | elm_test_focus: add a test case for focus movingWooHyun Jung2017-08-011-0/+85
| |
* | eldbus - make file local func added in fix staticCarsten Haitzler (Rasterman)2017-08-011-1/+1
| | | | | | | | better to keep the func local unless needed elsewhere too
* | ecore_imf: check return value of memory allocationJihoon Kim2017-08-012-1/+11
| | | | | | | | Signed-off-by: Jihoon Kim <jihoon48.kim@samsung.com>
* | ecore_drm2: Stop making dumb fbs and mmapping for gbm fdsDerek Foreman2017-07-311-12/+0
| | | | | | | | | | We were doing this so E could do screenshots in a hackish way, E has been fixed so this hack here no longer needs to live.
* | ecore_drm2: Remove dead code in blank time getDerek Foreman2017-07-311-1/+0
| | | | | | | | | | When I added the fallback block I left an if statement that could never evaluate to true. Remove it.
* | wayland-shm: Set CLOEXEC for dmabufDerek Foreman2017-07-311-1/+1
|/ | | | Stop leaking the render node fd to child processes.
* eldbus: check message serial before usingShinwoo Kim2017-07-311-7/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Summary: Whatever the dbus_connection_send_with_reply returns, the serial value should be checked, because if the seral value is invalid a process could be aborted. There is backtrace as below. The dbus_connection_send_with_reply could return TRUE even though it has a problem. Please refer to following comment: /* Refuse to send fds on a connection that cannot handle them. Unfortunately we cannot return a proper error here, so the best we can do is return TRUE but leave *pending_return as NULL. */ Test Plan: There is not a exact reproduce step. If the Tizen login manager is relaunched repeatedly, then the dbus and other service processes are relaunched. If a service process tries to use dbus when the dbus has problem as above, then it could be possilbe to get above backtrace. Reviewers: raster, zehortigoza Subscribers: cedric, jpeg Differential Revision: https://phab.enlightenment.org/D5053 @fix
* efl_ui_win: add EFL_UI_WIN_SOCKET_IMAGE type checkJiyoun Park2017-07-311-0/+1
| | | | | elm_win_socket_listen api only work for EFL_UI_WIN_SOCKET_IMAGE type window. So it is better to add type check code.
* evas - handle devices that on deletion delete other devices on shutdownCarsten Haitzler (Rasterman)2017-07-312-1/+12
| | | | this fixes a whole bunch of warnings that just recently turned up...
* elocation - fix return of status, address, position to actually ret dataCarsten Haitzler (Rasterman)2017-07-311-3/+3
| | | | | | | | the data was never returned! fix fix fix. found by PVS studio @fix
* ecore x - convert to using unsed instead of dummy state,ents after retCarsten Haitzler (Rasterman)2017-07-311-2/+1
| | | | found by PVS studio
* elm code - handle alloc failure for line splitCarsten Haitzler (Rasterman)2017-07-311-1/+1
| | | | | | | | | | this culd optentially use large offsets before accessing the ptr thus it may not segv inthe first page but actually work if positionoffsets are large enouh, so handle this case explicitly. @fix found by PVS studio
* elm code - handle alloc failure for line splitCarsten Haitzler (Rasterman)2017-07-311-2/+4
| | | | | | | | | | this culd optentially use large offsets before accessing the ptr thus it may not segv inthe first page but actually work if positionoffsets are large enouh, so handle this case explicitly. @fix found by PVS studio
* evas mem - remove redundenty unused evas mem absractionCarsten Haitzler (Rasterman)2017-07-315-47/+5
| | | | | | | was never used as was not practical to do so so it effectively does nothing so remove it to reduce analyser warnings found by PVS studio
* efreet icon cache creation binary - remove redundent checkCarsten Haitzler (Rasterman)2017-07-311-2/+2
| | | | found by PVS studio
* elm theme - remove redundent checkCarsten Haitzler (Rasterman)2017-07-311-1/+1
| | | | found by PVS studio
* elm config tool - fix redundent checkCarsten Haitzler (Rasterman)2017-07-311-1/+1
| | | | found by PVS studio
* evas textblock - remove redundant checkCarsten Haitzler (Rasterman)2017-07-311-1/+1
| | | | found by PVS studio
* epp - cpp - fix buffer end/null checks given static buffer pathsCarsten Haitzler (Rasterman)2017-07-311-6/+6
| | | | | | | | | this is an ncient bug that's never been triggered... but it's there. fix and compare to null buffer. found by PVS studio @fix
* epp - remove redundent check for charCarsten Haitzler (Rasterman)2017-07-311-1/+1
| | | | | | makes static analyisers happy... not a bug found by PVS studio
* evas cache2 - remove excess file and key checks - only check fileCarsten Haitzler (Rasterman)2017-07-311-2/+2
| | | | | | | no need for key as no key is handled and no file is handled too. cleaner code. found by PVS studio
* elm gesture layer - use fabs not abs when actually comparing floatsCarsten Haitzler (Rasterman)2017-07-311-2/+2
| | | | | | | | | 2 abs uses should have been fabs as they compe a src float with anot her float. found by PVS studio @fix
* eina debug - set length after freeing/iterating over the keysCarsten Haitzler (Rasterman)2017-07-311-1/+1
| | | | found by PVS studio
* elm calendar - fix gmtime call to use an actual valueCarsten Haitzler (Rasterman)2017-07-311-1/+1
| | | | | | | e85c92662bfec6a88bb7d2d091682daa429acba0 added this bug. before it was always set to 259200 (first sunday after epoch). found by PVS studio
* ecore_drm2 - fix startup if edid blob is null.. assume all 0'sCarsten Haitzler (Rasterman)2017-07-301-0/+6
| | | | | | | this causes a crash when no edid is available. survive and dont crash. this causes e wayland to not work at all on my baytrail laptop. @fix
* eldbus lower error print levesl from ER to WRN in get all propsCarsten Haitzler (Rasterman)2017-07-301-2/+2
| | | | | | this is generally unnecessary noise as the majority of the time i see this it's because the message was canceled by the caller... thats not an error though then... it's intended. less debug noise to sift through.
* ecore_evas_drm: Only check for gpu time offset if env var setDerek Foreman2017-07-291-0/+5
| | | | | | | | Since most systems shouldn't need this, and it's possible that it's causing trouble for some systems, make it off by default. Fix T5806
* tests - make ecore con test suite reliable and workCarsten Haitzler (Rasterman)2017-07-292-30/+7
| | | | | | | | basically ecore_con's test suite was broken. ipv6 checks were broken. you now need to set an env var to get ipv6 tested. using eocre_con to detetc it didnt init eina or ecore_con and if you do init these then the other tests that check return values from these fail... so make it an env var if you want to test ipv6. now it's reliable and works!
* ector software - fix min/max int range for fixedpoint mathCarsten Haitzler (Rasterman)2017-07-291-2/+5
| | | | | | | | | | min value bitshifts are negative and thus not portable. just tax max (as its 1 more than min it if we do -max) as the limit as its within range. this should fix it @fix found by PVS studio
* evas render - remove obj null check in proxy clip as obj not nullCarsten Haitzler (Rasterman)2017-07-291-1/+1
| | | | | | obj is never null. so this check is pointless. not a bug though. found by PVS studio
* evas object textblock - fix append if cur node is nullCarsten Haitzler (Rasterman)2017-07-291-1/+1
| | | | | | | | then it will appeand totally as it should be. small buglet found. fixed found by PVS studio @fix
* evas gl gnenric engine - dont check w and h ptrs - will never be nullCarsten Haitzler (Rasterman)2017-07-291-4/+4
| | | | | | analyser unhappiness. found by PVS studio
* embryo_cc - remove pointless test for null byte and spaceCarsten Haitzler (Rasterman)2017-07-291-1/+1
| | | | | | space wins. the nul byte test is pointless. not a bug but pointless code found by PVS studio
* ecore-x - remove useless extra assignments of value to itselfCarsten Haitzler (Rasterman)2017-07-291-2/+0
| | | | | | usless code that does nothing - not a bug though found by PVS studio
* edje_cc - remove pointless free of null ptrCarsten Haitzler (Rasterman)2017-07-291-1/+0
| | | | | | not a bug but not useful code either freeing what is already null found by PVS studio
* efl ui text - remove useless else + freeCarsten Haitzler (Rasterman)2017-07-291-1/+0
| | | | | | | string is null so no need to free. not a bug but cleaner and analysers dont like it. found by PVS studio
* edje entry - remove extra else and free as sdtring is null thereCarsten Haitzler (Rasterman)2017-07-291-1/+0
| | | | | | | string is null so no need to free. not a bug but cleaner and analysers dont like it. found by PVS studio
* eina simple xml parser - remove pointless memcmp of 0 bytesCarsten Haitzler (Rasterman)2017-07-291-3/+1
| | | | | | | | no point - all we are doing is having a final fallback of any tag that starts with ! that isnt a special one like !DOCTYPE, !-- comment and ![CDATA stuff... analysers dont like these pointless calls. found by PVS studio
* els box - remove rudundant double checkCarsten Haitzler (Rasterman)2017-07-291-2/+2
| | | | | | not a bug but checking something twice in a row isn't too useful found by PVS studio
* elm code - remove unneded extra check from copy and pasta it seemsCarsten Haitzler (Rasterman)2017-07-291-4/+3
| | | | | | | | extra check already done in if above - no need. looks like copy & paste of the checks from above thus why it's there, but not needed. not a bug, byt analysers dont like it so remove found by PVS studio
* emotion gst1 module - remove extra unneded if'sCarsten Haitzler (Rasterman)2017-07-291-2/+2
| | | | | | not a bug, but don't need these found by PVS studio
* eeze_disk - remove redundant if check already doneCarsten Haitzler (Rasterman)2017-07-291-2/+2
| | | | | | | removes redundant fi check. not a bug but analysers dont like it and it is redundant found by PVS studio
* efl net - fix ipv6 getsockname to use the proper struct to do itCarsten Haitzler (Rasterman)2017-07-293-3/+3
| | | | | | | | | | | | | | | | | struct sockaddr_storage *addr; AS the actual storage ( a ptr to store the whole data). kind of... wrong. it should be: struct sockaddr_storage addr; see examples of this kind of usage of getsockname: http://www.masterraghu.com/subjects/np/introduction/unix_network_programming_v1.3/ch04lev1sec10.html found by PVS studio @fix
* eeze disk - remove uneeded extra check for test varCarsten Haitzler (Rasterman)2017-07-291-1/+1
| | | | | | | | test for non null already done above, so test must obviously be null here so no need to check. analysers dont like this but it's not a bug. make them happy thought found by PVS studio
* evas psd loader - remove unneeded extra check in ifCarsten Haitzler (Rasterman)2017-07-291-1/+1
| | | | | | | | one if condition is always true by virtual of previous if statements and drop-through so can remove. not actually any bug but analysers don't like it found by PVS studio