ecore-drm2: Remove newly added atomic API functions

As we need to do atomic commits on a per-output basis, these 2 newly
added API functions can go because these functions did one atomic
commit for all outputs

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

ecore-drm2: Remove #ifdef for HAVE_ATOMIC in plane_assign function

As there is nothing inside this function which requires any Atomic API
calls, this #ifdef can be removed and the function can then still be
used to assign Primary planes for non-atomic use cases.

@fix

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

ecore-drm2: Minor formatting fix

NB: No functional changes

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

ecore-drm2: Fill output states regardless if atomic

This commit fills in various output 'state' structures during creation
so that those state structures can be reused for pageflip handling
even if Atomic support is not enabled.

@fix

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

ecore-drm2: Enable filling state structures without atomic support

This commit enables the ability to fill our state structures even if
atomic support is not enabled. This will allow us to reuse those state
structures for dealing with pageflip in both the atomic & non-atomic
use cases.

@fix

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

ecore-drm2: Enable reuse of existing state structures for non-atomic

As there is nothing 'atomic' specific in these structures, we can move
them outside the atomic ifdef and make use of them for handling
pageflip for both atomic and non-atomic use cases.

@fix

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

ecore-drm2: Minor formatting fix

NB: No functional changes

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

eo: fix missing event emission

There have been cases where the logic of _event_callback_call break'ed
too early in the event submission.
Reason for that was the line ((const unsigned char *) desc -
(const unsigned char *) it->desc) producing a overflow.

This means the if statement

if (!legacy_compare &&
   ((const unsigned char *) desc - (const unsigned char *) it->desc) < 0)

was true while the pointer desc was smaller than it->desc, which means
the event subscription got aborted, even if it should not.

This turned out on two 32 bit maschines. And led to not rendering apps
anymore.

It was introduced by commit in 605fec91ee7.

@fix

efl_ui_focus_object: that definition is not needed.

efl_ui_focus_manager_sub: replace a wrong comment

efl_ui_focus_manager: specify the event in the same way evas does

efl_ui_focus_manager: the caller gives away the ownership of this list

efl_ui_focus_manager: elaborate on what the last object is

efl_ui_focus_manager: make documentation clear

jpeg pointed out that the documentation here is missleading and unclear.

efl_ui_focus_manager: give feedback if setting root worked or not

so a potential caller can make sure if the set of root worked or not

efl_ui_focus_manager: rename api from geometry to focus_geometry

otherwise we would clash with efl.gfx.

eolian: allow future<void> for value-less futures

elm_code: fix make check, sorry

Evas_Common: Fix punctuation errors and typos

Reviewers: devilhorns

Reviewed By: devilhorns

Subscribers: devilhorns, stefan_schmidt, cedric, jpeg

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

evas: Rewrite Evas Eo documentation

Summary:
Cleans up grammar, simplifies wording, and elaborates on some details
for better clarity.  Assume the reader will already have a basic
understanding of reference counting and 3D graphics in order to avoid
overexplaining these concepts.

Subscribers: cedric, jpeg

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

examples/evas: Fix missing space before -lm in compilation directions

Subscribers: cedric, jpeg

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

evas_canvas3d_mesh: Fix typo

Summary:  Signed-off-by: Bryce Harrington <bryce@osg.samsung.com>

Subscribers: cedric, jpeg

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

elm_code: Expose API for inserting text at cursor

widget operations use NULL terminated strings, backend does not.
Refactor some selection code so all selection deletions trigger
undo events (except where they should not).

elm_code : LINE_APPEND Render fix

Summary: @fix

Reviewers: ajwillia.ms

Reviewed By: ajwillia.ms

Subscribers: cedric, jpeg

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

tests: elm: make sure the new focus_common.h file gets into dist

Needed to run check from the dist tarballs.

ecore_imf: Add ecore_imf_context_prediction_hint_set API

Summary: Added a new api to send the prediction hint string to IME.

Test Plan: Tested in Tizen device

Reviewers: woohyun, id213sin, jihoon

Reviewed By: jihoon

Subscribers: cedric, jsuya, z-wony, jpeg

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

docs: remove wrong line-height

there was a 'line-height: 0%;' that resulted in unreadable edc code in
the Edje Data Collection reference documentation page.

Thank you Jeremy Suntheimer for spotting that!

fix T5438

ecore_evas: make ecore_evas buffer asynchronous.

evas: only call output function on initialized output.

ecore_evas: do timeout reset on RENDER_POST for future asynchronous rendering.

ecore_evas: properly warn when we are already rendering.

ecore_evas: refactor ecore_evas buffer render logic.

First step into introducing async rendering for ecore_evas buffer.

elm_code: Fix selection,start signal

@fix

eio: improve Kevent (FreeBSD et. al)

Summary: Adds initial support for file/directory creation and deletion detection.

Reviewers: raster, cedric

Reviewed By: cedric

Subscribers: raster, jpeg

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

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

