ecore_evas: switch main animator to use the first windows source for tick.

eo: return a value when adding or removing callback to help the caller manage state.

edje: switch to use per window animator.

ecore_evas: optimize triggering of animator to happen only when someone is listening.

ecore_evas: introduce support for per window animator trigger.

This code is currently only using the older fallback code and not any
new event source, so all animator on all window are still triggered
whatever the case are.

evas: make evas object an animator provider.

evas: make evas a provider of animator.

efl: add Efl.Core.Animator interface.

eo: fix tests to actually account for callback,add and del signal to be @hot.

This example of code is a perfect demonstration of why we should have make
this events @hot from the beginning. As you can see, we ended up being unable
to detect reliably callback being registered and unregistered. Leading to an
negative accounting of callback being unregistered, making this signal unusable
in any scenario.

eo: mark both callback,add and callback,del as @hot.

This events needs to be hot to remain unfreezable or it is impossible
for anyone to track all callback setting/unsetting safely.

evas: move canvas event to be declared inside proper eolian file.

ecore: remove useless ecore_lock/unlock.

eolian: add parsing and generation of hot events

Unfreezable events can now be marked @hot.

@feature

Revert "Revert "Revert "revise edje_object_size_min_restricted_calc() logic."""

This reverts commit ade138a3d13f7cd1606d0c80cf40fa71cd1d983a.

Revert this commit because this basically causes e to hang solid if
you mousover ibar (and the window list popup slides in then i think
that tyriggers it). this is pretty bad.

evas/canvas: fix an incorrect sizing of smart object bounding box.

height of the children b-box should be arranged by parent y-pos.

Signed-Off-By: KeuckDo Bang <keuckdo.bang@samsung.com>
Signed-Off-By: MinKyu Kim <imetjade.kim@samsung.com>
@fix

Revert "Revert "increase recalculation_cycle.""

This reverts commit 9efdf309ce421affc0fdf87960df1d4c426217bd.

Revert "Revert "revise edje_object_size_min_restricted_calc() logic.""

This reverts commit b62d656dbe2a4b4a6b0170526db33ad839cd4aa4.

push it again since 1.17 was released.

Evas textblock: Use a common thickness and position at a underline.

Summary:
If a underline is drawn with seperated thickness and position, it doesn't look good.
It will take the thickest and the lowest underline.

@feature

Test Plan:
Set the following markup text in Evas Textblock.
<underline=on underline_color=#fff><font_size=20>Markup text <font_size=50>with</font_size> underline tag</font_size></underline>

It shows the underline is split to 3 underlines with different thickness and positions.
Commonly, underline has to be drawn with same thickness ans position per each line.

Reviewers: woohyun, herdsman, tasn

Reviewed By: tasn

Subscribers: jpeg, raster, subodh6129, cedric

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

Evas Text/Textblock: Use locale for lang as default.

Summary:
The configuration files for Fontconfig can describe
how font list is made according to language information.
EFL also set the language for each Evas textblock styles
and used for loading font list.

But, this is inconvenient to use if we want to apply language
for loading font list according to system-wide locale information.
This patch will apply locale information for font list if there is
no specific language in description.

And it also add [lang=auto] for Evas Textblock.
auto - It loads locale for language.
none - It disables language.

@feature

Test Plan: N/A

Reviewers: woohyun, herdsman, tasn

Subscribers: jpeg, cedric

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

edje: Reduce duplicated item obj creations and deletions.

Summary:
When text is changed, all of objects for item tag are deleted
and recreated. It is unnecessary work and can cause performance
issues. Actually, many of application developers wonder why
item provider callback functions are called every text changes.
@fix

Test Plan:
Run elementary_test -to "entry emoticon"
When you make a very little change on text,
36 emoticon objects are recreated.

Reviewers: woohyun, jaehwan, herdsman, tasn

Reviewed By: tasn

Subscribers: cedric, jpeg

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

configure: Switch to dev mode again. Merge window for 1.18 is now open.

release: Update NEWS and bump version for 1.17.0 release

Revert "revise edje_object_size_min_restricted_calc() logic."

