evas: remove float comparison warnings for evas_events

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

ecore-evas-x: remove float comparison warning from X backend

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

ecore-evas-wayland: remove float comparison warning from wayland backend

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

evas - async preload - stop hang when missing wakeup of preload data

this isn't pretty and i think some stuff needs a redesign... but...
this should fix T5025

@fix

ecore_con_legacy: fix compatibility behavior (eos-mark on receiver done).

While a socket can be closed to receive data resulting in EOS, it
could still be used to send stuff. Then it won't result in "finished",
just "read,finished" event.

However, previously this was considered a disconnect and we must
respect this otherwise tests (Ecore_Con_Eet suite) will hang waiting
for a disconnect.

efl_io_buffered_stream: eos-mark and flush sender on close.

By marking EOS on the outgoing queue we allow the sender to flush
pending data if that's possible.

This also fixes a typo in the condition for eos_mark, it operates on
outgoing, not incoming queue.

efl_io_reader/writer: silent errors when operating on unset-objects.

if we're operating on unset objects, fd are now -1 and would complain,
relax that.

efl_io_buffered_stream: if there is a copier, check if it's done.

Before we're exiting if there was nothing pending, but in some cases
this was too aggressive as the copier's source may be alive (ie:
not-EOS), however nothing was read to its internal buffer, thus no
pending.

Since the copier will track its source's EOS, just check if it's done.

efl_io_copier: fix typo in efl_io_copier_flush().

We should check if destination 'can write', not source.

do not emit events from efl_io_closer_close() on destructor.

On destructor we're not supposed to emit events, I even thought that
would be implicit, but it's not. If we do, for example an event
handler that would 'efl_del()' on "EFL_IO_CLOSER_EVENT_CLOSED" would
trigger too-many unrefs.

efl_io_reader/writer: always set 'can_*' to FALSE on errors.

If user tries to read or write and that fails then set 'can_read' or
'can_write' to false, regardless of the error code.

efl_io_copier: avoid extra copy, todo--.

Avoid the intermediate buffer and use the new eina_binbuf_expand() +
eina_binbuf_use(), reading directly to the binbuf.

efl_io_copier: remove TODO leftovers.

Copier is based on Efl.Io.Reader.can_read,changed and
Efl.Io.Writer.can_write,changed, these events do not keep being posted
until it's actually read/written, instead they trigger only once when
the property change -- unlike Efl.Loop_Fd read/write events.

eina_binbuf: allow expand & usage of extra bytes.

Some code needs to read directly into eina_binbuf to avoid an extra
copy from eina_binbuf_append* variants.

This can be achieved by using eina_binbuf_expand(), which returns a
write-able slice of the spare bytes. Once they are used,
eina_binbuf_use() should be called to increment "buf->len", which is
used by all other binbuf functions such as eina_binbuf_length_get() or
eina_binbuf_append_slice().

eo_debug: print tracebacks for more errors.

If eo_debug (libeo_dbg/EO_DEBUG), then print tracebacks if lifecycle
is being tracked.

elm_scroller: focus_next needs to give itself when no focusable child

@fix

docs: elm: fix stack_base property doc to cover getter function as well

Documenting the property itself will make sure getter as well as setter
are documented.

win: Fix shadow warnings

Revert "evas: Add source_region property to proxy objects"

This reverts commit 4e110a34bffe09abe4dd793f9ecf1cf3884ccf22.

Urgh. I'm stupid. Conceptually I still like the idea of the
region proxy, that only renders part of the source object
in a proxy surface. The problem right now is that the proxy
subrender mechanism renders to a surface that belongs to
the **source** and not the proxy object. As a consequence,
the region would become a property of the source, rather than
the proxy, which is not at all the intention of the original
patch. In other words, everything would fall apart if an object
has more than one proxy, for whatever reason.

I realized that when trying to actually test the region proxy.
It didn't work at all. Revert for now.

Ecore_win32: do not handle WM_SYSKEY* events.

Summary:
In case if WM_SYSKEYDOWN or WM_SYSKEYUP events are handled in
DefWindowProc - system doesn't init event for a system shortcuts.
For example any EFL application on Windows couldn't be closed by
Alt+F4 combination.

@fix

Test Plan:
Launch Elemenatry_test app.
           Try to close by Alt+F4 combination.

Reviewers: vtorri, raster

Reviewed By: raster

Subscribers: an.kroitor, bowonryu, cedric, jpeg, #eflete

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

efl stack wins - add base property/boolean to indicate win is a base

part of adding stacks...

evas/ply: Add filename in error logs

As suggested by @k-s

