ecore_con: reduce amount of warning related to clobbered variable.

efl_debug: silent unused use of static variable.

evas: fix comparison between unsigned and signed warning.

ecore_con: fix incompatible type warning.

ecore_con: fix misleading indentation warning.

ecore-x: fix Mode_switch key mapping

- remove redundant XKeysymToString when sym == sym2

@fix

eolian: enable typechecked expression validation

This code was formerly disabled, but it's safe now.

Eo: remove unreachable code in isa.

This condition can never be true. It can't be NULL here. A NULL here
would have caused a crash earlier, though it can only be NULL if an
allocation fails, which is something that we don't really handle
for smallish allocations.

CID1366823

eolian: handle pointer types in typechecking expr eval

Also improve expr error messages and fix Eolian tests.

ecore con tests: fix missing include

eolian gen: specialize default values by type

Now we can generate NULL for stuff that is pointers and empty
struct literals for struct instead of just 0 for everything. The
previous behavior was incorrect for those cases and generated
broken code.

eolian: handle pointers in aliased_base_get

eolian: unary expr eval for floats and add a signed number mask

This adds a new mask for all signed numbers (sint + float) and
fixes unary expr evaluation for floats, as well as fixes eval
error messages.

ecore-wl2: restore previous session recovery behavior on hide

ref 9492ee21dfa642123043da5d5ef7954e886b0f9c

elput: Properly check return values from libinput config functions

As libinput_config_status may contain 3 possible return values, we
need to santize the return values there into Eina_Bool for use in our
own functions.

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

elput: Add API functions to get/set if tap-to-click is enabled

This patch adds API functions to get or set if tap-to-click is enabled
on a touchpad device

@feature

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

elput: Add API functions to get/set touchpad click method

This patch adds API functions to get or set the click method used on
touch devices. The click method defines when to generate software
emulated buttons

@feature

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

elput: Add API functions to get/set the scroll method for a touchpad
device

This patch adds API functions which can be used to get or set the
scroll method used for a given device. Scroll method defines when to
generate scroll axis events

@feature

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

elput: Add API functions to enable/disable dwt support on touchpads

This patch adds API functions to get/set if dwt (disable-while-typing) is
enabled on a touchpad.

@feature

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

elput: Add API functions to enable/disable drag-lock on touch devices

This patch adds API functions to get or set if drag-lock is enabled on
touch devices.

@feature

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

elput: Add API functions to enable/disable tap-and-drag

This patch adds new API functions for Elput touch devices to get or set if
tap-and-drag is enabled on a touchpad device

@feature

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

Fix generate ecore key events on windows.

If Ctrl+number combination pressed/unpressed function
_ecore_win32_event_keystroke_get return NULL. It happens because
ToUnicode WinAPI func fails to prepare unicode string for given scancode
and the keyboard state.
This commit add exception for  the case with digits keys. In case if
there no translate
string, but the digit key processed - will be created a normal event as
usual.

elementary: slider uses max value when atspi tries to set value.

Because double type value could be 1.00000000000000022204.

naviframe: Raise previous view after item pop is finished.

Previously, previous view was raised immediately when item pop is
started so previous view covers current view when item pop is started.
Now, previous view is raised after item pop is finished so previous view
covers current view when item pop is finished.

Change-Id: I86f343e0b49f3801d00e553755896f0eb756daa0

edje_calc: remove redundant checking

We check amin > ZERO && amax > ZERO, so no need to do it them again.

Ecore Evas VNC: Properly shutdown server during cleanup.

Summary:
The function rfbScreenCleanup() does not close the open sockets,
a call to rfbShutdownServer() must be done to properly close them.

Reviewers: cedric, bdilly

Reviewed By: bdilly

Subscribers: cedric, jpeg

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

edje_pick: fix memory leak while on group dependency build

@fix

elc_naviframe: Delete naviframe items in LIFO manner on widget deletion.

Summary:
Naviframe manages items in the form of a stack, but deletion is happening
in FIFO manner, the deletion of items on widget deletion should also happen
in LIFO manner.

Use Case: Application allocates memory on first push and passes down the same
handle for all subsequent pushes, now on deletion as first item is deleted first
crash happens when the memory is accessed in second item on its deletion.
hence Naviframe should also delete items in LIFO manner.

@feature

Test Plan:
elementary_test->naviframe->push multiple pages
Now terminate the app, the items should be deleted in LIFO
manner.

Reviewers: Hermet, Jaehyun, Jaehyun_Cho

Reviewed By: Jaehyun_Cho

Subscribers: cedric, govi, rajeshps, jpeg

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

