fileselector: Fix crash on file select

Fixes T4686 (further fix)

Efl.Loop.User: Use prefix to rename efl_loop_user_loop_get

efl_loop_user_loop_get is idiotic. efl_loop_get should be good enough.

ecore-evas - x fix intial iconified state so terminology -I works

this fixes intitial iconic state for x11 as demonstrated by

terminology -I

but enlightenment is broken though... xterm -iconic also shows the
same break with a black window.

@fix

ecore_evas x - dont set withdrawn to fals on show but wait for wm state

so we handled override cases and set withdrawn to false on show, but
when normally managed it might be nicer to wait for a state change via
the wm state property to know we are "normal"

this should fix T4699

@fix

fileselector: Fix eio model usage (future_all)

The value stored by efl_future_all is an accessor, not an
iterator. This fixes the usage of promises internally to
the fileselector widget.

See 1a2014a12234593f4ba27f2f76f87dc481447e7c

Fixes T4686

eio_model: Provide main loop even without parent

An eio model (and I believe most other models as well, in fact)
require a Efl.Loop.User as parent in order for efl future/promises
to work. This adds a fallback code directly inside eio model. The
alternative was to parent and efl_del all models in fileselector.

See T4686

evas: Provide loop from Evas.Canvas

And indirectly also Efl.Canvas.Object.

I believe those two classes should even inherit from Efl.Loop.User.
Right now this patch relies on the new dependence of Evas over Ecore,
and is maybe a bit ugly as is.

Ping @cedric

See T4686

Revert "ecore_evas x - dont set withdrawn to fals on show but wait for wm state"

This reverts commit 2c736adc873bfb998103da60650ebc156cba70ad.

well that was totally unexpected. - efl app windows dont show at all..
wtf? this should not have affected that at all..

ecore_evas x - dont set withdrawn to fals on show but wait for wm state

so we handled override cases and set withdrawn to false on show, but
when normally managed it might be nicer to wait for a state change via
the wm state property to know we are "normal"

this should fix T4699

epp: fix memory corruption when using #warning and #error

The epp instructions #warning and #error would led to a segmentation
fault (invalid free) because the malloced buffer's base pointer was
moved.

@fix

Ui text: port link anchors from entry

This is without the viewport optimization in the meantime.

Canvas text: add annotation_positions_get method

We need a method that allows us to place the cursors at the start and end of an
annotation. This is required for things like getting the geometry of a range.

@feature

Efl canvas text: fix api doc a bit

Canvas text cursor: add simple_geometry_get method

The ported geometry_get was actually the legacy simple_geometry_get.
For getting simple geometries like selection this was enough, but I forgot that
we also need to query more complex geometries e.g. links.
This is required to implement link anchors in Ui Text.

Now geometry_get and simple_geometry_get are the same as their legacy
counterparts.

@feature

Ui text: update all decorations on resize

Ui text: port atspi implementation

eina: Fix compilation with clang

It's not because the bug with __builtin_prefetch is inside
clang/llvm that we must break the build for people who prefer it
over gcc. As soon as a non-broken version is out, the ifdef must
be either removed (and ask people to update their clang install)
or add a version check based on __clang_xxx__.

Compilation tested with clang 3.8.1 and gcc 6.2.1.

naviframe: Add "push,finished" and "pop,finished" smart callbacks.

"push,finished" and "pop,finished" smart callbacks are added to notice
when push and pop operation is finished.

New smart callback signals are added as follows.

"push,finished,item,activated" : Called when item push is finished and
void *event_info in smart callback is activated item (new item).

"push,finished,item,deactivated" : Called when item push is finished and
void *event_info in smart callback is deactivated item (current item).

"pop,finished,item,activated" : Called when item pop is finished and
void *event_info in smart callback is activated item (previous item).

"pop,finished,item,deactivated" : Called when item pop is finished and
void *event_info in smart callback is deactivated item (current item).

naviframe: Fix to show/hide item view.

Naviframe item view did not become visible if top item view is deleted
by elm_object_item_del().
To resolve this problem, program for "elm,state,visible" is fixed and
"elm,state,invisible" is added.

naviframe: Remove redundant programs.

The redundant programs emit duplicate signals incorrectly.

naviframe: Fix to reset pushing flag of pushed item.

If multiple items are pushed, then pushing flag of the pushed item is
not reset.
To resolve this problem, pushing flag of the pushed item is reset.

evas/vg: renamed the legacy vg shape apis

