ecore pub headers - use __sun not __sun__ for ifdef for solaris

Eina_Value: strdup() is used, so include string.h

Ecore: siginfo_t type is defined in sys/siginfo.h on Solaris

gengrid: fix the crash in _elm_gengrid_item_edge_check

_elm_gengrid_item_edge_check can have eo_it as NULL if none of widget item
is focused. This could happen if item_focus is not enabled on items.

test case: elm test -> gengrid 2 (enable only "focus hightligt set") and move focus.

@fix

Signed-off-by: Amitesh Singh <amitesh.sh@samsung.com>

Gengrid: Bring the item into view scope before swapping.

Summary:
The gengrid item should be into view scope while reordering the items.

@fix
Signed-off-by: Umesh Tanwar <umesh.tanwar@samsung.com>
Test Plan: elmementary_test -> gengrid 2 -> horizontal mode + reordering

Reviewers: singh.amitesh, cedric

Subscribers: cedric, atulfokk, jpeg

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

Signed-off-by: Amitesh Singh <amitesh.sh@samsung.com>

Ecore_Con: Fix compilation on Solaris

FIONREAD is defined in sys/filio.h

elm_code: fix CID 1368489

@fix

efreet: Don't prefer the second perfect over the first in desktop_exec_find()

We test a second match for a perfect match, then stop all further processing,
but we never test the first match. This leads to a situation where a system
wide .desktop file is given precedence over a local override.

Instead, check the first match too.

 #test-e

ecore-evas: better handling for pointer_warp with buffer canvas

if buffer canvas is not image object, this needs to emit a move event
to be consistent with other engines

probably this should emit events in all cases, but adding for image buffers
this close to release seems potentially risky so I'll leave that for later

ref 4a691f79df88d4b26c5af48ffb811e28f031e2f1

evas_events: look for seats on hash instead of input devices

Summary:
Otherwise it won't ever find the modifier masks.

Fixes T5146

@fix

Reviewers: iscaro, Jaehyun_Cho

Subscribers: cedric, jpeg

Maniphest Tasks: T5146

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

ecore-drm2: Remove unused field in Output structure

The 'planes' field in Ecore_Drm2_Output structure is unused and can
thus be removed here.

@fix

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

eet cmdline tool - dont give garbage certificate length if there is none

ecore audio: Fix loss of last stream

Summary:
pa_stream's write callback requires to size of stream data
using 'len' parameter.
This size depend on pulse audio's internal status and not
consistent.

