fix warnings in sanhyeons's patch/fix

[CONFIG] call elm_rescale() only when scale value is changed

Summary: it doesn't need to call elm_rescale() in _config_flush_get() when the scale value is not changed.

Reviewers: jaehwan, cedric, raster

Subscribers: minkyu

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

genlist: fix memory-leak in filter queue and refactoring queue iteration

Summary:
There are critical memory-leak in filter queue process.
         It looks typo so I fix it and refactoring queue iteration logic to spend
 less time in for and while loop.

Test Plan: Tested by filter elementary test

Reviewers: shashank0990

Reviewed By: shashank0990

Subscribers: raster, cedric, jpeg

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

Fix memory leak issue on genlist_example_02

Summary: The realized items list should be freed by either eina_list_free() or EINA_LIST_FREE when it is no longer needed

Reviewers: cedric, jpeg

Reviewed By: jpeg

Subscribers: minkyu

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

evas generic gst loader - handle fwrite return value for coverity

fixes CID 1360467

evas canvas destruction - detect zombie objs and hack at them with axes

ok. so here's the issue at least now. we have eo objects in the canvas
and they have a refcount of 2 user_refcount is 0. the calls stack does
NOT show we are calling callbacks at that time on these objects. they
are not in the backtrace (the canvas is, the objects themselves are
not).

SOMETHING is keeping 2 eo "internal" refs on these objects and i have
no idea what/how/who. it's a royal pain in the butt to find out as the
only way is lots and lots of logging and you get drowned in the
logging...

so what I have now done is a super ugly workaround that detects these
zombie objects that refuse to die and just FORCES them to die when the
evas canvas frees and clears out layers.
ac10a00acc6bacf01bfd208cc78b4eb2a6a925d8 doesn't really cause the
issue, it just brings it out in the open for all to see far more
easily. but something is deeply wrong SOMEWHERE with SOME objects and
our refcounts.

this fixes T4187

docs: support for writing edge styles for individual connections

docs: initial bits for theming support for graphs

tests: Add missing function implementations

In eolian test cases.
Not a problem when EOAPI is defined as weak, but compilation
will fail if the symbols are strong. Eg. on Solaris (it's just
a guess).

Fixes T3889

entry: allow force recalc on "size,eval" "elm" signal

Summary:
In some use cases, font (style) of elm.text in elm entry is changed
on focus and unfocus. user must send "size,eval",elm signal to entry to
force recalc, but this was not happening. This patch fixes that issue.

fixes T4233

Reviewers: tasn, herdsman, raster

Subscribers: cedric, seoz, jpeg

Maniphest Tasks: T4233

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

win: Fix evas framespace under X

Mouse inputs would not work if the window is rotated,
because the framespace was invalid.

Fixes T4132

edje entry - check all minor struct callocs and on fail report + unwind

lots of unchecked malloc/calloc returns, so check these, unwind safely
and complain (highly unlikely to happen unless the system is about to
fall over). this should fix T4230

elementary: fix legacy path set/get for Fileselector specializations

Fix legacy functions elm_fileselector_path_get and
elm_fileselector_path_set.
Make these functions call the correct function when called with a
specialization of Elm.Fileselector (instead of the base class function).

Create internal versions of path set/get functions to avoid warnings
about deprecated function calls.

Fix T4198

@fix

ecore-drm2: Silence compiler warning

We probably can't actually use this variable uninitialized, but
let's init it anyway to keep gcc happy.

ecore-drm2: Fix issue of VT-Switching going blank on returns

As it turns out, when we vt-switch Back to a running Enlightenment,
it ends up processing all outputs and causes Evas to create a giant
canvas to cover all outputs. This new canvas causes a blank screen as
the pageflipping fails (due to fb being too large). As we do not yet
support output on more an one screen anyway, temporarily disable any
output which is not marked as primary.

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

ecore-drm2: Add API for vt-switching

This commit adds a function we can call from inside Enlightenment in
order to fix an issue with VT-Switching. The problem was when a client
application is open, the compositor does not pass along the key-combo
for vt-switch but rather sends it to an application. This patch (along
with upcoming Enlightenment one) fixes the issue.

NB: This late API add approved by Stefan ;)