Reviewers: Hermet, cedric, jpeg

Subscribers: cedric, jpeg

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

eina_cpu: Fix arm build

commit e2875cefc49768f919919fbfa478f81570168688 accidentally removed
some unrelated headers needed for testing ARM NEON support on linux.

evas-software-x11: Remove unused variable and mark unused parameters

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

ecore_file/ipc: clean up documentation

Summary: move comment from c source to header and adjust ingroup relationship

Reviewers: cedric, jpeg, Hermet

Reviewed By: Hermet

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

eio - put in fix by disabling some fallback code

as per https://phab.enlightenment.org/T4619

fixes 4619

@fix

ecore_x - remove ecore_x_vsync tool as it isnt enabled or used runtime

so ecore_x_vsync as a tool uses glx for nvidia drivers plut "wait for
vblank" extensions to try vsync "sync". the problem is this is flakey
because the drivers may or may not continue vsyncing after screen off
or syspend/resume or vt changes and all the workarounds dont seem to
be reliable, so since this causes this to be disabled, no point
keeping all the code and build stuff around, so remove this "unused
junk" we have in the tree.

Edje_Edit: add forgotten part_item_index API into header

It was already implemented but looks like forgotten to be added into
Edje_Edit header, so we got banch of unused and unavailable API

@fix

eina_list prefetch macros - handle misstyped list ptr by casting

so it seems enlightenment someqhere uses a void * as a list per for
EINA_LIST_FREE ... so do some casing of that type to handle such badness

tests: oops, remove invalid trailing slash

tests: add missing symbols

make check failed on OSX (clang) because of missing symbols.

tests: check for the presence of clock_gettime()

macOS Sierra provides clock_gettime().
Instead of testing for the Mach kernel to re-implement clock_gettime(),
let first check that clock_gettime() is not already declared. Older OSX
versions will have the Mach-only fallback.

@fix

tests: mimic POSIX API in clock_gettime()

Let the clock_gettime() re-implementation have the same signature than
its POSIX definition.

eina: fix macos spinlocks and upgrade API to sierra

eo - eo ptr lookup - do some prefetches to get some micro-speedups

prefetching a bit helps.. a bit like 0.2% or so... but it does help. :)

eina list - make use of prefetch for minor speedups

i see a speedup of about 8% over a series of list walking and freeing
functions given this change. it's a small speedup but still not too
shabby just for some prefetches thrown in. ymmv depending on memory
subsystem, memory speed itself, cpu and architecture.

@optimize

eina - add prefetch macros to map to compiler builtins if they exist

this allows you to portably use prefetch compiler builtins. this adds
EINA_PREFETCH(), EINA_PREFETCH_WRITE(), EINA_PREFETCH_NOCACHE() and
EINA_PREFETCH_NOCACHE_WRITE() macros to do this that are "nothing" if
your compiler doesnt support it. of course it also requires your
compielr compile instructions for your architecture and it can only do
so if the architecture it compiles for has these instructions, so be
aware.

@feat

Evas_Device: Fix API name.

Summary:
It should be evas_device_add_full() in order to follow the EFL
name pattern.

Reviewers: DaveMDS, bdilly

Reviewed By: DaveMDS, bdilly

Subscribers: cedric, jpeg

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

efl: Update Efl.Model to use new Efl_Promise and Efl_Future

Replaced all Eina_Promise_Owner and Eina_Promise with Efl_Promise and
Efl_Future.

eo - goto-ify a chunk iof code moving debug printfs out of hot paths

this moves a lot of error case handling into goto's so the code gets
out of the hot path and this should help expecially since variou
smacros do things like:

   do { char buf[256]; sprintf(buf, fmt, ptr); _eo_pointer_error(buf); } while (0)

   _Efl_Class *klass; \
   do { \
      klass = _eo_class_pointer_get(klass_id); \
      if (!klass) { \
         _EO_POINTER_ERR("Class (%p) is an invalid ref.", klass_id); \
      return ret; \
   } \
} while (0)

so putting quite a chunk of code inside a rare "if this errors"
handler that will cause l1 cache misses and this we don't want, thus
moving stuff in eo core out of hot paths to cut down on overhead. yes
it might not be pretty but it's kind of the right thing at such a core
level of efl. this also does the same to the eo base class as this is
also going to be relatively hot given it's the core of every other
object.

eo - fix error case lock not unlocking with shared objects from coverity

this fixes CID 1363294

eo - remove pointless if found by coverity

