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)

emile: Added example program for base64 encoding and decoding.

Summary: Signed-off-by: Srivardhan Hebbar <sri.hebbar@samsung.com>

Reviewers: cedric, jpeg

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

ecore_wl2: fix key down event called to often

if many events are coming in, a release event can be skipped. if this happens
the timer continues with the new keycode. so if a new key pressed event
enter checks for the old timer and delete the timer if there is one.

If the timer kicks in and emits additional key down events dont call the
callback which gets called by wl, and emit the signal directly.

ecore_drm: close all devices in shutdown

ecore_drm: check if device is already opened before open and free

check if fd is -1 before opening a device, so a device cannot be opened
twice.

Also check if fd is != -1 before closing it.

eina: add all test cases for eina_vector2.

Summary: All tests passes for eina_vector2 functions.

Reviewers: cedric, stefan_schmidt

Subscribers: jpeg

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

ecore_drm: make sure we include math.h and link against libm for floor()

When we introduced the usage of floor() in ecore_drm_evdev.c we forgot to
include math.h for it as well as linking against libm. This caused linking
failures on some systems while it still worked on others. Better fix it for
good.

Original patch by aerodynamik.

ref T2953

build: check for dlopen() when we build the ecore_evas drm module

We use it to check for libglapi.so.0

Original patch by aerodynamik

ref T2953

ecore_wayland: send mouse_move when sets touch_focus window in cb_touch_down

Summary:
When user touches the screen, ecore_drm generates MOUSE_MOVE event before MOUSE_BUTTON_DOWN.
But when ecore_wayland gets touch_motion before touch_down, touch_focus window is NULL and sending MOUSE_MOVE event fails.
Also in terms of 'touch' input, touch_focus needs be set in cb_touch_down, not in cb_pointer_enter.
So this commit makes sure that ecore_wayland generates MOUSE_MOVE when touch_focus is set.

@fix

Test Plan:
when the application is launched for first time, user clicks on any area of the screen.
We can see the posion of touch event wrong.

Reviewers: raster, zmike, devilhorns, gwanglim, ManMower

Reviewed By: ManMower

Subscribers: jpeg, JHyun, cedric, input.hacker

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

po: update po files

eina: Add test-case for eina_normal3_matrix_get

Summary: Test case fail, need apply that D3563

Reviewers: cedric, Hermet, stefan_schmidt

Subscribers: jpeg

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

eina: Fix possible wrong value due to null in determinant of matrix

Reviewers: cedric, Hermet

Subscribers: jpeg

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

eina: add test case for eina_vector2

Summary: Create eina_test_vector and add first test case.

Reviewers: cedric, stefan_schmidt

Subscribers: jpeg

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

edje: enhance edje_calc circular dependency error message

Ector GL: Remove duplicated functions (eo)

Evas filter: Rename constructor/destructor to ctor/dtor

This avoids a name clash with Eo.Base.

efl: edje: make comments for something we need to break for efl2 not now

make notes about https://phab.enlightenment.org/D2980

EFL: Mark EOLIAN functions as static when possible

This affects mostly evas filters.

Evas: remove extra if from evas_3d_utils.h

Summary: remove extra if from evas_3d_utils.h according to Coverity (CID1339788)

Reviewers: raster, Hermet, cedric

Subscribers: jpeg, artem.popov

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

Edje: enhance embryo stack broken error message

Summary: Enhance embry stack broken error message

Test Plan: Make a run_program run a script

Reviewers: raster, zmike, jpeg

Reviewed By: jpeg

Subscribers: cedric, seoz, jpeg

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

Evas filters: Fix unused variables

Thanks @stefan_schmidt for the report

Evas render: Fix typo (use & instead of &)

Thanks @stefan_schmidt for the report

Evas GL: Reset current surface for all contexts to null when surface is destroyed

When evas gl surface is destroyed, not only the current surface for the current context
should be set to NULL, but current surface for all contexts should be reset.

evas: fix resource file for billboard in evas-3d-shadow example

Summary: Due to evas.canvas3d shader flips resource image.

Reviewers: cedric, Hermet

Subscribers: jpeg

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

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

edje: convert Edje_Text_Class, Edje_Size_Class data type to public

Summary:
APIs iterating works through active text classes and size classes were added,
but Edje_Text_Class and Edje_Size_Class data type are still concealed.
This patch uncovers those data types so as to allow developers to use them.

Reviewers: jpeg, cedric

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

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

ecore-drm: Call inputs shutdown before eina_shutdown