bump minimum version requirement of freetype2 to 16.2.10 which equals release 2.5.0.1

For this one I feel like I earned an archaeologist medal. :-)

This is related to the commit raster made in aa92cddb to accommodate for older
freetype version. The TT_INTERPRETER_VERSION_35 define was actually introduced
in the release 2.5.0 which was released in June 2013 which means it had almost
4 years to get picked up. I would consider this safe to be a minimum dependency
for current efl versions.

The trick here is that the release version does not match the library version
spit out by pkg-config. 16.2.10 equals the freetype 2.5.0.1 release in this case.
Better make a note about it in configure for the next poor soul trying to
understand this.

Thanks again to llelectronics for reporting this.
Fix T5437
@fix

build: bump minimum version requirement of gnutls to 3.3.6

This releases offers the gnutls_certificate_set_x509_trust_dir() API we use in
ecore_con. This gnutls version was release in July 2014 which gave it enough
time to be picked up by distributions.

Thanks a lot to llelectronics for reporting and tracking down the problem.

ref T5437
@fix

efl readme - add libunwind to highly recommended for err bt

efl readme - remove liblz4 from recommendations

it's probably not a good idea as it adds some dirty pages for a tiny
lib that we already supply in our source. if your distro demands you
do this - fine. but i wouldnt recommend this highly.

efl readme - vlc really shouldn't be recommended list if gst is def

recommending vlc and having gts default is probably not good. choose
one or the other ... but not both. so remove from the recommended list
just ebcause it's causing excess deps by packagers apparently

evas font code - build on older freetypes without new definitions

use the actual #35 as:

in freetype and it cant change without breaking abi...

@fix

thread fence code - move elsewhere in file to avoid conflicts

edje_cc: Fix warning caused by conflict between local & global variable

ecore_evas: fix test in the hide callback in Windows module

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

edje: Fix resource leak

The function parse_str returns allocated memory which should be freed
before we exit this function.

Fix Coverity CID1374644

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

edje: Fix resource leak

The function parse_str returns allocated memory which should be freed
before we exit this function.

Fix Coverity CID1374647

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

wayland engines: Don't store evas in engine info

We don't need it anymore.

wayland_shm: Stop using the evas directly to block for async render

Use the new fence function instead.

This makes Cedric less unhappy.

evas: Add a function to block for render thread completion

This is needed by dmabuf engine fallback when it realizes it locally
allocated a buffer, has been rendering to it, but the compositor can't use
it.

So the engine copies its buffer contents into a new wl_shm buffer and
continues from there - however we need to make sure the async renderer
has finished first, so we don't copy a partial buffer.

This allows us to block for all previously submit actions in the render
queue to complete.

gl_drm:Rename import_simple_dmabuf to gl_import_simple_dmabuf

There's also an ecore_drm2 function to import dmabuf as fb, so it's a
little less confusing to be a little more explicit here.

ecore_evas_wayland: Make local functions static

ecore_evas_wayland: Remove unused functions

ecore_evas_wayland: Remove unnecessary function prototypes

ecore_evas_wayland_common: Reorder functions

Try to put these in order so we don't need prototypes

No functional changes

ecore_evas: Add doxygen for ecore_evas_gl_x11_options_new()

Reviewers: devilhorns

Subscribers: cedric, jpeg

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

eo: move all the key_ properties to C

eo: move wref_add/wref_del to C

eo: move future_link to C

ecore: efl_promise: remove superfluous NULL check

f is already checked against NULL in the eina array macro. No need to check
again.

CID: 1362730

eldbus: check return of dbus_signature_iter_next()

CID: 1341856

evas_filter: initialize was_empty to false to have a defined state

It is used further below in an if clause but might not have been
initialized at that point.

CID: 1374276

evas: gl_filters: fix leak of evas image

Make sure we free the allocated use_map evas image before leaving scope.

CID: 1374275

evas: fix logic in evas_events.c

Summary:
when collecting the objects under a mouse pointer,
evas uses the geometry of an object to decide
whether the mouse pointer is inside the area of the object,
which is inappropriate for a mapped object.
so mapped objects don't receive mouse events when they should.
this patch fixes the issue.

Test Plan: A sample code will be added as comments

Reviewers: jpeg, raster

Subscribers: cedric, eunue

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

elm_fileselector: made fileselector styling more flexible

Previously the progressbar in fileselector use hardcoded style name
"wheel", that made unpossible to create different style for
fileselector. This commit made it possible.

@fix

elm_cursor: add style name for sub cursor

Missing style name for sub cursor call the next error output:

ERR<24193>:elementary lib/elementary/els_cursor.c:734 elm_cursor_theme()
Could not apply the theme to the cursor style=(null)

Also this error call extra recalc for cursor hot spots.

Fixes T5408
@fix

Ecore_Evas: use window states and event property to fix ecore_evas_fullscreen_set()

This implies that elm_win_fullscreen_set() is also fixed