When a efl read audio's last stream, length of read('bread')
is less than write callback's 'len' parameter.
If the gap between 'len' and 'bread' is small,
last stream is played fortunately.
Otherwise, the last stream is discarded.
(It is doubtful about pa_stream's pre-buffering.)

To prevent it, push silent stream which is amount
of deficient length.

@fix T5281

Reviewers: raster

Reviewed By: raster

Subscribers: cedric, jpeg, woohyun

Maniphest Tasks: T5281

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

ecore_evas: delete exit idler when done with it

Eina_Xattr: fix memory corruption

Summary:
There should be reallocation +1 (for last '\0') and also
checking >0, not !=0, because of getxattr can return -1 in case of error
@fix

Reviewers: cedric, raster, NikaWhite, jpeg

Reviewed By: NikaWhite

Subscribers: myoungwoon

Tags: #efl

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

ecore_evas extn - fix buffer n check for lock files with untrusted val

the code added by minkyoung has a definite security flaw here trusting
e->response to be within a small range when all it is is an int -
range is not limited other than that... so fix the code to check for
range like further code below does.

this commit went in 2 days ago... so not an existing bug fix.

evas: Fix a rare issue with GL and map and mask

The things you learn to love...

The situation was:
- An object is mapped (naviframe in an animation)
- One of its children has a mask
- The window is rotated by 90 or 270 degrees (landscape)

The mask glsl code to invert the x,y coordinate depends on the
screen rotation and this somehow was wrong.

Tested on Tizen and in elm_test "Masking", made with @jiin.moon.

@fix

elm_code: fix keyboard selection forward

@fix

elm_map: Fix work of elm_map_overlay_icon_set if icon is NULL.

Summary:
Delete check on NULL for icon object due to incorrect work. Function must delete
icon in map overlay (according to documentation) if we set NULL, but with this check -
nothing happening and after elm_map_overlay_icon_get we have not NULL returned value.
@fix

Reviewers: cedric, Hermet, raster, jpeg

Reviewed By: cedric

Subscribers: artem.popov

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

ecore: proper macro usage for double comparision. Summary: The comparisions are done between doubles.  EINA_FLT_EQ -> EINA_DBL_EQ

Summary: Signed-off-by: Umesh Tanwar <umesh.tanwar@samsung.com>

Reviewers: singh.amitesh, cedric

Subscribers: jpeg, atulfokk

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

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

ecore_evas_extn: do not update plug image when lockfile is invalid.

Summary:
This prevent invalid shared memory access.
Invalid access occur when server is resized sequentially from now to A-size
to B-size, and client receive A resize message after resizing B.
Then client try to render plug image with A-size, but shared memory is B-size
buffer. Size are mismatch. This makes segmentation fault when uploading texture(gl)
or rendering image(sw).

Test Plan: Indicator rendering on Tizen3.0 platform.

Reviewers: jypark, wonsik, dkdk, scholb.kim, jiin.moon, jpeg, cedric

Reviewed By: cedric

Subscribers: cedric, jpeg

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

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

genlist: Fix another issue with insert sorted

Fixes T5274

@fix

evas gl: Fix a compilation warning

The fallback lib handle is not required unless EGL/GLES is used.

eina_debug: do not leak a spinlock when the path is not given

gl_drm: Fix KHR partial update support

I fixed this in other engines but missed this one.  The result is some
terrible flickering on gl_drm on recent mali drivers.

ecore_drm2: Move function call after the NULL check

Was dereferencing ob before NULL checking it.  What could possibly go
wrong?

gl_drm: Properly disable EGL_EXT_buffer_age when we don't have it

If we don't have EGL_EXT_buffer_age then we don't have buffer age - it's
a completely independent extension to KHR partial update.

This partially fixes partial update on recent mali drivers.

theme: remove event gaps in enlightenment window frames

@fix

efreet: CRI and fail after 10 attempts to connect to efreetd

if efreetd cannot be connected to, stop infinitely trying to spawn it
since this generates crazy cpu load

probably this path should also send some cache events so that watchers
do not simply idle forever

ref T5200

ecore-evas: handle title_set and name_class_set for buffer engine

theme: make bryce vertical clipping work as expected

ref a94351a789ae7c11efb72c1b9f35cd7fae4d7fa2

evas table: CRI when attempting to pack a table child into another table

@fix

elm_code: Update and demo status messages inline

elementary: enhance atspi accessible documentation

gengrid: avoid call select callback twice in mode ELM_OBJECT_MULTI_SELECT_MODE_WITH_CONTROL

If multi select mode is ELM_OBJECT_MULTI_SELECT_MODE_WITH_CONTROL and
user code try unselect a item and select another part which not
realized select callback calls twice in this case. For avoid this no
needs focus newly selected item.

@fix

luncher: improving engage animations

eina_file: fixed T2244

Summary:
dirty fix for 'eet' problems with file deletion on WINDOWS
to work with the commit: D4698, D4699

Reviewers: cedric, NikaWhite, rimmed, vtorri, raster

Reviewed By: cedric

Subscribers: artem.popov, cedric, jpeg

Maniphest Tasks: T2244

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

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

evas: drop redundant pointer check

Summary:
The function already has a precondition test for !font, so this
additional test will never trigger.

@fix CID1371525

Reviewers: cedric, jpeg

Subscribers: jpeg, cedric

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

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

elementary: Fix minor typo in efl_ui_win.eo

Small patch to fix a minor typo in the "noblank" property of
efl_ui_win.eo

@fix

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

genlist: Fix a leak

Thanks @herdsman

ecore: Fix memory leak and logic

@cedric... how could this even happen??

release: Update NEWS and bump version for 1.19.0-beta3 release

ecore_animator: improve comparision check for stopping animator.

Summary:
Time cmparision does not ensure the corrctness of posotion.
It is beacuse of double number calculation. Double number calculation
are not convertable.for example:
   1/39 = 0.02564102564;
   but 0.02564102564 * 39 = 0.99999999996 != 1;

The addition check for pos ensure the pos to be reached at its correct
position.

@fix

Signed-off-by: Umesh Tanwar <umesh.tanwar@samsung.com>
Reviewers: cedric, singh.amitesh, jpeg, raster

Reviewed By: raster

Subscribers: atulfokk, cedric, jpeg

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

Signed-off-by: Amitesh Singh <amitesh.sh@samsung.com>

evas gl: Add a hack for GLES 3.1 with NVIDIA + GLVND

I believe there's a limitation in libglvnd where it doesn't
propagate the calls to eglGetProcAddress() properly to the vendor
library.

See also 265c851a8f2ac3cd1f93d523736d4cce0454fe2c

evas: Fix crash with events on the canvas

Since ecore now uses efl events to feed input events to the
canvas, anyone can now listen to any event on the evas. But
when using the legacy API the event info needs to be the legacy
struct, and not the eo event info otherwise crashes will happen.

While this is a new use of events, I consider it valid and it's
better to fix it rather than disallowing it. Fixed by wrapping
evas events the same way evas object events were handled.

Fixes T5266

evas gl: Fix usage of OSMesa

It seems OSMesa was recently updated to not expose symbols statically,
so dlsym() returns invariably NULL. GetProcAddress must be used. Note
though that the extension "EGL_KHR_get_all_proc_addresses" is not
present (OSMesa is OpenGL, not GLES), and there is anyway no list
of extensions in OSMesa (at the WSI level, glGetString() returns a
ton of GL extensions as expected).

My OSMesa version is 11.2.0 (mesa 17.0.1).

This fixes make check.

@fix

edje: Move prev_description out of ephysics ifdef

prev_description was used when HAVE_EPHYSICS is set, which is the
default, but I also added a use in 7072fbc2bfe25881 where the map
was not properly reset.

This removes an ugly #ifdef and opens the door to other fixes
similar to that map one.

genlist: Fix some more decorate mode issues

The item, after having been unswallowed from its decorate
item, becomes unclipped and unparented. The parent was well
reset, but the clip wasn't.

Test case:
  elementary_test -to "Genlist Decorate Item Mode"

I'm sure some bugs are still lurking. Genlist is so lovely.

Fixes T1551

edje: Fix rare issue with map

In the following sequence, the swallowed object map property is
never reset as it should have been:

- swallow object
- start program, change state to have a map
- do something
- start program, change state to have no map
    but before render, unswallow the object

At this point, the object will never be un-mapped. This is weird.
Somehow edje_calc avoids calling evas_object_map_[enable_]_set
excessively, but I believe the issue is that the object does not
need recalc. Its container needed recalc, not the child (which is
mapped). I'm not 100% sure.

Test case:
  elementary_test -to "Genlist Decorate Item Mode"
  Click on rotate, select a few items, scroll up and down. Enjoy.

Ref T1551

@fix

genlist: Make sure only one item is decorated

In "Genlist Decorate Item Mode" after decorating a few items
(rotate or slide, doesn't matter), only one item or none should
be decorated. Scrolling up and down the genlist should work just
fine. This fixes massive render issues and inconsistent states
of the items in this test case.

"rotate" mode is still going nuts.

Ref T1551

genlist: Make logic more readable

genlist: Fix invalid call to stack above/below

Test process:
  elementary_test -to "Genlist Decorate Item Mode"

Slide any item to the right, observe error logs.

Ref T1551

evas: Add more debug info to an ERR message (stack)

Also use eina_safety as those are safety checks. Helps with GDB
(single breakpoint in eina_safety).

Revert "ecore-evas reduce modifier modification on every event to cut cpu"

This reverts commit 3a9d54085b8751cbd0bca4698a8ff702d9bdc7b4.

I got crash and a lot of valgrind warning with this patch. All in all, I
think we can just wait for next release and do a proper cleanup of our API
to not rely on strings at all.

For references this is the first valgrind warning I get (Not going to past the 100 following one) :

==11860== Invalid write of size 4
==11860==    at 0xB10DDD1: _ecore_event_evas_modifier_lock_update (ecore_input_evas.c:432)
==11860==    by 0xB10E3DD: ecore_event_evas_mouse_move (ecore_input_evas.c:725)
==11860==    by 0x5D5115D: _ecore_call_handler_cb (ecore_private.h:317)
==11860==    by 0x5D5115D: _ecore_event_call (ecore_events.c:518)
==11860==    by 0x5D5CC47: _ecore_main_loop_iterate_internal (ecore_main.c:2381)
==11860==    by 0x5D5D42E: ecore_main_loop_begin (ecore_main.c:1289)
==11860==    by 0x5D5D490: _efl_loop_begin (ecore_main.c:2831)
==11860==    by 0x5D59555: efl_loop_begin (efl_loop.eo.c:32)
==11860==    by 0x14F275: main (test.c:1188)
==11860==  Address 0x19a36828 is 7 bytes after a block of size 1 alloc'd
==11860==    at 0x4C2AACE: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==11860==    by 0x4C2CC81: realloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==11860==    by 0xB10DDA8: _ecore_event_evas_modifier_lock_update (ecore_input_evas.c:425)
==11860==    by 0xB10E3DD: ecore_event_evas_mouse_move (ecore_input_evas.c:725)
==11860==    by 0x5D5115D: _ecore_call_handler_cb (ecore_private.h:317)
==11860==    by 0x5D5115D: _ecore_event_call (ecore_events.c:518)
==11860==    by 0x5D5CC47: _ecore_main_loop_iterate_internal (ecore_main.c:2381)
==11860==    by 0x5D5D42E: ecore_main_loop_begin (ecore_main.c:1289)
==11860==    by 0x5D5D490: _efl_loop_begin (ecore_main.c:2831)
==11860==    by 0x5D59555: efl_loop_begin (efl_loop.eo.c:32)
==11860==    by 0x14F275: main (test.c:1188)

eina: Fix more typo in doxygen

The eina_(rw)_slice_startswith functions both incorrectly describe how
the 'prefix' parameter is used, so fix those also

@fix

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

eina: Fix typo in doxygen

eina_(rw)_slice_startswith functions both incorrectly describe the
return value as 'slice ends with', when clearly the function is used
to find if a slice 'starts with' a prefix

@fix

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

ecore-evas reduce modifier modification on every event to cut cpu

this only modifiers modifiers if the modifier mask changed. it stores
it per seat and matches up before deciding to actually modifier the
modifiers. this SHOULD fix T5252

@fix

elementary: refactor code to avoid duplication in elm_code_widget.

ecore_evas: keep VNC API BETA until multi evas output is in, just in case.

elm_code: Don't leak rows when we resize down

Thanks Cedric :)
@fix

elementary: force elm_code line fill on object resize.

This fix elementary_test use of elm_code where the object needed
to get clicked to have their content displayed. The reason was that
the line where appended while the object size was 0 and once it was
resized the line where not displayed.

luncher: Smoothing engage theme

edje/examples: don't alter source directory

We would have removed files from the source directory before... oops...

elementary: fix elm_code widget tests by actually setting a font.

luncher: Update theme to ensure we're sending scale reset signls

ecore-evas-wayland: update no-op configure commit case for new acking method

ref 1b9970658f17d6b130a24cf9a960e1a43b696160
ref 5e7456a3150ddaed135f702c70800c7ed3d4f16f

Revert "evas_table: abort when there is already the option struct"

This reverts commit 008711b3b0f0b33e47e6b2fae7f925a0a9fb76ee.

this breaks elm_table_pack_set() as well as other valid usage and is
not a valid change to the code

ecore-evas-x: set flag for maximized state

fixes initial maximize state

ecore-evas-wayland: commit on no-op configure events

this handles the case of a window configure not triggering any visible change

fix T5192

elm_code: Fix position of cursor in selection

Make sure cursor is always at the left of our selection
@fix

Gengrid: Correct double comparision.

Summary:
 Use the more accurate EINA_DBL_EQ for double comparision.

@fix

Signed-off-by: Umesh Tanwar <umesh.tanwar@samsung.com>
Reviewers: singh.amitesh, cedric, jpeg, raster

Reviewed By: singh.amitesh

Subscribers: atulfokk, cedric, jpeg

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

Signed-off-by: Amitesh Singh <amitesh.sh@samsung.com>

ecore_animator: use EINA_DBL_EQ for comparing double values

Summary:
EINA_FLT_EQ was used to compare double values. replace it with EINA_DBL_EQ

@fix
Signed-off-by: Umesh Tanwar <umesh.tanwar@samsung.com>
Reviewers: raster, cedric, singh.amitesh

Reviewed By: singh.amitesh

Subscribers: atulfokk

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

Signed-off-by: Amitesh Singh <amitesh.sh@samsung.com>

Revert "Genlist: Calculate the min size of content before item realize"

This reverts commit 60566ca34d0036d21e5d134ba299b6f0fa26ce86.

This broke the layout in "Genlist Full Widget"

Genlist: Calculate the min size of content before item realize

Summary:
Genlist item doesn't change its size when its content size is changed,
but its size is determined in realization.
Therefore, deferred calculations for content should be performed immediately
before swallowing it by genlist item.

Test Plan: make and run attached sample

Reviewers: cedric, SanghyeonLee, jpeg

Reviewed By: jpeg

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

genlist: Fix invalid state of reused content

If an item is marked as disabled it should be re-enabled
before being put in the reusable contents cache. Otherwise
a following use of this object may result in a disabled
item being used, making the UI effectively disfunctional.

Also modify the test case to show and test this behaviour.

Add an efl_isa() to protect calls to elm_widget APIs.

Fixes T5236

@fix

evas - gl - remove warning for unused var with opengl vs gles

emotion: avoid crash during shutdown due to Ecore_Event queue.

elocation: avoid crash during shutdown due to Ecore_Event queue.

elput: avoid crash during shutdown due to Ecore_Event queue.

elementary: avoid crash during shutdown due to Ecore_Event queue.

eio: avoid crash during shutdown due to Ecore_Event queue.

efreet: avoid crash during shutdown due to Ecore_Event queue.

eeze: avoid crash during shutdown due to Ecore_Event queue.

ecore_x: avoid crash during shutdown due to Ecore_Event queue.

ecore_wl2: avoid crash during shutdown due to Ecore_Event queue.

ecore_win32: avoid crash during shutdown due to Ecore_Event queue.

ecore_wayland: avoid crash during shutdown due to Ecore_Event queue.

ecore_sdl: avoid crash during shutdown due to Ecore_Event queue.

ecore_psl1ght: avoid crash during shutdown due to Ecore_Event queue.

ecore_ipc: avoid crash during shutdown due to Ecore_Event queue.

ecore_input: avoid crash during shutdown due to Ecore_Event queue.

ecore_imf: avoid crash during shutdown due to Ecore_Event queue.

ecore_evas: avoid crash during shutdown due to Ecore_Event queue.

ecore_drm: avoid crash during shutdown due to Ecore_Event queue.

ecore_cocoa: avoid crash during shutdown due to Ecore_Event queue.

ecore_con: avoid potential crash during shutdown due to Ecore_Event queue.

ecore: avoid potential crash on shutdown in Ecore_Exe

ecore: add ecore_event_type_flush.

During shutdown it is possible that some event are still in ecore events
queue and get processed after the shutdown of the module that did emit them.
This would lead to crash in some case. The answer to this problem is to
normally manually track all ecore event in the queue and destroy them
before shutdown... Of course that make the API difficult to use and
basically nobody got it right.

This new API do actually as it says remove all the ecore event of a
certain type from ecore events queue. It is to be called on shutdown.

@fix

ecore_cocoa: fix windows initial rendering

We need to refresh the Cocoa's content view when showing the window.
In some particular cases (including the elementary_test initial window),
the content view is redrawn because of external events (the entry
animator). When no action at all is performed, the window and the
content view are resized... nice... but not drew...

Such a simple fix for this hell of a bug!

Fixes T5210

ecore_evas_cocoa: better debugging log