layout: Remove method theme_enable

This was an internal thing, ugly, replaced by another ugly
solution, but maybe less ugly.

Ref T5315

scroller: Implement default content part

This:
  efl_content_set(efl_part(scroller, "default"), obj)
worked fine, but, this:
  efl_content_set(scroller, obj)
didn't work as expected.

Thanks @JackDanielz for the report.

Note: There is a problem still... "default" should not work
      with efl_part. This is quite bad, actually. It should
      probably be "content" instead.

layout: Implement freeze and thaw from interface

This implements part of the interface
  Efl.Canvas.Layout_Calc

layout: Implement cursor part APIs with efl_part

This makes layout parts implement Efl.Ui.Cursor.
This also adds the missing bool returns from that interface.

This removes 7 APIs from Elm.Layout.

Ref T5315

cursor: EO-ify elm_cursor API

This creates an interface and makes Elm.Widget implement it.
Next step: make elm_layout parts implement it.

Ref T5315

layout: Move edje_object_can_access to legacy

This is an API enabling accessibility on text(block) parts
in a layout. But it is said to have many issues. I can already
see that it only changes a flag but doesn't trigger any code
to create the appropriate objects, so definitely not fully
working.

According to @kimcinoo this may remain in legacy land for now.

edje: Split off calc APIs to an interface (EO)

Those APIs can then be used by Elm.Layout, hopefully
simplifying the API.

I wonder if the APIs should be prefixed "calc_" (as is)
or "layout_calc_". The extra "layout_" prefix would make
it common with other layout APIs (eg. signals, data,
size min/max, ...).

Ref T5315

elm: Make content and text aliases internal only

This removes the APIs from the EO API.

Ref T5315

win: implement efl_provider_find for EFL_UI_WIN_CLASS

Summary:
Implement efl_provider_find function for efl_ui_win class.
This will support to search window class by efl_provider_find function.

Reviewers: jpeg, cedric, Jaehyun_Cho, thiepha, woohyun, Blackmole

Reviewed By: jpeg, cedric

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

frame: Implement text set/get (EO only)

This is required for the default style which has a title.

evas: More debug_name information

evas: Add debug_name to image objects

eina_slstr: Fix leak in an error case

This function takes ownership of the incoming strbuf

eo debug name: Use strbuf instead of slstr

This should be a little bit more efficient, even if more "risky"
in terms of leaks. The final API still looks exactly the same.

evas/edje/elm: Override debug_name in core classes

This is really only a demonstration of what kind of information
we can print with efl_debug_name_get(). Hopefully this can help
debugging with printf/ERR logs and even help with live debugging
inside gdb.

This shouldn't be used for other purposes than debugging, as the
exact string format is not defined.

@feature

eo: Add debug_name read-only property to ease debugging

This will include the following information, by default:
 - class name
 - whether the class is an override
 - eo id (pointer)
 - refcount
 - name if one was set (Efl.Object property)

This also supports classes, which is why it's an EAPI in eo.c
and not only a method of Efl.Object

This can be overriden by subclasses using the empty method
Efl.Object.debug_name_override.get

If the function is overriden, then the returned string is used
as is and so it is left to the subclass to include all the
necessary information (as above). This can easily be achieved
by calling efl_debug_name_get(efl_super()) and then concatenating
the strings.

Think of this function as something like Java's toString(), but
only for debugging (i.e. a string class should not just return
its string value).

@feature

Eo: Allow multiple functions overrides

This allows two things:
 - adding new override functions on an object that already has
   overrides
 - resetting a specific function (or list of functions) to the
   parent class implementation by passing NULL as implementation

Fixes T5580

@feature

elm_test: Remove calls to visible_set for EO objects

evas: Make all EO canvas objects visible by default

All legacy objects remain invisible by default. Any call to
visible_set() will prevent the automatic show() to happen.

show() will be done just before render time, which may be a
bit too late in order to propagate the necessary changes.

This may break some things where some objects are created
internally using efl_add() instead of the legacy API, and
the intent was not to show the object.

@feature

efl src - mark fallthrough switch statements in 3rd party code

in code we importend that doesnt use eina we have warnings of
fallthroughs. all o them are commented to be fallthrough so add the
attribute there too to have fewer warnings.

configure: switch to dev mode again

Merge window for 1.21 is now open.

release: Update NEWS and bump version for 1.20.0 release

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