This reverts commit 32c12f60c99cbc363c1f65fcb9c7915398be087e.

rollback until release 1.17

Revert "increase recalculation_cycle."

This reverts commit 524fd2f6fc9afdd44fd371d99179d44a28bfd28a.

rollback until release 1.17

Revert "evas/canvas: simple optimization."

This reverts commit ca124ce5cea162f130eeb80b6c95ee3fadd64761.

rollback until release 1.17

evas/canvas: simple optimization.

We don't need to reorder the clist here as far as I reviewed.
Because, the clist nodes will be calculated all in one.

If I'm wrong please ping me.

increase recalculation_cycle.

256 is too short.
entry can be much larger than this calculation cycle.

revise edje_object_size_min_restricted_calc() logic.

This change is to fix the 4000x4000 limitation issue in edje.
Previously, the sizing calc logic was experimentally confirmed with 4000x4000.
It worked somehow but was not perfect. This change may not be perfect either but
it gets rid of the limitation problem without any performance drop in
computating. Maybe it would be better than before.
I'd rather push the temporary version before we confirmed no side effects.

@fix T2978

eldbus: Correctly read/write boolean to eina_value

DBus bool type is a uint32_t not a uint8_t, just changing how read
and write, not chaging how it is stored in eina_value to save some
bytes of RAM.

This fix the stack overflow pointed out here: https://phab.enlightenment.org/T3089

@fix

Eina_Value: do not mix declarations and code

Reviewers: cedric, jpeg

Reviewed By: jpeg

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

evas: Fix typo in previous clip issue fix

See 100a7006b855fe4da2c2a5c1c4ec58e06b3a0bc0.

This typo actually caused some really rare crashes to happen
(even rarer than the one I fixed originally). Crashes nonetheless.

eeze: Fix eeze_udev_find_by_subsystem_sysname function

This function was severly broken when used for finding devices of a
particular subsystem. Previously, this function would return every
device in the system, regardless if subsystem or sysname was passed in.
This patch fixes the issue by adding udev_enumerate matches before we
scan for devices. This way, when we enumerate the scan results, we now
get only devices that match the subsystem (if supplied).

@fix

Signed-off-by: Chris Michael <cpmichael@osg.samsung.com>

eina: fix invalid prototype

The prototype means "unspecified arguments". However, the implementation
specifies no parameters (void).
GCC's warning -Wstrict-prototypes complains a lots about this old-style
prototype, spamming the console when compiling more than one compiling
units including Eina.

@fix

ecore_evas: trigger pre render function as expected.

This should fix exactness and elementary documentation screenshot bug discovered
by commit 6149fd7a0a9ef3f9495270cf1c4ab7727a30f21c in elementary.

Evas clip: Fix rare crash in _render_pre_clipper_change

Use delete callback instead of direct call to clip_unset,
which lets us know that clip_unset() is called during the
clipper's deletion, as opposed to a simple call.

We can then make sure that the previous object state does
not point to invalid data anymore.

Here is a scenario that could have crashed:
- load and show an edje object, hide it
- change its theme or style
- show it again

@fix

evas: Fix explicit NULL dereference

This patch fixes an Coverity issue that if 'dst' gets set to NULL (as can
happen above) then this eo_do call may segfault as it is directly
accessing 'dst->buffer'.

@fix
CID1347415

Signed-off-by: Chris Michael <cpmichael@osg.samsung.com>

ecore_imf/scim: fix uninitiazed value issue detected by valgrind

evas: Fix segfault when trying to determine overlay usage

Previously, _evas_render_can_use_overlay would segfault here when
trying to make use of an Evas_Video_Surface. This is because eo_tmp
variable was never reassigned to be the smart parent before we tried
to get eo data from it.

@fix

NB: Thanks Frenchie !! ;)

Signed-off-by: Chris Michael <cpmichael@osg.samsung.com>

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

efreet: add locking for efreet_desktop_x_field_get()

in a threaded environment, reading from a hash is not atomic and is
subject to race conditions

@fix

ref 39417cf0ea4e457fbc8d50d120bb8ac776aa366f