@fix

Ecore_Win32: add Property event API

@feature

Ecore_Win32: add ecore_win32_window_state_get() API to retrieve a window states

@feature

Ecore_Win32: update fullscreen state

Ecore_Win32: follow a bit more X code

Ecore_Evas: clean up Windows module

Ecore_Win32: fix information message

Ecore_Win32: fix spelling

edje_cc: Convert anchors to beta feature (require -beta when compiled)

edje_cc: Allow to set float position for anchors

edje_cc: Allow part to be anchored to GROUP and the part named as "GROUP"

edje_cc: Fix warning caused by missing const modifier

efl_ui_focus_manager: check for null return from node_get

Summary:
node_get() can return NULL sometimes.  Add a missing check, and add dox
for the function to document the error return.

>>>     CID 1374434:  Null pointer dereferences  (NULL_RETURNS)
>>>     Assigning: "node" = null return value from "node_get".

@fix CID1374434

Subscribers: cedric, jpeg

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

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

examples/evas: fix spelling

Subscribers: cedric, jpeg

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

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

edje_cc: introduce "Anchors" - easy way to set parts relationship

Test Plan: compile src/examples/edc-anchors.c and run

Reviewers: zmike, raster, cedric, jpeg

Reviewed By: raster, jpeg

Subscribers: raster, barbieri, zmike, SanghyeonLee, taxi2se, Jaehyun_Cho

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

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

evas: do not rely on Evas canvas for Evas Ector engine backend.

elm_box/table: set the order directly when a container is registered

The following situation:
 - A Box in a naviframe, with n children.
 - All added children register to the focus graph with the box as parent,
   order gets set correctly.
 - Naviframe hides this item, so box property tree unfocusable gets set
   to true, it gets unregistered from the focus graph, even every single
   child gets unregistered.
 - The item gets shown - every child and the table are getting
   registered again.
 - Order is not set again, since the box does not get changed
 - Order of the children is mixed up.

This should fix this case since the order is flushed every time the box
gets registered.

efl_ui_focus_manager: safe fields from the node

There could be the case that the item gets freed due to some handling in
a event handler of the event EFL_UI_FOCUS_MANAGER_EVENT_FOCUSED.
So the code now sets the node to NULL after the event is called and
saves the fields that are rfom use later.

efl.net: rename Ip.Address to Ip_Address.

It's unlikely that we'll have other stuff under Ip namespace, also not
that likely to have other than Ip Addresses (to invert it to
Address.Ip), thus make a toplevel entry Ip_Address as suggested by
DaveMDS.

elm_entry: reset 'icon' and 'end' swallow when style changed

Fixes T5397

efl/gfx: Remove unused function

This fixes a warning with -Wunused-function

evas: Fix crash in idle_flush

As reported by raster on the ML. The pointer passed to the
engines is not an engine output but an Efl_Canvas_Output.

itests: elm_focus: rename local variable to avoid clash with main()

tests/elementary/elm_test_focus_sub.c:88:25: warning: ‘main’ is usually a function [-Wmain]

docs: efl_ui_focus_manager: document last missing items in this new file

docs: elm_widget: document new foc us_register method

docs: efl_ui_focus_manager: document focus relations struct members

evas render: Fix issue with map render

Test case:
  elementary_test -to "Gesture Layer"

Just move the pictures around (eg. to the bottom). They could
disappear entirely.

This is because the geometry used was based on the smart
object "bounding box" rather than the mapped output.

@fix

edje_cc: Fix external param smart parser

evas: remove engine.data.output.

This is the first step toward handling multi output. This patch
remove engine.data.output from Evas structure and use an Eina_List
for it instead. It also start moving code around to fetch an output
or an engine context (which are the same at the moment, but will be
split in a later patch).

evas: start setting up output independently of the engine.

evas: evas_render actually need to access that structure.

evas: Fix grammar of which vs. that

Subscribers: cedric, jpeg

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

evas: Fix grammar for which vs. that

Subscribers: cedric, jpeg

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

ecore_evas: Fix grammar for at/to/for/on/which/etc.

Summary:
Fixes some grammar confusion for in that/this, that/which, to/at,
to/for, at/by, etc.

Subscribers: cedric, jpeg

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

Fix spelling of 'alpha channel'

Subscribers: cedric, jpeg

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

evas: round glyph's advance before adding it to pen_x

Summary:
Rounding the sum of glyph's advance could cause inconsistency of
each glyph's positions. When Evas enables Harfbuzz library,
Each glyph's position has to be handled by only nearby glyphs.
But, currently, totally unrelated glyph's advacne could change
other glyphs positions.

ex) 1. "connect."
    2. "Tap here to connect."

You can see different gap between "c" and "o" of word "connect".
It should be same even if there was a different text before the word "connect".
@fix

Test Plan: N/A

Reviewers: raster, herdsman, jpeg

Reviewed By: raster

Subscribers: cedric

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