As we end up freeing the input hash inside the
_ecore_drm_inputs_shutdown function, it should be called before we
shutdown eina

@fix

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

Evas textblock: Remove redundant NULL check.

Detected by smatch.

Evas textblock: slight style fixes.

Edje UI mirroring: Fix UI mirroring for GROUP parts.

GROUP parts were not mirrored with the rest of the edje object,
this commits fixes that.

This is an improvement of the now reverted
55b62bbd70e32f5824cbc760bc2f8e49eafa4cdd.

Thanks to cedric for the pointer that lead to this commit.

Ref T3021

Revert "Edje UI mirroring: Fix UI mirroring for GROUP parts."

The following commit solves it in a better way.

This reverts commit 55b62bbd70e32f5824cbc760bc2f8e49eafa4cdd.

edje: unswallow objects from parent edje when swallowing into new edje

the passed edje is not guaranteed to be the one which owns the swallowed
object at any given time, so get the current owner to ensure that the
correct one is passed

ref a645c8153f82c593920190416b8115b546b3668f

 #MasterOfAllBorkers

@fix

eldbus model: no need to double const a Eina_Stringshare

Eina_Stringshare is already const, no need to const it again.

eina_inline_value: use void to force empty function parameters

We have to use void in a function declaration if we want no function
parameters. Using just empty parenthesis means the function takes an
unspecified number of parameters.

eina-cxx: Remove Eo classes defined manually

Use Eolian to generate the Eo classes for the tests. This should fix T2940.

@fix

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

configure: Fail if opengl=full is used with EGL

So many problems from people who customize their builds in this
incompatible manner.

There really should be only one configure option as we don't
support opengl+egl (although possible in theory) or gles without
egl. Keeping both to not break existing builds.

evas: fix optimization of vertex count in convex hull

Summary: FLT_COMPARISON should be used everywhere in evas_3d_utils.h

Reviewers: cedric, raster, Hermet

Subscribers: jpeg, artem.popov

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

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

eina: add test case for eina_strbuf_manage_read_only_new_length API.

Summary: Signed-off-by: Srivardhan Hebbar <sri.hebbar@samsung.com>

Reviewers: jpeg, cedric

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

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

eina: removing base64 from eina example.

Summary:
As base64 API's are no longer part of eina, removing them. Will add
separate file for emile examples.

Signed-off-by: Srivardhan Hebbar <sri.hebbar@samsung.com>
Reviewers: cedric, jpeg

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

Edje entry: Fix memory leak.

Summary:
Fix memory leak

If user or extra cursor is used, these should be free on
shutdown.

@fix

Test Plan: NA

Reviewers: tasn, herdsman

Subscribers: shilpasingh, cedric, jpeg

Projects: #efl

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

Edje UI mirroring: Fix UI mirroring for GROUP parts.

GROUP parts were not mirrored with the rest of the edje object,
this commits fixes that.

Fixes T3021

@fix

build: hopefully last fix for efl_gfx.x

Cedrics original change made sure the file was actually dist_installed but
failed to be placed inthe correct inlucde folder and thus failed the elm build.
While raster fixed this he broke the distcheck build as the file was no longer
put into the tarball being marked as nodist.
This fix handles all the cases I tried in my testing here.

efl - fix build break with efl_gfx.x

edje: correct function that should have been Eo API already.

emile: move all eina_str_base64 to emile_base64.

efl: move Ector_Color to be defined in Efl.Gfx as it makes more sense.

This create some possible naming clash and is why I come up with
efl_gfx_color*_type_set functions. We will have to think about this more
carefully as it makes sense to now pass this colors directly to our color
API. Ideally the default 8 bits interface would become just a convenience
wrapper around the more complex possibility.

js: fix binding after namespace change of connector

Summary:
Update register function after change introduced in
the commit dc560521509b93a58905c8280d7cf9622d4e87c4
changed the native API.

@fix

Reviewers: q66, cedric, felipealmeida

Subscribers: jpeg

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

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

ecore-evas: Rename ecore_evas_wayland_window_get2 function

This renames the ecore_evas_wayland_window_get2 function to be
ecore_evas_wayland2_window_get before the 1.17 roll out.

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

ecore-wl2: API expose ecore_wl2_input_grab function

This input_grab function should have been exposed via EAPI for use in
enlightenment, but was missed before the push

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

eina mp: also include malloc_np.h on FreeBSD

eina mp: only include malloc.h on linux

Elsewhere, everything is in stdlib.h.

@fix