elm_test: Add a test case for gfx filters

This adds a test window for the efl_gfx_filter (also known as
evas filters). The builtin examples are not incredibly pretty
but they showcase some of the most basic features. They also
show that some of the filters need to be improved.

Note that the code contains a lot of FIXME due to the incompleteness
of our EO API. Also, Efl.Ui.Text still has quite a few issues (sizing,
lack of a working change event, ...) so the UI doesn't even look as
designed (a label is missing). Hopefully this should get fixed over
time.

Oh and this is mostly using EO APIs but that's a terrible idea as
evas_object_text is not going to be exposed in EO land... except
it's the only object implementing filter support (with image).

spinner: Fix mouse wheel events

Fix invalid cast of event_info

evas: Make evas object loop users

This makes efl_loop_get() work on evas objects, returning the
main loop as expected. Also make the loop a property of the
Loop_User class (shouldn't it be called Efl.Loop.User instead?)

evas: Fix stacked box align

I added a way to respect the object's alignment when adding them
to a stacked box, but that alignment should only be used when the
box align is set to fill, otherwise both aligns would conflict.

See 3df7b717c932adac481aa0ec527d4d5893b07bb8

Fix compilation warning due to redeclared variable

Merge branch 'devs/iscaro/memory'

This series reduce the memory footprint caused by
_evas_object_pointer_data_get() previous implementation.

Patches by Guilherme Iscaro <iscaro@profusion.mobi>

Reviewed By: bdilly, cedric, jpeg, raster

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

Efl.Canvas.Object: Remove the extra parementer at _evas_object_pointer_data_get()

The Evas_Pointer_Data struct already contains a Efl.Input.Device pointer.

Efl.Canvas.Object: Reduce memory used to store Evas_Object_Pointer_Data.

The hash implementation demonstrated that too much memory was being used
to store the Evas_Object_Pointer_Data. In order to reduce the memory usage
this patches now changes the Evas_Object_Pointer_Data storage to an Eina_Inlist and
now Massif profiles shows that the memory usage was drastically reduced.

evas: fix memory leak in software generic in error case.

Summary:
pixels_tmp  memory need to be free in default case too.

Signed-off-by: Awadhesh Singh <awadhesh1.s@samsung.com>
Reviewers: singh.amitesh, raster, cedric

Reviewed By: cedric

Subscribers: atulfokk, cedric, jpeg

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

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

spinner: call changed callback on value set.

Summary:
If user set spinner value other than it's
current value, this is change in value. So changed
callback must be called on value set.

@fix

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

Subscribers: atulfokk, cedric, jpeg

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

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

Merge branch 'devs/iscaro/event-filter'

Series of patches adding support to filter input and focus
events per seat for any evas object.

So it will make it possible to block or unblock keyboard,
mouse and focus events that was originated by a specific seat.
Requested on RFC regarding multiseaet support on Edje -
such feature will be also supported on Edje later.

Patches by Guilherme Iscaro <iscaro@profusion.mobi>

Reviewed By: bdilly, cedric, jpeg

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

Evas: Add event filter example.

This example exercises the new seat event filter API.

Evas: Objects must not be focused with the seat is not allowed to emit events.

Seats that are not allowed to emit events must not be able to focus objects.

Efl.Canvas.Object: Override efl_event_callback_[legacy]_call.

In order to properly block events from a given seat, Efl.Canvas.Object must
override the efl_event_callback_[legacy]_call to check if the event
is allowed or not.

Evas: Add seat_event_filter_set()/get() API.

With this new API one can block or unblock keyboard, mouse and
focus events that was originated from a seat. This is useful to
create applications that wants to establish some kind of seat segregation.

elm: theme: avoid double free in theme version check

If we would have a version below 110 we would run into a double free here.
Just warn and let the freeing happen two lines below.

CID: 1366926

ethumb: fix resource leak introduced in file path sanitization rework

This extra snaitized path assign path does not make sense and leads to a leak
in the end. We already have sanitized_path and handle that correctly.

Fixes 62a0c41fd3bf8796efec55db52395f13a78fb27f

CID: 1366925

ecore_con_legacy: fix coverity issues (use after free)

for hard failures (highly unlikely), the handle would be deleted and
the next use would crash.

guard against those or reorder when it makes sense.

efl_net_ip_address: make coverity happy.

host and port were already checked, but let's quiet CID 1366921

test efl_net_ip_address: also check localhost6.

Fedora and other systems use localhost6 for ::1, then use that in
addition to localhost.

add missing includes for BSD.

Thanks @netstar.

elm_code: Fix tests for parameter reordering

Apologies for the badness of not running the tests before

test efl_net_ip_address: check if /etc/hosts has '::1 localhost'

it seems that the bot and some users do not have ::1 localhost in
/etc/hosts, check for that and print it out.

be more verbose on resolve errors, showing the results and error code.

also allow more time for resolution, otherwise the local timeout may
expire before getaddrinfo() returns with failures.

efl_net_ip_address: improve docs.

thanks stefan for reminding me of these :-)