@feature

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

elementary: Reduce variable usage

As we don't use the framespace x, y here then there is really no need
to have them.

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

elementary: Fix issue of wrong minimum size for some E dialogs

When we are calculating minimum size hints for the window object, we
should be adding framespace to maximum size, not subtracting it...

Honestly this just looks like a brain-dead typo.

Fixes T44225

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

edje_edit: avoid such values like 2.77556e-17

using spinner to change double values means that sometimes
if set such weird values while should go to 0.0

@fix

elementary: Avoid possibly setting opaque region twice

Previously, if the window had no frame object, then this could would
have been setting opaque region twice (for the alpha case). We can
avoid this if we move the alpha check to below the frame check.

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

elm: Fix extra mouse,in events in layout_content_unset

Problem: crash in assert() in terminology.

Scenario:
 Open Terminology,
 Split V by keyboard,
 Move mouse to split 2,
 Create tab by keyboard
 --> abort() in terminology

Cause:
 An extra mouse,in event happens during edje_object_unswallow
 inside elm_layout_content_unset.

Root cause:
 efl_part() in elm_layout had a side effect: edje_recalc on the
 edje object. Causing its geometry to be "properly" defined and
 the mouse event to trigger callbacks.

Solution:
 Avoid calling recalc... somehow.

Conclusion:
 Without adding any new API, edje edit provides internally the
 information that we want: type of an edje part (for box & table).

Fixes T4221
See T4028
See T3509

efl - ecore-x - work around e's badness in makeing an x image for all wins

so e is being stupid and creating an ecore-x image forevery single
window if in x11 mode if it needs it or not. this results in having ti
allocate an actual x image and shm segments. work around this and get
bit order from somewhere else than the x image itself thus avoiding
the allocation until a real get or put is done.

@fix

elput: Use correct filename in unlink

We switched from mkstemp to eina_file_mkstemp() but ended up unlinking
the template filename, which didn't exist on the filesystem.

wayland_shm: Use correct filename in unlink

We switched from mkstemp to eina_file_mkstemp() but ended up unlinking
the template filename, which didn't exist on the filesystem.

ecore-wl2: Add safety check for valid compositor

As we cannot create a new wl_region without a valid compositor, add a
safety check.

@fix

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

release: Update NEWS and bump version for 1.18.0-beta1 release

ecore-wl2: Remove error message during connect/reconnect

Since session recovery can cause apps to constantly try and reconnect
to the display server, we probably should not be filling logs here. On
the other hand, any failures on first connect are now not going to
display any error messages.

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

edje_pick: copy block 'data'

Copy global data from files. Main resoan for it - elementary.
Elementary needs data "version" for load widget styles.

@fix

edje: remove now unused variables

Commit e3ed888aa994d96007da15be89649a92c12bcbae removed the code where these
variables have been used before.

examples: edje: make sure we handle the various corner cases for the po files

Delete the earlier copied files only. Handles out of tree and in tree builds.

edje: Remove invalid call to recalc in efl_part

What a mess... Assuming efl_part() had no side effect on the
object it took me hours to figure out that there was a wrong
call to _edje_recalc_do in the efl_part() function itself.
That was bad, and existed because efl_part() used to be
efl_content_get().

efl_part() should not have any side effect.

Also, fix a return value in content_remove.

Fixes T4214 (invalid redraw and crash in terminology).

elm cnp windows - remove unused vars

elm cnp win32 - do one side at least of unix newlines to windows newln

this PARTLY addresses T3556 ... this handles our own conversion of
makrup to string then to a windows newlined string. this half. i wrote
the conversion code for the other way but it's unused as i am unsure
where exactly to plug in it. following the cnp code makes me not sure
where it goes so for now - not there, but ready to go.

to be clear. out API is unix text. utf8 strings at our api with UNIX
newlines. that is our api. that is the text we accept and produce. if
you deal with another file or interface that does not provide this
then the job of conversion is AT THAT POINT. eg elm_cnp.c has to do
this. as would file loads of text files (and saves) etc. - anything
else like forgivingly handling anything at the api level makes it
totally unclear what our api is and what should go in and come out.