efreet: reject invalid "" data for Efreet_Desktop->x hash

ref T3096

@fix

edje: Fix potential NULL pointer dereference

It is possible that evas_object_data_get here does not return an Edje,
so check the return value. This fixes a coverity reported issue.

@fix
CID1349866

Signed-off-by: Chris Michael <cpmichael@osg.samsung.com>

ecore_evas_extn: Add safety checks, fixing a crash in elm_test

See T2835 (my crash was different from the one reported).

The buffer may be initialized after the first render_pre, due to
whatever timing issue. Check in elm_test "Window Socket" and then
open several "Window Plug".

@fix

Ethumb: make the emotion plugin correctly use the edje frame if requested.

The emotion plugin was lacking the ability to use the edje frame as expected,
this patch add this ability, so now video thumbnail works like the normal ones.

This commit is a bit of a new feature, but I consider this a @fix as now
the ethumb_frame_set() API works as expected, also the implementation
is so simple that I think it can go in without any problems.

   evas: Updated documentation of evas_object_image_save.

Evas filters: Change FIXME comment in test case

   evas: Updated documentation of evas_object_image_save.

edje_cc: remove redundant comments which makes reference seem weird

Summary:

Group.Parts.Part.Description.Domain @edcsubription is in wrong place,
(actually that is not necessary one.)
so it makes Group.Parts.Part.Description.Text block seem weird.

Reviewers: jpeg, cedric

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

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

Evas filters: Re-enable transform test case, remove dead code

Now that transform is fixed, this should be reenabled.

Evas filters: Fix transform filter (crashes & blend)

out was simply not initialized if the source and destination
were the same. The COW flag is required here to separate input
and output properly. Also, the source & destination pointers
were badly calculated and could overflow.

Instead of using memcpy this filter was supposed to use the
blend functions. This patch also fixes that.

In order to make this filter actually useful (think text reflection
on a flat surface), more information needs to be passed to the Lua
script, such as the text ascent, descent, etc...

Evas filters: Fix inconsistency in transform API

This is a minor change, makes 'output' the default target for
the transform filter. This is consistent with all the other
filters. This is not exactly a bug fix but it doesn't break
compatibility with the earlier explicit form and improves the API
for 1.17.

Evas filters: Disable debug mode!

Ooops, I must have let this in by mistake.

Eolian Cxx: Fix warnings related to deprecated EO_EV_DEL

Switch the generator to EO_BASE_EVENT_DEL.

ector: fix potential crash if cairo is not found on the system.

@fix

T2938

ecore-drm: Unify and cleanup tty code

This patch brings the tty code more inline with what the logind code
Was doing. It also fixes the return value from the vt_signal handler,
and opens the tty in non-blocking mode now.

@fix

Signed-off-by: Chris Michael <cpmichael@osg.samsung.com>

ecore-drm: Don't unset logind variable when disconnecting a device

For a case where we have more than one device open, if we set the
'logind' variable to false, then the second device would never close
properly.

@fix

Signed-off-by: Chris Michael <cpmichael@osg.samsung.com>

ecore-drm: Destroy any existing eeze watch when we free the device

If a user frees a device without closing it we could end up with a
dangling eeze watch. This small patch just makes sure to delete the
watch if it exists

@fix

Signed-off-by: Chris Michael <cpmichael@osg.samsung.com>

ecore-drm: Refactor logind code to use tty functions

This patch cleans up some of the logind code to make use of the
existing tty functions as they both did the same thing

@fix

Signed-off-by: Chris Michael <cpmichael@osg.samsung.com>

ecore-evas-drm: Cleanup drm outputs during shutdown

As we no longer cleanup outputs inside the ecore_drm_device_free
function (see previous commit), we should be cleaning up outputs here
as they are created during the _ecore_evas_drm_init function.

@fix

Signed-off-by: Chris Michael <cpmichael@osg.samsung.com>

ecore-drm: Cleanup ecore-drm shutdown routine

This patch addresses an issue where when closing Enlightenment, the
shutdown procedure would previously end up calling the same functions
twice. These functions should be called from the ecore_evas drm
shutdown routine as the sprites, inputs, outputs, etc are all called
from the ecore_evas drm init routine.