I'm pretty sure that this parser will easily fail. Eg. find uchar
properties but it's not r,g,b in this exact order? No color for you.
Find no uchar at all? Fail immediately even if we can handle a
no color situation just fine.

elm_test: Also filter categories when typing in search bar

ecore: remove float comparison warning from Ecore_Animator code.

ecore: remove float comparison warning from Ecore_Poller code.

ecore: remove floating point comparison warning from Ecore_Thread code.

ecore: remove floating point comparison warning from Ecore main loop code.

ecore: remove float comparison warning from Ecore_Pipe.

efl: remove float comparison warning from Efl.Gfx.Path code.

ecore_evas: remove float comparison warning from DRM backend.

eina fixup matrix test

evas: get rid of float comparison warning in Evas_3D.

eina: get rid of float comparison warning on quaternion tests.

eina: get rid of float comparison warning on quad tests.

eina: get rid of float comparison warning from str test.

eina: use proper logic for comparing double in matrix tests.

eina fixup bezier

xcf: fix float comparison in xcf generic loader.

eina: fix double comparison in eina quaternion.

eina: fix double comparison in eina matrix.

eina: fix double comparison in eina bezier.

eina: correctly compar double in eina xattr test.

eio: correctly compar double in xattr test.

eina: add general purpose function to compar float and double.

wayland_egl: Use common implementation of eglCreateImage

This might fix nested compositors on some architectures...

wayland_egl: Remove some weirdness from our egl extension queries

Still totally wrong, of course, but at least a little less wacky.

gl_x11: Use common implementation of eglCreateImage

Throw away the local copy and use the one in gl_common.

gl_drm: Use common eglCreateImage implementation

Refactor to reduce copy and paste.

gl_common: Make secsym_eglCreateImage a wrapper

eglCreateImage and eglCreateImageKHR are subtly different.  Now we try to
handle them both properly by exposing an abstraction.

ector_gl: Add some EGL typedefs to internal header

This lets us include EGL/egl.h in gl_common.

gl_common: fix eglCreateImageKHR query

There are surprisingly few GL extensions that start with GL_EGL...

This is apparently more widely deployed than OES, so let's bump its
priority too.

efl_net_server_udp_client: make error easier to understand.

UDP client has no socket on its own, so it can't be marked as
close-on-exec, however failing to resolve the method is bad as
confuses the user.

Instead provide a method that checks if the setting is different from
parent, in such case provide a meaningful message. Otherwise just be
quiet.

efl_net_socket_udp: expose init() as protected method.

remove one more TODO: since Efl.Net.Ip.Address was introduced we can
now expose Efl.Net.Socket.Udp.init as a protected method that will
configure the internal address we use for the remote peer. This allow
subclasses to override or call such methods.

efl_net_socket_ssl: remove TODO leftover.

There are no jobs to stop there, it was a left over since this code
was somehow derived from Efl.Loop_Fd where "read" and "write" would
constantly be called until the kernel flag was cleared, there the
callback must be removed to stop jobs, not here.

efl_io_buffer: adopt read-only and read-write buffers.

todo--, allow buffer backing store to be provided by Eina_Slice
(rdonly) or Eina_Rw_Slice (rdwr).

evas: improve logic regarding events filtering enablement

Only the size of events_whitelist isn't enough, because in some
cases the user may be disabling the usage of a specific seat.

Considering the following scenario, the issue will easy to understand:
 - an application with two entries (one to be used by seat 1 and other
     by seat 2)
 - the first seat is announced - it is enabled for entry 1 and
     disabled for entry 2
 - the second seat is announced

Before second seat is announced, the first seat would be able
to input the entry 1, because the events_whitelist of such
object will continue empty.

So a flag will be used to identify an object with active
filter.

Reviewed By: iscaro

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

evas: remove focus when removed from filter whitelist

It doesn't make sense to remove it when a seat is added
to the list. It should be removed only when this seat
is blocked.

But when the list receives its first item, then
it also should be checked if the focused seat
is the one just added, otherwise the previous one
must be removed.

tests/ecore_con: skip ipv6 tests if that's disabled on the running system.

more eo simplified to use Eina_Slice instead of pointer to.

Working directly with Eina_Slice is easier than a pointer to it,
requires no validation of the pointers and is cheap since it's just
putting together size_t + void*.

However we can't hint the user of 'const(Eina.Slice)' properties as
Eolian is incorrectly generating getters as:

   const Eina_Slice class_property_get(...)

which is makes compilers complain about ignored qualifiers:

   ../src/lib/ecore/efl_io_copier.eo.h:329:7: warning: 'const' type qualifier on return type has no effect [-Wignored-qualifiers]