Ui text: remove unimplemented imf api

Canvas text: call "changed" on style_set

Needs to be picked up by widgets e.g. Ui.Text to update the positions of
decorations.

Ui text: move tests to test_efl_ui_text.c

Merge this into the file that already tests the label-like behavior.

Canvas text: call "changed" on annotation_insert

This was missing from non-item type of annotations.

Ui text interactive: fix passing of wrong object

Ui text: implement non-editable beahvior

Non-editable behavior resembles a label. It doesn't allow changing the
text in any interactive form. The blinking cursor is also hidden.
Selection, however, it allowed.

Among the changes, I added a test which you can run using:
"elementary_test -to "efl ui text label".

Ui text: disable selection handler from test

Ui text: fix selection handler behavior

Ui text: implement scrolling with line wrap

docs: ecore_con: fill gaps in newly added efl_net_ip_address documentation

docs: ecore_con: document newly added extern struct item

elm_code: Fix parameter ordering for consistency

elm_spinner: Spinner internal focus logic fixed to following new focused UI concept.

Summary:
The spinner entry will be activated when user gives a focus to text button in new focused ui concept.

To support this, we have to change internal logic about change text button to entry, entry to text button.

@fix

Test Plan:
elementary_test
spinner sample.

Reviewers: woohyun, Hermet

Subscribers: cedric, jpeg

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

efl_net_dialer_http: fix HEAD requests.

We do not need to keep a "only_head" flag, but we must set
CURLOPT_NOBODY instead of going the "CUSTOMREQUEST" route, otherwise
curl won't follow redirects, etc.

efl_net_dialer_http: fix reference check.

With the last patch to fix delete-from-curl callback it went too much,
considering it was always dead (in the test scenario it was, so it was
"right"), but broke normal cases.

efl_net: add Efl.Net.Ip_Address

This is a string parser, serializer and asynchronous resolver.

It's purpose is to convert to and from the strings we use in our
dialers and servers, such as "127.0.0.1:1234" or "[::1]:1234",
properties allow to check the family, port, address bytes (slice) and
even get a struct sockaddr pointer to use with bind()/connect() in
outside code.

It will also offer some utilities present in netinet/in.h in an easy
to use way, after all IN6_IS_ADDR_LOOPBACK() works one way, while
there is no IN_LOOPBACK and comparing with INADDR_LOOPBACK will lead
to errors since it's in network order.

Last but not least, it will do asynchronous resolve of host and port
names using an internal thread and getaddrinfo(). The results are
delivered using a Future with an array of objects.

efl_net: move error COULDNT_RESOLVE_HOST to broader scope.

This error is shared by Dialer and Server, will also be used by IP
resolution.

Efl_Future: class name (string) using "." for namespace.

Following recent eolian change, the string must have "." to split
namespace, not "_".

yoda-style inactivity timeout.

s/inactivity_timeout/timeout_inactivity/g so it matches the EFL names
with scope first, like timeout_connect, etc.

efl_net: spell check s/revogation/revocation/g

Sorry, in pt_BR it's revoGar... so I did misspell :-)

efl_canvas_proxy: fix the parameter of EINA_COW_WRITE

Summary:
EINA_COW_WRITE_BEGIN and EINA_COW_WRITE_END's parameters must be same.
But 'Read' paramter is different. So wrong variable's value is changed.
@fix

Reviewers: woohyun, Hermet, cedric, raster, jpeg

Reviewed By: jpeg

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

elementary: naviframe uses widget item for atspi

Application developer could rewrite accessible information of naviframe item.
But view object - VIEW(item) - is not accessible from application side.
So the accessible information should be set to widget item.

ecore_con: Ecore_Con_Server now on top of Efl_Net!

This is a major work and unfortunately couldn't be split into smaller
pieces as old code was highly coupled.

Ecore_Con_Server is now a wrapper around Efl_Net_Dialer_Simple
(ecore_con_server_connect()) and Efl_Net_Server_Simple
(ecore_con_server_add()), doing all that the original version did with
some fixes so ecore_con_ssl_server_upgrade() and
ecore_con_ssl_client_upgrade() are more usable -- see the examples and
-t/--type=tcp+ssl.