fix CID 1363295

eo - silence coverit "leak" on eo init if eoid table tls alloc fails

creating a new tls for the eoid table should just never fail so this
is moot anyway, but it silences CID 1362735

elm: nstate - cleanup edje signal cb

@fix

elementary: refactor elm_atspi_bridge function

In order to avoid the following build warning:

lib/elementary/elm_atspi_bridge.c: In function
  ‘_children_changed_signal_send’:
lib/elementary/elm_atspi_bridge.c:3971:4: warning: ‘idx’ may
  be used uninitialized in this function [-Wmaybe-uninitialized]
    _bridge_signal_send(data, event->object, ATSPI_DBUS_INTERFACE_EVENT_OBJECT

A little refactoring was done to function _children_changed_signal_send.
Actually I wasn't able to see a codepath to use idx unintialized,
but this way is more clear and avoid compiler possible confusion.

examples/elementary: drop generated.h from SOURCES

Remove codegen_example_generated.h from codegen_example_SOURCES
and let it only on nodist_codegen_example_SOURCES and
on BUILT_SOURCES.

Also add dependency between codegen_example.c
and codegen_example_generated.h since it's required
to compile.

Avoid the following build error:
  CODEGEN      codegen_example_generated.c
  codegen_example.c:26:39: fatal error: codegen_example_generated.h:
      No such file or directory
  compilation terminated.
  Makefile:4960: recipe for target 'codegen_example.o' failed

examples/ecore: add example suffix to ecore_evas_vnc

Make its name similar to all the other example
files on this folder.

configure: show if vnc server was enabled or not

So we can see by configure log if it should be working =)

examples/ecore: fix vnc example build

Fix missing dependency.

Get rid of the following error:

/usr/bin/ld: ecore_evas_vnc.o: undefined reference
  to symbol 'ECORE_EVENT_MOUSE_BUTTON_UP'
src/lib/ecore_input/.libs/libecore_input.so.1: error adding symbols:
  DSO missing from command line
collect2: error: ld returned 1 exit status
Makefile:2306: recipe for target 'ecore_evas_vnc' failed

data: elm: images: fix double data file definitions leading to races

For a while now I had strange races during make install on Jenkins where some
image files either already existed or chmod was called on files not there yet.

It took some digging but it turns out commit
8dcd5207cc61e604efc942a62137bdeda4060aa0 broke this as a side effect. The black
magic that already installed the files JP refers to is sitting in
data/Makefile.am. It is plain autofoo stuff and just includes the images
Makefile to get the list of files.

JP was correct though that the glayer files have not been installed into the
correct subdir. Fixing this as well here.

edje_cc: don't warn when image attributes are missing in inherit_only group

Summary:
The inherit_only groups could be used for template of other groups.
So, even if it has image part, it dose not need to have image attributes.
@fix

Test Plan: N/A

Reviewers: cedric, conr2d, jpeg, raster

Reviewed By: raster

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

update gitignore to ignore eo_dbug output script from .in file

ector: make the ector objects as shared object after eo changes.
 As ector objects are acessed by draw thread we need to create it as
 shared object in order to access it from other thread.
 Note: there is some performance lag...

Summary: make ector object as shared eo object to acess from other thread.

Reviewers: cedric, jpeg, raster

Reviewed By: jpeg, raster

Subscribers: cedric, jpeg

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

elementary: remove unused part in segmentcontrol theme

The empty ("") state was never used, and is actually just an alias to
the "default" state, so let's just remove it.

elementary: fix invalid state in segmentcontrol

This fixes runtime errors such as:

ERR<29027>:edje lib/edje/edje_calc.c:756 _edje_part_description_apply()
Cannot find description "normal" in part "text2" from group
"elm/segment_control/item/default". Fallback to default description.

The "normal" (non-existant) state corresponds to a disabled state,
which is provided by the "default" state.

edje_program: add NULL checking

eo id and shared domain objects - do locking properly and better

