ecore-wl2: Rename local function and make it static

As this function is not called from anywhere outside of
ecore_wl2_window.c file, this can be declared static.

NB: This patch also changes the function name to match the library
(ecore_wl2).

Signed-off-by: Chris Michael <cp.michael@samsung.com>

efl: terrible kludge so avoid termination crash on osx

Currently, elementary programs crash on termination on macOS (seems
Sierra-specific). This is very nasty, looks like deep memory corruption...
Without valgrind (or like) support on Sierra, it is difficult to
pinpoint the origin of the problem.

Due to the imminient release, and after discussion with @stefan, this
kludge will allow the release to happen.

This commit MUST be reverted just after the release, so we don't
blindfold ourselves!

Ref T5245

eina debug - remove other swap and comment on the weirdness of this

eina: failing to load a module is actually an ERR that user should see.

Eina_File: Avoid using munmap() on file->global_map.

When a virtualized file is created the file->global_map will not
point to a mmapped region, thus it's not safe to use munmap() during
the file cleanup. Only use munmap() if the file is backed by a FD.

Fixes: T5234.
Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

Efl_Copier: Add efl_wref_del() for pd->job.

In order to perform IO operations the copier will create futures using
efl_future_use(&pd->job, ...), which will set pd->job to NULL once the
future is destroyed. However this may lead to problems, because in some
cases the copier may be deleted at the _efl_io_copier_job() function,
which is the future's callback. Since the copier may be deleted before
the future, the area pointed by pd->job will have disappeared by the time the future
tries to set pd->job to NULL. To avoid this problem the copier must
explicily call efl_wref_del().

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

eina debug - new debug infra - comment out and make a note on bad endian

bad endian... code... see the comment in the src about why i think
this is bad as obviously the buffer pointed to is a 64bit type always
that is a pointer to something...

evas: Add safety code for smart object

Summary:
If user use the evas_object_smart_callback_add with no smart object,
it should be returned

@fix

Test Plan: self

Reviewers: jpeg, cedric, jypark

Differential Revision: https://phab.enlightenment.org/D5056

evas filters: Disable extra debug

Forgot to remove that! It's better to disable for the release,
DBG() and XDBG() are disabled anyway.

CID 1308611

evas filters: Shut up coverity

Add some safety checks.

CID 1374271

evas: Rename calculate property to calculating

Recently introduced by @cedric
I think the name is not very clear, and is clashing with the
actual method of the same name.

epp - fix buffer size for possibly bigger int snpritnfs

8 bytes wasn't enough for the full int range. use 12 and we're good.
removes warning for sure

@fix

efl_wl - fix warning for possible uninitted list access

could potentially be an issue in one theoretical case. less warnings
is better anyway

evas model loader - restructure code to silence warning

keep our build quiet warnings-wise and change the code to be so... so
we see real warnings. not a bug... just a warning.

elm_code: small optimisation for large files

eina hamster count - use a fixed version, revision etc. so builds repro

this should help with repdocucable builds by having our hamsters
constant for a build of a specific code revision.

@fix

efl build - fix lua old support on debian

didnt detect cflags properly. now it does.

@fix

ector - fix coveirty fix - vmin is -vmax not -vmin

efl_ui_win: add error hanling code for malloc fail

ecore_x: Avoid crash on NULL

This happens in E with software compositing, since E's commit
5702f0975e890f07cfb. E should be fixed shortly but segv is not
acceptable. Without segv E is still massively broken so it's
not like the bug would be hidden (large black areas in windows,
after switch vdesks with enough windows).

evas: add a way to know if we are already computing smart object calc.

Revert "evas: Always call show/hide intercept"

This reverts commit 936ea58cb9ac3e93aaabb6ec731fc3845cf95826.

this is an ideal change, but it breaks previously-expected behaviors
for interceptors and thus enlightenment is completely broken

ecore_x: init XEvents before passing to x

it turns out that xlib is going to copy the complete struct into
something internal. This might lead to the condition that a uninitalized
value might be part of the struct, and when later the struct is read
again the code might do wrong stuff since that value could be set now to
a randomly other meaningfull value.