@fix

Signed-off-by: Chris Michael <cpmichael@osg.samsung.com>

Eo: Actually mark EO_EV_ as EINA_DEPRECATED

This should add compilation warnings whereever they're used.

Eo: Replace deprecated EO_EV_ by EO_BASE_EVENT_

Evas GL: Actually save the checksum inside the file

This would be cleaner than adding it as a suffix to the filename.
Use simple write and read_direct for simplicity.

This is an improvement on 3dcd903a537e29342121c and should make sure
everyone is actually using the latest version of the shaders even
if they track master or the alpha/beta releases.

[efreet] document NULL return possibility.

If file is not found then NULL is the fallback

build: fix some last corner cases for the hyphen dir being set correctly

Making sure here that we really define EVAS_DICTS_HYPHEN_DIR after all possible
changes have been made to it and we have the final result.

Revert "edje_cc: fix detecting missing part for insert_before/after"

This reverts commit 2ca99b8b68bd52ce187c2da54944144f12afb0b6.

This breaks elm compile:

  EDJ      default.edj
edje_cc: Critical. In group "genlist_arrow": Unable to find part "sel_bevel" for insert_after in part "arrow_img_l".

Evas: Avoid saving the shaders cache at every shutdown

Check the right flag.

Evas: Discard shaders cache when the code changed

Since we now have only two shader strings, this is trivial to do:
hash the strings and add them to the cache filename.

This will allow people using 1.17.0-alpha, etc... to discard their
old shaders cache automagically and use the latest version of the
shaders (because alpha is not in the filename).

If we end up adding more runtime generated shaders, we might need
a better strategy, but this should be good enough for now.

Ector: Fix warning on Windows (use PRIx64 to print uint64_t)

Thanks @vtorri for pointing this out.

build: sane default setting for hyphen dir

Without this compilation would fail with undefinded EVAS_DICTS_HYPHEN_DIR if
--with-xattr-tests-path was not given but hyphen enabled. Use a sane default
(/usr/share/hyphen/) to avoid this. Customization is still possible.

build: add hyphen as evas feature to be displayed in summary

Having it as a feature for evas here makes sure we can see if it is enabled
or disabled on the configure summary output.

Evas: Actually fix this 'afill' thing

This mostly reverts 448720fed4da252fa5eefa02ae37c79c5fb63499