so there were a few issues. one we had a spinlokc on the eoid table
for shared objects AND then had a mutex for accessing those objects
(released on return from any eo function). BUT this missed some funcs
like eo_ref, eo_unref and so on in eo.c ... oops. so fixed. but then i
realized there was a race condition. we locked the eoid table then
unlocked with our pointer THEN locked the sharted object mutex ...
then unlocked it. that was a race condtion gap. so we should share the
same lock anyway - if it's a shared object, grab the shared object
mutex then do a lookup and if the lookup does not fail, KEEP the lock
until it is released by the return from eo function or by some special
macro/funcs that released a matching lock. since its a recursive lock
this is all fine. as its also a universal single lock for all objects
we just need the eoid to know if it's shared and needs locking based
on the domain bits. so now do this locking properly with just a single
mutex, not both a spinlock and mutex and keep the lock around until
totally done with the object. this plugs the race condition holes and
goes from 1 spinlock lock and unlock then a mutex lock and unlokc to
just a single mutex lock and unlock. this means shared objects are
actually truly safe across threads and only have the overhead of a
single recursive mutex to lock and unlock in every api call.

elementary: fix copy & paste error, thanks Dave.

elm: rename globally used variable 'ret' to prevent shadow warnings

please do not do this.

ecore_evas_wayland: Add NULL checking for surface

Summary: Need to NULL check before using surface like other cases.

Test Plan: N/A

Reviewers: jpeg, raster, ManMower, devilhorns

Reviewed By: devilhorns

Subscribers: cedric

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

elm_panel : Fix bug with set scrollable panel orient.

Summary:
1. Fix wrong logic of ELM_PANEL_ORIENT_RIGHT case in _state_sync.
2. Move pre calc function in anim_cb to sizing_eval function.
   This code have caused problem that scrollable panel is not animated during drawer open and close.
3. Maintain content size ratio when orient is changed.
4. Freeze scroller when orient is changed.
   This code will need to change orientation when drawer opened.

Reviewers: cedric, eunue

Subscribers: cedric, jpeg

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

evas sw x11 - disable the loop begin/end that deadlocks evas

this is quick to get evas working again. this disables part of
7e2d700d06ff970d68ae208d59bca76f6ea07467 that creates deadlocks. this
thread waits on mainloop. mainloop waits on this render thread. BOOM.
deadlock.

ecore_cocoa: upgrade API to macOS Sierra

Since macOS 10.12, several enumarations have been deprecated in favor of
new ones, with more meaningful ones, which are defined in SDK 10.12.

efl configure - show that systemd is enabled by default

autotools: correct the configure help for systemd

by default, systemd on linux is enabled. mention this in help

Merge branch 'devs/iscaro/ecore_evas_vnc_multiseat'

This series adds the Ecore_Evas multi-seat VNC support
using the software X11 backend.
This implementation tries to mimic the Wayland's multi-seat support.

This series also introduces two new kinds of EFL events, which are:
 * EFL_CANVAS_EVENT_DEVICE_ADDED - Which is emitted every
   time an Evas_Device is created.
 * EFL_CANVAS_EVENT_DEVICE_REMOVED - Which is emmited every
   time an Evas_Device is removed/deleted.

The new events are useful when one wants to monitor how many
and what kind of devices are connected to the system.

Patches by iscaro.
Differential Revision: https://phab.enlightenment.org/D4295

@feature

Examples: Add an Ecore Evas VNC example.

This commit adds an Ecore_Evas examples that can be used to test
the VNC multi-seat support.

Ecore_Evas_X: Dispatch Ecore_Events for VNC clients.

This patch adds the support for Ecore events from a remove
VNC client. Every time it happens a VNC mouse move/click/wheel or a
VNC keyboard event an Ecore event event will be created and dispatched.

Ecore_Input: Add Efl_Input_Device in Ecore_Events structs.

The Efl_Input_Device will contain the device which originated the event,
this will be useful for dealing with multi-seat environment. When the device
is NULL it means that the event originated from the default seat.

Ecore_Evas_X: Add VNC draw support.

This patch adds the support to draw the X11 screen contents to
all remove VNC clients.

Evas Software X11: Add a callback that informs the current screen content pixels.

This is necessary in order to implement a VNC server. Using this
callback the VNC server will be able to draw the current screen to the
VNC clients.

Evas_Device: Use efl_del() in evas_device_del().

The Evas_Input_Device should be have its parent set to NULL, otherwise
Eo might complain.

@fix

Evas_Device: Add a new event for added and removed devices.

This commits adds two event types for EFL_CANVAS. Every time
a device is added or removed the matching event will be emitted.

Evas_Device: Add evas_device_full_add() API.

This will make it simpler to add a Evas_Device with its fields already set.

Evas_Common: Fix typo.

@fix

Efl_Input_Device: Implement efl_input_device_parent_set().

This commits moves the parent_set implementation where it should be.

po - update

efl - fix build to build multip.edj from the right edc source