This turned out on me in e as i could not write any letters like ßöäü,
since that lead to a not returning call to _XReply internal of xlib.
Dugging that showed that xlib was waiting on a reply of a call that
never got executed, and the XEvent it is waiting on just contians a
randomly correct value.

@fix

edje_svg: Fix handling of realloc

This amends 8e311db414950e399099acc1c0a as the logic
was badly broken. A "shadow" variable warning
clearly showed that "tmp" was not used properly.

This fixes the "IBM" logo in svg-test (the only one
using polygon/polyline, it seems).

Revert "elm gesture layer - use fabs not abs when actually comparing floats"

This reverts commit 2d4d836018387470b5df2c59119ab6cc81188e94.

Don't pretend to fix an issue found by a static analyzer by
introducing another issue that even clang warns about.

Fixes:
elm_gesture_layer.c:2533: warning: using floating point absolute value
  function 'fabs' when argument is of integer type [-Wabsolute-value]
        if ((fabs(st->info.mx) > ELM_GESTURE_MINIMUM_MOMENTUM) ||
             ^

elementary: access overrides Accessible.state_set.get

The elm_access a widget makes primitive object accessible. If a primitive
object is not visible, then the elm_access should NOT have state VISIBLE, and
SHOWING. From now the elm_access is checking visibility of primitive object.

eldbus - make file local func added in fix static

better to keep the func local unless needed elsewhere too

ecore_imf: check return value of memory allocation

Signed-off-by: Jihoon Kim <jihoon48.kim@samsung.com>

ecore_drm2: Stop making dumb fbs and mmapping for gbm fds

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 get

When I added the fallback block I left an if statement that could
never evaluate to true.  Remove it.

wayland-shm: Set CLOEXEC for dmabuf

Stop leaking the render node fd to child processes.

eldbus: check message serial before using

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 check

    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 shutdown

this fixes a whole bunch of warnings that just recently turned up...

elocation - fix return of status, address, position to actually ret data

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 ret

found by PVS studio

elm code - handle alloc failure for line split

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 split

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 absraction

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 check

found by PVS studio

elm theme - remove redundent check

found by PVS studio

elm config tool - fix redundent check

found by PVS studio

evas textblock - remove redundant check

found by PVS studio

epp - cpp - fix buffer end/null checks given static buffer paths

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 char

makes static analyisers happy... not a bug

found by PVS studio

evas cache2 - remove excess file and key checks - only check file

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 floats

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 keys

found by PVS studio

elm calendar - fix gmtime call to use an actual value

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's

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 props

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 set

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 work

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 math

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 null

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 null

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 null

analyser unhappiness.

found by PVS studio

embryo_cc - remove pointless test for null byte and space

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 itself

usless code that does nothing - not a bug though

found by PVS studio

edje_cc - remove pointless free of null ptr

not a bug but not useful code either freeing what is already null

found by PVS studio

efl ui text - remove useless else + free

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 there

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 bytes

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 check

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 seems

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's

not a bug, but don't need these

found by PVS studio

eeze_disk - remove redundant if check already done

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 it

      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 var

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 if

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

eina debug - if calloc for session struct failed - return NULL

makes analysers happy but really doesnt change anything - moves the
abort/crash/exit somewhere else.

found by PVS studio

edje_cc - make list free simpler to make analysers happy

analysers like PVS studio dont know the eina_list_free returning NULL
design pattern for consistency and so dislike the var being set twice
in a row. this will make them happy without any skin off our backs.
not a bug at all.

found by PVS studio

evas git loader - move findo reset inside else as only that changes it

only the else changes finfo so reset inside there. not really any bug
at all byt style-wise a bit better and analysers don't like it

found by PVS studio

elm list - remove harmless extra null set to keep analysers happy

this was totally harmless and not an actual bug but it keeps analysers
happy.

found by PVS studio

elm dayselector - remove double set of content var

this looks like a leftover from legacy to eo porting. it was harmless
but analysis hates it

found by PVS studio

elm code - remove double set of textlen

analysis doesnt like it, but not a bug

found by PVS studio

edje edit - remove pointless setting attr_mount to 0

static analysers dont like it bug not a bug at all

found by PVS studio

evas pmap loader - remove pointless extra assign of end of string to 0

not an actual bug - just analysis impacting

found by PVS studio

eldbus parser - remove duplicate tag handler thats already handled

confusing and dead code as it's already handled. not an actual bug but
analysis impacting

found by PVS studio

evas events - remove set of list to null then back to value - confusing

eina_list_free always returns NULL. it's meant as a design pattern to
follow like the append/prepend that ensures a freed list is NULL, but
it can be confusing to analysers so remove the assing. not a bug
actually.

found by PVS studio

eina evlog debugging - when freeing debug buf with mmap 0 size after

munmap would not munmap the memory with  a size of 0... so zero the
size after the munmap.

found by PVS studio

@fix

evas obj smart - remove pointless if

s is already checked for null... so no need to check again

found by PVS studio

@fix

edje edit - when adding size class set max width and height to -1

since we're not using size class yet.. we hvent had to debug this...
but it is a bug.

fond by PVS studio

@fix

efl net ssl constructor - remove double set of same var

all struct vars are set... one 2x. not a bug but pointless code in the
src

found by PVS studio

evas bmp loader - fix warnings about same type handled twice

so a type we handle earlir inan if we re-handle as invalid later. this
wouldnt lead to a crash or bugs as the if's would ned to be evaluated
in order normally, but it's good to get it right.

found by PVS studio

etc to rgba conversion - dont read stack garbage into the est imgs

surprising this wasn't caught when testing fallbacks. etc2 is used
hyper-rarely but still... basically it'd skip pixels and read stack
junk (which really would not crash but we'd have junk in image data).
or should.

found by PVS studio

@fix

ephysics - fix stacking comparsion correctly if 1 smaller than 1

this fixes a pretty trivial comparison issue..

found by PVS studio

@fix

evas legacy event mask - cast to unit64_t before bitshift

this fixes a bit wraparound in the shift as the 1 is an int (32bit)
type that then gets shifted .. then after that cast to 64bit.

found by PVS studio

@fix

eina inarray accessor - use right type in parameter

this actually wasn't a bug that would cause a crash. cloning an array
access would fail as the magic check would find its an accessor not an
array. indeed a bug... but we never used this anywhere i can find.

this was cast to the correct func ptr callabck in the accessor struct
as the clone method though.. thus everyhting was happy with it
seemingly.

found by PVS studio

@fix

elput: Fix synthetic relative motion events based on abs events

libinput won't actually give us this info, and logs an error instead.
We can't synthesize based on pointer position due to pointer
warping.

So we must track abs reports and use previous abs reports to
synthesize rel events.

ecore_evas_drm: Calculate gpu blank time offset once at startup

So vmware's graphics driver reports the MONOTONIC drm cap, yet uses
CLOCK_REALTIME instead.  This leaves us with a gigantic offset between the
gpu timestamp and the times ecore_time_get() gets from CLOCK_MONOTONIC.

Since ticking screws directly with loop time this results in some long
distance clock jumping.

This commit fixes drm/gl_drm operation under vmware.

ecore_drm2: Add a fallback method for vblank waiting

We can't depend on vblank waits being implemented by the driver, but we
can count on page flips functioning, so add a fallback that does a page
flip and waits for it.

ecore-wl2: implement support for aux hints

this is a direct copy of a feature from tizen git with no modifications other
than what was required for compilation and functionality

https://review.tizen.org/git/?p=platform/upstream/efl.git;a=commitdiff_plain;h=01e72b7e3484ece4b589f95315990ba2c366c231
https://review.tizen.org/git/?p=platform/upstream/efl.git;a=commitdiff;h=670d84b579f248ae0e3df48e9953fe8128da9468

fix T5780

@feature

Revert "evas: Fix use of invalid EO object"

This reverts commit 9b1e785f8ebba618016dd217aa18835b4fb675b7.

this breaks devices with vt switching under drm

elput: fix seat cleanup

ensure seat deletion does not conflict with elput shutdown

@fix

elput: unref devices in post event cb

@fix

elm_table: do not propagate max size

evas table does not set max size, propagating this triggers unnecessary
callbacks

@fix

elm_box: use correct aspect ratio for HORIZONTAL layout

ratio should be flipped in this case

@fix

elm_box: use homogeneous layout if only one child exists

this should be a small speedup for a corner case of box use

elm_win: do not implement aspect_get()

this breaks aspect hints.

@fix