Leave some TODO so @q66 can fix those.

Efl.Io.{Queue,Buffer,Buffered_Stream}: slice_get is now a property.

Previously we couldn't return a slice, instead required the user to
pass a slice and we'd fill it since Eolian couldn't generate fallbacks
for structures.

Since @q66 fixed eolian, we can now return the structure itself as
initially wanted, ditching some TODO from the code.

Merge branch 'devs/bdilly/edje_multiseat'

This series implement multiseat support on Edje:
 * emit signals with seat name as suffix
 * handle multiseat focus
 * add optional argument to FOCUS_SET and FOCUS_OBJECT actions
   to specify a seat
 * add Edje naming for seats (incremental) with flag
   to disable this behaviour - so custom (evas) names will be used
 * add 3 examples exploring all these new features

Reviewed By: cedric, iscaro, jpeg

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

@feature

edje/examples: add example of use_custom_seat_names flag

Since seat names defined on evas will depend on evas backends,
maybe udev rules, etc, we aren't able to demonstrate it
without manually setting another name for such seats
with efl_input_devices_name_set(). Otherwise
we wouldn't have guarantees it would match
names on EDC.

edje/examples: add example for focus actions

Just a regular (non multiseat) focus example for
actions FOCUS_SET and FOCUS_OBJECT

edje: add flag to choose between evas or edje seat naming

collections.group.use_custom_seat_names should be set to '1'
to use seat names on signals as provided by Evas.

By default just follow Edje naming approach
("seat1", "seat2", ...)

examples/edje: add dynamic seats example

It's an example of how themes can be implemented even if the number
of seats using the UI is unknown, and how it can be done
using scripts or C code.

edje: support action seats on edje_edit

edje: accept optional seat parameter on FOCUS_OBJECT

If not provided, FOCUS_OBJECT action will keep acting
over default seat.

Also include a usage on edje-multiseat test
(actually no example was exercising this action).

examples/edje: add multiseat example

edje: add names for seats following an incremental pattern

This way it'll be possible to write EDC with seat names that
will work no matter how seats are named by each ecore_evas
backend or users.

edje: support seat on focus action and signals

Seat goes as an optional parameter for FOCUS_SET (if not
provided, act over default seat), and emit signals
with seat suffix:
 * focus,in,$SEAT
 * focus,out,$SEAT
 * focus,part,in,$SEAT
 * focus,part,out,$SEAT

edje: add new signals providing seat information

Add the following signals:
 * mouse,in,$SEAT
 * mouse,out,$SEAT
 * mouse,move,$SEAT
 * mouse,clicked,$BUTTON,$SEAT
 * hold,on,$SEAT
 * hold,off,$SEAT
 * drag,$SEAT
 * drag,start,$SEAT
 * drag,stop,$SEAT

evas: add getter for devices by name

Make it possible to get the evas device given
its name. It sounds useful for Edje since
programs will reference seats by name.

build: enable -Wfloat-equal for compiling

 #WarningOfTheMonth

ecore_con/ecore_ipc: flush should wait for connection.

Some applications will create the handle, immediately send data, flush
and delete it, expecting the data to be sent to remote peer.

This is a bad behavior as the application would become unresponsive
until the connection is established, data can be written (since
depends on server consuming it), then allow it to be closed.

A proper behavior here would be to chain based on events, with the
usage of a copier would be simply wait for "done" event.

However the legacy API allowed this and terminology depends on this
awkward "feature", thus be bug-compatible.

This fixes T5015.

efl_io_file: initialize all fd to -1.

Since 0 is a valid fd, we should initialize these to -1.

efl_net_socket_fd: initialize fds to INVALID_SOCKET

Since 0 is a valid fd, then we'd try to do socket operations on it,
which would generate misleading errors.

ecore_con/ecore_ipc legacy: fail early for local sockets.

In the old/legacy API the socket would be opened early in non-blocking
mode (connect returned errno==EINPROGRESS), with UNIX socket being
path-validated early and returning NULL as 'server' handle.

Some applications relied on this instead of monitoring the "ERROR"
events, considering the connection to be successful if there was a
handle -- this was the case with Terminology after it moved from DBus
to Ecore_Ipc.

Although this is not correct, we must keep compatibility and thus we
stat() in compatibility layer, failing early as the old API would do.

ecore_con/ecore_ipc: keep log domain alive after shutdown.

These legacy API had the nasty behavior of keeping handles alive until
the pending events were dispatched, this could happen after the module
itself was shutdown, resulting in log to unregistered domains.