After my previous patch, semi-transparent windows would render incorrectly
in E. The AFILL flag should make sure the source texture is "opaque" (ie.
if it's not marked as having an alpha channel), and masking or color
multiply should then be applied later on.

Sorry for the mess...

Evas: Fix shader flag 'afill' (forcing alpha to 255)

Apparently the shaders refactor broke the afill select logic, and it
no longer applied to objects where it should have (eg. opaque windows
in E).

edje_cc: fix detecting missing part for insert_before/after

Summary:
Detecting missing part for insert_before/after is broken.
This patch makes the feature work and clarifies error message,
and removes redundant internal function.

Reviewers: cedric, jpeg

Subscribers: jpeg

Maniphest Tasks: T2513

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

edje/edje_edit: make posible to unset the program filter part and state

@fix

Ecore: Remove all temporary files after running make check

Eina: Add proper test case for eina_mk[ds]temp

Evas GL: Also fix surface cap cache

This follows the fix in eina_file_mkstemp. Not sure how it could
even work before.

@fix

Eina: Convert eina_test_file.c to UTF-8

It was encoded as ISO-8859 (Latin-1) before.
Maybe this will break these checks on Windows, but all of EFL APIs take
UTF-8 strings, so it doesn't make sense to have ISO-8859 here.

elua: Fix use of eina_file_mkstemp after previous patch

The API (that didn't work in the first place) was used wrongly
as it was assumed its behaviour was the same as mkstemp (duh!).
It turns out eina's version doesn't replace the input string but
returns a tmpstr instead.

@fix

Evas GL: Fix shader binaries cache

It was completetly broken due to a misuse of mkstemp.

Eina: Fix eina_file_mk[ds]temp when a path is passed

If the template is a path, mkstemp and mkdtemp would fail
miserably as they would try to create a file inside
/run/user/1000//path/to/file.XXXXXX even if the path did not
exist.

This patch fixes that by creating temp files inside the sys temp
dir iif the templatename is just a basic name without path
separator.

@fix

ecore-x: add safety checks for _ecore_x_disp in some functions

xlib immediately crashes upon being passed a null DISPLAY object,
so every function in ecore-x should likely have safety checks such as
these. an easy project for anyone who has some free time

@fix

release: Update NEWS and bump version for 1.17.0-beta2 release

eina_cxx: make sure simple.eo lands in the tarball in all cases

This would fail if you have not enabled tests and run distcheck.

Fix build if openjpeg-2 is installed

libopenjp2.so on Debian belongs actually to openjpeg-2.1, while
libopenjpeg.so belongs to openjpeg-1.5.

The function opj_set_default_decoder_parameters exists in both versions.
Thus another unique to openjpeg-1.5 function must be chosen to check,
whether libopenjp2 is a right one.

Fixes https://phab.enlightenment.org/T3064

ecore imf: fix imf module load hangs when not even configured

so on some distributions scim tries to connect and hangs waiting for
connect to work. this is a problem for just loading any imf module
unless EXPLICITLY configured, so disable using an imf module unless it
is specifically configured to work. this fixes this hang. this is a
workaround though, but what else are we to do?

@fix

embryo: change the log catagory of printf in embryo

If the catagory is DBG, we should rebuild efl to print the message
by using printf in edc. So change it to use it more convenient.

@fix

win32: Fix compilation due to recent changes in Evil

Summary:
dlfcn.h is not available anymore on Windows, Evil provides all the
necessary declarations.

Reviewers: cedric, jpeg

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

@fix

Eo: Spell "Class" correctly when printing error.

Leave variables named Klass so it behaves with syntax highlighting and doesn't confuse programmers.  However when Erroring, the message should be spelled correctly with "Class".

ecore: check eina_main_loop before lock.

Summary:
 - EINA_MAIN_LOOP_CHECK_RETURN should be called before ecore lock
   because this may return without ecore_unlock.
 - remove EINA_UNLIKELY(!eina_main_loop_is()) which is redundant.

Reviewers: jpeg, jaehwan, cedric, raster

Reviewed By: raster

Subscribers: raster, conr2d, cedric, jpeg

Projects: #efl

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

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

ecore_drm/ecore_wl2: logging: remove errant usage of %m

Summary:
printf %m stringifies and prints errno.  I've tried to remove its use
anywhere that the immediately preceding function might not set errno
or is a complicated function for which knowing errno doesn't really
give any useful information.

I've left a few of the drmMode calls because they're just wrappers
around ioctl, which legitimately sets errno.

@fix

Reviewers: zmike, devilhorns

Reviewed By: zmike, devilhorns

Subscribers: cedric, jpeg

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

ecore_wayland: move check for pointer capability

Summary:
We don't need to work for cursor if system doesn't have pointer devices.
Since this function is called by elementary(e.g. entry) also, we move a check condition into the function.

@fix

Reviewers: raster, zmike, devilhorns, ManMower, gwanglim

Subscribers: cedric, JHyun, input.hacker, jpeg

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

eina_test: Add test-case for eina_quaternion_transform

Reviewers: stefan_schmidt, cedric

Subscribers: jpeg

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

ecore_evas/tests: add tests for Ecore_Evas_Cocoa

ecore_evas/tests: remove trailing whitespaces

Cosmetic changes only

evas: follow file rename to keep distcheck working

In 8b00eb1dd1241290025e041973f6c8748b3e8fe4 the file was moved but the rename
forgotten in the Makefile.

Evas 3d example: Print message for usage

Evas: fix example for convex hull

Summary: Made new models with no normal component to print
сorrect print message for testing not only by visual result.

Reviewers: cedric, raster, Hermet

Subscribers: jpeg, artem.popov

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

Examples: Fix compilation of elocation ex (out of tree)