I tried to be bug-compatible, with code annotations where things
doesn't make sense. This was based on ecore_con_suite tests and some
manual experimenting with the examples, these can be helpful if you
find regressions (report/assign to me).

ecore_con_server_example: add --socket-activated

ecore_con_server_example: add tcp+ssl, upgrades a live client.

add a new -t/--type=tcp+ssl, there you can send "Upgrade: SSL\n" to
request the server to start the handshake.

This can be paired with the ecore_con_client_example, there you can
type:

   Upgrade: SSL\n
   STARTTLS\n

The second is a dialer local command to upgrade it to SSL, matching
what the server expects.

ecore_con_client_example: allow tcp->ssl upgrades.

The example now offers --type=tcp+ssl, in this case it won't send a
"hello!" message to avoid messing with the handshake. Once the client
(dialer) receives the user command STARTTLS
(--starttls-local-command), then it will upgrade the connection.

Usually in real life you need to send some command to server, such as
upgrade, STARTTLS and then upgrade... unless you connect to a SSL-only
server (ie: ecore_con_server_example --type=ssl).

elm_code: Fix cursor x offset calculations

ecore_con_url: only close the request if we got a final response.

if we're doing 30x redirects or anything else, keep going. If it's an
error it will be reported later, otherwise EOS. Fact is we're only
interested if 20x.

efl_net_dialer_http: fix delete-from-CURL callback.

This was annoying to identify as the sequence is kinda difficult to
get, but Terminology was doing a HEAD request and it was triggering
this case in particular.

Fixes T4975.

elementary: set atspi role before sending a signal

It is necessary to check atspi role before sending a object:state-changed:showing signal.
The signal is fired when _elm_widget_efl_gfx_visible_set is called.

elm_code: Use cursor theme from elm_entry

efl_net: fix formatting warnings from windows.

efl_net_dialer_http: fix build on windows.

Windows time_t is not a long, but long-long, then stick with int64_t
so it works everywhere (converts to time_t internally).

And there is no gmtime_r(), then use the gmtime() if not detected.

efl_io_closer_fd: fix warning on windows.

docgen: migrate to new Eolian-provided tokenizer

elm_code: use geometry of cells to paint a simple cursor

Also allows others to figure out where cells are :)

efl_net_server_unix: do not mkpath for abstract sockets.

efl_net_dialer: emit 'resolved' even if connection failed.

If we resolved the address but couldn't connect, use
efl_net_socket_address_remote_set() and emit
EFL_NET_DIALER_EVENT_RESOLVED.

efl_net: optimize serving of IP addresses.

If we can parse the IP using inet_pton() and the port, there is no
reason to call getaddrinfo() in a thread.

This is required since ecore_con_suite (for ecore_con-over-efl_net) will
assume the server is running as soon as it's created.

efl_io_copier: callbacks may close the copier.

direct or indirect events may trigger the user to close the buffer.

efl_io_buffer: callbacks may close the buffer.

direct or indirect events (ie: can_read_set/can_write_set) may trigger
the user to close the buffer.

efl_io_queue: callbacks may close the queue.

For example, _efl_io_queue_update_cans() triggers "can_read,changed"
and from there users may close the queue, in such case we shouldn't
set can_write.

efl_io_buffered_stream: del inner_io if we're the parent.

This is a nice convenience if inner_io was reparented to the wrapper.

efl_net_server_simple: fix leak reported by eo_debug

eo_debug now reports leaked objects, which is very nice :-)

efl_net_dialer_socket: allow us to create a dialer from existing object.

If we want to upgrade a dialer, then we must have a way to know if
that socket has already adopted another socket so we don't create it.

We can't simply use efl_net_socket_ssl, otherwise we'd miss some
methods such as efl_net_dialer_address_dial_get() and events such as
connected.

efl_net_socket_fd: do not close stdout if nothing was set.

if we create an object, say a TCP dialer, and don't connect/bind, then
we have no FD (=0). If we set FD to INVALID_SOCKET on start, other
parts of the code will fail since they consider that "closed", but
we're not closed yet.

Then check for family == AF_UNSPEC && fd == 0, if so don't close it.

efl_net_ssl: do not access torn down sockets.

OpenSSL crashes if given a NULL pointer, then be safe and remember if
we did the tear down -- print error so bugs can be identified more
easily.

efl_net_ssl_conn-openssl: fix dlsym() for X509_check_ip_asc.

At least in ArchLinux the function has no "_" in the symbol name,
matching perfectly what's in the header.

If in other systems it misses such symbol, then check for both.