Then do not unregister the domain -- eina_shutdown will avoid leaks
anyway.

elput: Fix resource leak

Coverty reports that we leak 'reply' here if we fall into the error
condition.

CID1367506

@fix

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

elput: Fix resource leak

Coverity reports that 'reply' is leaked here if we fall into the error
condition.

CID1367507

@fix

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

coverity: add modeling file to reduce false positives.

efl_net: fix CID 1367489 due copy & paste.

we're looking at "x" since it's the return of getaddrinfo(), not "r"
which is used for something else.

efl_debug_common: remove stale code/macros.

These are not needed anymore.

evas/ply: Fix some memleaks

I have a strong feeling that this parser is not very robust...

Closes T5001

elm_code: Don't assume control characters make a change

@fix

Eio_monitor: support native monitor for Win32 (T4978)

Test Plan:
eio_monitor_backend_add()  and eio_monitor_fallback_add use different structures
**eio_monitor_backend_add()  **
struct _Eio_Monitor_Backend
{
   Eio_Monitor               *parent;
   Eio_Monitor_Win32_Watcher *watcher_file;
   Eio_Monitor_Win32_Watcher *watcher_dir;
   Eio_Monitor_Win32_Watcher *watcher_parent;
};

**eio_monitor_fallback_add()**
struct _Eio_Monitor_Backend
{
   Eio_Monitor *parent;

   Eina_Stat self;
   Eina_Hash *children;

   Ecore_Timer *timer;
   Ecore_Idler *idler;
   Ecore_Thread *work;

   int version;

   Eina_Bool delete_me : 1;
   Eina_Bool initialised : 1;
   Eina_Bool destroyed : 1;
};

therefore, an error occurs
**void eio_monitor_backend_del**(Eio_Monitor *monitor)
{

   if (!_eio_monitor_win32_native)          <------_eio_monitor_win32_native is EINA_TRUE
     {

        eio_monitor_fallback_del(monitor);  <------ignored------
        return;                             <------ignored------
     }

   _eio_monitor_win32_watcher_free(monitor->backend->watcher_parent);
   _eio_monitor_win32_watcher_free(monitor->backend->watcher_dir);
   _eio_monitor_win32_watcher_free(monitor->backend->watcher_file);
   free(monitor->backend);
   monitor->backend = NULL;
}

Reviewers: raster, vtorri, an.kroitor, jpeg, reutskiy.v.v, FurryMyad, cedric

Reviewed By: cedric

Subscribers: vtorri, artem.popov, cedric, jpeg

Tags: #windows, #efl

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

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

scrollable_interface: fixed logic for calcualting position to scroll.

Summary:
When width of parameter(w) is bigger than or equal to scroller's width(pw),
scrollable object must be scrolled to x position.

Test Plan: elementary_test -> focus 4

Reviewers: woohyun, SanghyeonLee, Hermet, cedric, jpeg, raster

Subscribers: cedric, jpeg

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

elm_widget: add logic for updating geometry after moving

Summary:
scrollable object.

In multiple scrollable case, the target object should be updated.
Because the first object is scrolled. So the parent of the first object should be scrolled.

Reviewers: woohyun, Hermet, jpeg, cedric, raster

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

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

edje: handle case when strdup would be NULL.

alternate solution to D4488. Would be better to have a buffer and recycle it.

elm.hoversel: fix item size calculation

Summary:
The way to find the position of hoversel expansion is based on geometry
currently, but it causes errors when theme is not made as expected.
This patch makes hoversel use string to find the right position and
calculate item width correctly when items are shorter than hoversel itself.

Test Plan:
elementary_test -to hoversel
(error case shows in tizen mobile device when screen is rotated)

Reviewers: cedric, jpeg

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

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

eio: check error case for eina_value_get.

CID 1356609

ecore: add test for efl_promise_connect.

ecore: add helper to automatically connect a future result to a promise.

This helper is to be used likely with next and an external request.

ecore: add tests for new promise delayed behavior.

ecore: fix test to generate one main loop iteration to see promise fulfilled.

ecore: delay promise fulfillment to next loop iteration.

ecore: remove useless information from header.

ecore: refactor promise propagation code.

evas engines: Stop using arrays of 1 member for egl context and surface

There's actually an array of 2 in gl_x11, but it appears we only use the
first 1, except in an #if 0 block.

I gather this is legacy for some reason - it sure doesn't seem to make
any sense now, so I'm chopping off all the extra array indexes.

Note: I've not changed gl_sdl - it looks like it doesn't use the context
or surface at all and they could just be removed, but I don't build that
one here and don't want to touch it.

edje_pick: copy images from image sets

@fix