the makefile was wrong making multip.edj for the multitouch test in
elm the wrong src thus breaking the crosshairs. this fixes that again
so the test is right.

@fix

ecore: add a prototype ecore_thread helper with Efl_Promise/Efl_Future coupled.

elm_atspi_bridge: Active descendant changed signal not sent issue fix.

Summary:
Active descendant changed signal send function checks for wrong object,
hence always returns with out actually sending signal.

The state is set to pd->object_broadcast_mask and not
pd->object_children_broadcast_mask in _set_broadcast_flag function.

Test Plan:
Send the active descandant changed signal from any widget, but signal
is not sent.

Reviewers: kimcinoo, lukasz.stanislawski, cedric

Reviewed By: cedric

Subscribers: jpeg, govi, rajeshps

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

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

elementary: set the extension on the extension, not the overlay !

@fix

This is a port of D4317

elementary: Fix potential resource leak

If we fail to allocate memory for savedtypes.types here, then the
"known" array would end up leaking. Defer creation of 'known' array
until after savedtypes.types is allocated, this way we don't leak.

Fixes CID1363216

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

Ecore promise: Renamed shadow variable.

Please everyone, use -Wshadow and fix your damn wranings.

elementary entry: send a signal to edje for notifying scrollable mode

Summary:
If entry edje should be changed according to scrollable mode,
the signal will be used. The following signals are added.
"elm,scroll,enable"
"elm,scroll,disable"

Test Plan: N/A

Reviewers: raster, tasn, herdsman, cedric

Subscribers: minkyu, jpeg, akanad, z-wony, Blackmole

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

ecore_wl2: fix array overflow

fixes CID 1363215

ecore_wl2: support parallel receiving of different mimetypes

The mimetypes are now populated with the event and the data.

This also fixes CID 1363217

ecore_wl2: fix null dereference issue

if we dont get the userdata of a offer something in the protocol went
very very bad, we should not emit there a error, just return.

Also if we are getting a null offer we should dereference the nulled out
offer object.

This fixes CID 1363214

elm_config: Remove unnecessary NULL check.

Since _config_load() is called and _elm_config is not initialized, NULL
check for _elm_config is not necessary.
The unnecessary NULL check causes false alarm because _elm_config is not
checked in other cases.

eina_matrix/quad: clean up documentation

Summary: adjust grouping to show reference correctly

Reviewers: Hermet, cedric, jpeg

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

eo ptr indir - handle unlock on error with shared eoids correctly

this fixes the error handling for shared objects which would fail to
unlock on lookup error.

elm: Update toolbar test to show fix

elm_toolbar: Show icons in icon only mode

When passing NULL as label the icon would get hidden.
@fix

efreet - fix command generation by fixing string buffer expansion

so by chance i discovered efreet is doing bad things(tm) when
expanding/appending to string buffers to generate commands based off
desktop files. the string append basically was buggy, so fixed it by
making it a lot simpler and more obvious and now reliable.

@fix

Emotion: modify the playback finish behavior as before

The callbacks are never invoked due to sd->play equal to 0. The
function _emotion_decode_stop is called before and resets this field.

Before the change to Efl.Canvas.Video, sd->play was not checked.

ecore_wl2: utilize the event window_ids

if a dnd operation enters a surface the window_id is clear, so a offer
is always specific to a window. If we have a source we try to fetch the
id from the focus/prevfocus or keyboardfocus.

This fixes dropping into a efl app which never got focus before.

elm_cnp: handle format of a drop correctly

the format can also be the result of mutliple or´ed values.
The new code now also uses the same mimetypes to format type relation
than the selection code.

This fixes dragging onto a container with multiple formats.
Spotted by ApBBB while dragging from ephoto to terminology.

This also fixes T3320

eo op resolv - do micro optimization to move init to goto blob

as per other recent benchmarking, moving rearely run code (in this
case code to init the op etc.) out of the l1 cacheline prefetch inot a
blob of code at the end of the function where we goto and goto back
again should provide decent-ish speedups for the resolv cache in
avoding this code. yes it makes the code less pretty to read but at
this really low level hot path ... evil things must happen to get the
speed we want/need.

eio model: use empty mime type when efreet is unable to determine it

FIX T4502

@fix

Add updated drop part for the new pager gadget.  This will be required for the new pager to work.

ecore-drm2: Use Atomic State to enable/disable an output

If Atomic Modesetting is supported, we can use that to enable/disable
a given output.

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