| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|\
| |
| |
| | |
ssh://git.enlightenment.org/core/efl into devs/woohyun/focus_tests
|
| |
| |
| |
| |
| | |
In this test, you can add any test case which is based on legacy
focus APIs with elementary widgets.
|
| | |
|
| |
| |
| |
| |
| | |
In this test, you can add any test case which is based on legacy
focus APIs with elementary widgets.
|
| | |
|
| |
| |
| |
| | |
better to keep the func local unless needed elsewhere too
|
| |
| |
| |
| | |
Signed-off-by: Jihoon Kim <jihoon48.kim@samsung.com>
|
| |
| |
| |
| |
| | |
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.
|
| |
| |
| |
| |
| | |
When I added the fallback block I left an if statement that could
never evaluate to true. Remove it.
|
|/
|
|
| |
Stop leaking the render node fd to child processes.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
| |
elm_win_socket_listen api only work for EFL_UI_WIN_SOCKET_IMAGE type window.
So it is better to add type check code.
|
|
|
|
| |
this fixes a whole bunch of warnings that just recently turned up...
|
|
|
|
|
|
|
|
| |
the data was never returned! fix fix fix.
found by PVS studio
@fix
|
|
|
|
| |
found by PVS studio
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
| |
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
|
|
|
|
| |
found by PVS studio
|
|
|
|
| |
found by PVS studio
|
|
|
|
| |
found by PVS studio
|
|
|
|
| |
found by PVS studio
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
| |
makes static analyisers happy... not a bug
found by PVS studio
|
|
|
|
|
|
|
| |
no need for key as no key is handled and no file is handled too. cleaner
code.
found by PVS studio
|
|
|
|
|
|
|
|
|
| |
2 abs uses should have been fabs as they compe a src float with
anot her float.
found by PVS studio
@fix
|
|
|
|
| |
found by PVS studio
|
|
|
|
|
|
|
| |
e85c92662bfec6a88bb7d2d091682daa429acba0 added this bug. before it was
always set to 259200 (first sunday after epoch).
found by PVS studio
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
| |
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.
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
| |
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!
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
| |
obj is never null. so this check is pointless. not a bug though.
found by PVS studio
|
|
|
|
|
|
|
|
| |
then it will appeand totally as it should be. small buglet found. fixed
found by PVS studio
@fix
|
|
|
|
|
|
| |
analyser unhappiness.
found by PVS studio
|
|
|
|
|
|
| |
space wins. the nul byte test is pointless. not a bug but pointless code
found by PVS studio
|
|
|
|
|
|
| |
usless code that does nothing - not a bug though
found by PVS studio
|
|
|
|
|
|
| |
not a bug but not useful code either freeing what is already null
found by PVS studio
|
|
|
|
|
|
|
| |
string is null so no need to free. not a bug but cleaner and analysers
dont like it.
found by PVS studio
|
|
|
|
|
|
|
| |
string is null so no need to free. not a bug but cleaner and analysers
dont like it.
found by PVS studio
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
| |
not a bug but checking something twice in a row isn't too useful
found by PVS studio
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
| |
not a bug, but don't need these
found by PVS studio
|
|
|
|
|
|
|
| |
removes redundant fi check. not a bug but analysers dont like it and
it is redundant
found by PVS studio
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
| |
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
|