to be portable we have to define what it is and the most portable
thing to do is at the api level within a process we define one and
only one format. UNIX \n format.

@fix - partial

efl ui image - fix view size get to NOT apply scaling

this fixes T3254

edje: fixed issue with applying transformation and stroke width

Reviewers: cedric, jpeg

Reviewed By: jpeg

Subscribers: cedric, jpeg

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

edje examples - when you cp also just assume success if same file

fix makefiles for examples.... more

@fix

example/edje: updated vg example with interpolation usecase

Reviewers: cedric, jpeg

Reviewed By: jpeg

Subscribers: cedric, jpeg

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

edje: fixed issue with vector part state inheritance.

Reviewers: cedric, jpeg

Reviewed By: jpeg

Subscribers: cedric, jpeg

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

example/evas: update evas vg example.

Reviewers: cedric, jpeg

Reviewed By: jpeg

Subscribers: cedric, jpeg

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

efl/interface: fixed the shape interpolation implementation.

Reviewers: cedric, jpeg

Reviewed By: jpeg

Subscribers: cedric, jpeg

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

edje: Minor fix (return proper bool value)

edje: Use eo_del instead of eo_unref where needed

This fixes the following error message:
Object '0x40000002f0000030' still has a parent at the time of destruction.

Though I actually wonder why ref == 0 and there was a parent.
Something is still a bit fishy here.

edje examples - stop deleting the po files from the src tree!

whatever someones was trying to do.. they just endewd up deleting po
files in the src tree every time you clean! argh! fix!

evas software x11 - be agressive with shm cache and keep it low

shm cache size is now 1.5x the total number of pixels in sw x11
outbutf's and that's it. 32 segment max limit too. this should keep
things down

@opt

evas and ecore_x shm segment management - fix over allocation and perms

so our sysv shm segments were both over-permissive (nothing bad
really, just other users could read and write to/from our pixel data
destined for the screen... they could do this to x11 directly anyway
so no real issue), but be more restrictive and use 0600 as xserver
runs as root so can read/write anyway and we only want our own uid
access. but even more - fix our shm segment flushing to not keep lots
of segments floating about like a bad smell when we don't need them.
we had a cache but it wasnt flushed when it should be since async
rendering turned up. this fixes that and we're back to agressively
flushing them out when idle.

@fix

Revert "access: add an example of elm_object_part_access_object_get"

This reverts commit f8aa67cd18e0f02368ca3aaedc159872feed459d.

access: add an example of elm_object_part_access_object_get

elm_bg: fix typo in doc

Summary: Signed-off-by: Minkyu Kang <mk7.kang@samsung.com>

Reviewers: cedric, jpeg

Reviewed By: jpeg

Subscribers: seoz, jpeg

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

elm_conform: set data pointer to obj from NULL.

Summary:
 - In the callback for ELM_PLUG_EVENT_IMAGE_DELETED,
   it tries to get elm_conform's data from NULL.
@fix

Reviewers: Hermet, woohyun, cedric, jpeg

Reviewed By: jpeg

Subscribers: cedric, jpeg

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

evas: Fix inefficient updates with no_render

is_visible returns 0 if no_render is true, so should
was_visible as well. Yeah, there will be problems if
no_render changes on the fly. Don't do that.

Fixes T4193

wayland_shm: Fix fd leak in dmabuf

Need to close the fd from PrimeHandleToFD.

ecore-wl2: Reduce debug noise on key events

This is annoying to see key events flooding a terminal ... so comment
out this debug statement. If a developer really needs this, then
enable it locally.

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

Revert "access: add API to get access object of given part"

This reverts commit 57d77c9041086ebbf84c2a30d93e95d47fc12705.

We are in freeze right now and this is a new feature. Considering the access
API is still in beta I might have let this slip by but its does not even build

../../../src/tests/elementary/elm_test_popup.c:27:1: error: conflicting types for ‘elm_object_part_access_object_get’
 START_TEST (elm_object_part_access_object_get)
 ^
In file included from ../../../src/lib/elementary/Elementary.h:240:0,
                 from ../../../src/tests/elementary/elm_test_popup.c:6:
../../../src/lib/elementary/elm_object.h:521:54: note: previous declaration of ‘elm_object_part_access_object_get’ was here
 EAPI Evas_Object *elm_object_part_access_object_get(const Evas_Object *obj, const char *part);
                                                      ^
../../../src/tests/elementary/elm_test_popup.c: In function ‘elm_object_part_access_object_get’:
../../../src/tests/elementary/elm_test_popup.c:39:47: warning: passing argument 1 of ‘elm_object_part_access_object_get’ makes integer from pointer without a cast [-Wint-conversion]
    access = elm_object_part_access_object_get(popup, "access.title");
                                               ^
../../../src/tests/elementary/elm_test_popup.c:27:1: note: expected ‘int’ but argument is of type ‘Evas_Object * {aka struct _Eo_Opaque *}’
 START_TEST (elm_object_part_access_object_get)
 ^
../../../src/tests/elementary/elm_test_popup.c:39:13: error: too many arguments to function ‘elm_object_part_access_object_get’
    access = elm_object_part_access_object_get(popup, "access.title");
             ^
../../../src/tests/elementary/elm_test_popup.c:27:1: note: declared here
 START_TEST (elm_object_part_access_object_get)
 ^
../../../src/tests/elementary/elm_test_popup.c:39:11: error: void value not ignored as it ought to be
    access = elm_object_part_access_object_get(popup, "access.title");
           ^
Makefile:43996: recipe for target 'tests/elementary/tests_elementary_elm_suite-elm_test_popup.o' failed
make[5]: *** [tests/elementary/tests_elementary_elm_suite-elm_test_popup.o] Error 1

If you still want this in make sure it actually can compile the tests you added.

build: enable examples build during distcheck

No example should be left behind. Make sure we run the build for them during
distcheck so we can be sure we did not introduce any build problems here.

examples: elementary: make sure the prefs_example* files get into dist

These files have been missing from dist and thus some example builds failed.
We also need to clean them up afterwards now.

examples: edje: fix build of edje files with translation within distcheck

The edje text example covers the translation functionality which need the mo
and po files during edje_cc compilation. Further more the folder need to be
writable to create the new mo/po files.

We need to ensure that this is the case during a distcheck which is read only
on the source tree.

evas: filters: add return value in all cases

Function with non-void return value so we need to add a return in all cases.
Thanks to gcc for complaining here.

elementary: Fix formatting

NB: No functional changes

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

ecore_wl2: also add the del control code

otherwise we are sending the controlcode for delete, which is wrong.

This fixes the backspace on delete problem.

elput: Fix issue of losing mouse input when removing a device

When we are adding new input devices, if we already have an existing
one (keyboard, pointer, touch, etc) then we should just be
incrementing the device count and returning from the init function.
Previous code here would increment the device count, then continue on
and create a new internal representation of the device. This was
causing issues when a device gets removed.

Fixes T4192

@fix

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

access: add API to get access object of given part

elm table and box resize themselves on sizing eval - fix this - dont resize

thsi fixes T3031

@fix

elm_access: fix typo of atspi action

elm button theme - fix anchor style to not pass events when disabled

@fix

elm button default theme - fix disabled events to not pass through

this fixes T2955
@fix

efl - when evas generic loaders merged the gst loader was missed. fix

this fixes the missing gst loader from generic loaders.

elm default config - change default select on focus to be disabled

this also fixes T3198 but only dave's comment.

@fix

ector module - remove sueless chekc for null done already CID 1347411

fixes a pointeless check coverity found

ecore-imf - remove usueless checks for context - CID 1351451

useless checks for ic in code. fix

efl ui image async locking -fix CID 1356199 - locked data cant be messed with

this addresses a coverity issue that in theory looks right

fix CID 1356199

@fix

efl ui video - fix coverity CID 1356997

really minor lack of check if emotion init succeeds. check and
complain if it fails.

@fix

edje entry - fix coverity CID 1357752

fix potential crash if calloc fails and sel is NULL

@fix

evas cache2 - fix coverity CID 1357754

@fix

elm quicklaunch binary - fix coverity CID 1353607

@fix

evas image updates - optimize by over-rendering less with hidden content

so if an img obj had some update regions and was opaque, it didnt del
an update region first before adding the updates, thus possibly
letting update regions pass through the opaque image area

@fix

Gstreamer1: implement subtitles mute/unmute ability

This seems the only way to disable subtitles embedded in the video file.

WIthout this patch calling em_spu_mute_set/get do not work, thus I consider this a @fix

release: Update NEWS and bump version for 1.18.0-alpha2 release

wayland_shm: Speed up dmabuf on intel

using map_bo/unmap_bo instead of gem_map_bo_gtt/gem_unmap_bo_gtt
results in a cacheable mapping and a large performance boost.

(dmabuf will still remain turned off by default for the release)

docs: split type serializers and keyword ref into their own module in docgen

elementary: change cursor types for cocoa

Map more cursor types for cocoa.
Fix T3352

ecore_wl2: be compatible with x input

This commit introduces a new state per input, this state is just for the
keymap and no modifieres, the state is used to get the keysym from a
keycode. This is done to stay compatible with x since keyname in efl is
the characters which are describing the key which is pressed, without
the transformation of shift etc.

This also escapes keycodes from beeing a utf8 character, and passes out
a keyname instead of the utf8 symbol.

fix T3976
fix T4184

edje: fix specular color

Summary: We use int from 0 to 255 for color, not float

Reviewers: raster, Hermet, cedric

Reviewed By: cedric

Subscribers: jpeg, artem.popov

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

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

elementary: disable visual hint at odd/even.

This is a work around a huge bug in how genlist handle index. Fixing
it in genlist for this release is way to risky. The default theme is
temporary updated to not use this feature and this patch should be
reverted as soon as we start working on 1.19.

edje: allow inheritance from an aliased group.

elm_win: move window contents back to 0,0 when deleting frame object

fix T4092

elm_win: simplify borderless_set when reapplying the same state

elementary: Remove logically dead code

In all code paths leading to this goto statement being called,
'iter_array' variable will already be NULL, so checking it's validity
is just logically dead code.

Fixes Coverity CID1357392

@fix

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

elput: Use proper variable type

The 'getenv' function returns char * ... not a const char *, so adjust
variable type to match

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

eina: Fix null pointer derefernce

Coverity reports that eina_safepointer_get returns a NULL promise here
(checked 20 out of 21 times). As eina_safepointer_get can return NULL,
we should check the validity of 'promise' here before trying to
derefernce it later.

Fixes Coverity CID1356625

@fix

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

elementary: Remove logically dead code

Coverity reports that this is logically dead code. As we check the
validity of 'model' above this function, then testing 'model' for
validity again in this expression is just logically dead.

Fixes Coverity CID1356617

@fix

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

elementary: Remove logically dead code

Coverity reports that this is logically dead code. As eo_item has been
dereferenced on all paths leading here, and it is also set above, then
it cannot be NULL at this point, thus this check is logically dead.

Fixes Covierty CID1355585

@fix

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

eldbus: Check return value and error out if necessary

Coverity reports that we do not check the return value of
_eldbus_model_proxy_load here. We likely should be checking that in
case the model proxy fails to load so we can return proper values here.

Fixes Coverity CID1355233

@fix

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

edje: Fix unchecked return value

Coverity reports that we call _circular_dependency_find here without
checking the return value

Fixes Coverity CID1349867

@fix

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

evas: Fix missing breaks in switch

Coverity reports that we are missing breaks in the switch blocks here,
so add missing breaks.

Fixes Coverity CID1347413 and CID1347414

@fix

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

edje_pick: reset alias flag for copied group

edje_pick make copy of whole group, even if group is alias. So if
group was alias need to reset this flag.

@fix

eo: Fix override test case to match new policy

The new policy is a lot more restrictive.
Honestly, I don't like it.

eo: Prevent eo_override to be called on an already overridden object

I don't see the point of it but can't be bothered to argue.
For Tom.

wayland_shm: Add missing headers to wayland_shm sources

Some headers weren't being placed in the release tarball.

Revert "elementary: Make wl frame borders above content"

This reverts commit 46904e19a71dbeecbcdb6950c0f372c34f6d0db6.

this breaks stacking entirely and prevents a large number of widgets from
ever becoming visible

docs: split off writer and utils into modules in docgen