Test rework #18: Eolian

Test rework #17: Eo

I've only touched the suite/ subdir. Maybe we should merege the others into
the test suite in suite/

Test rework #16: Emile

Test rework #15: Elua

Test rework #14: ElDbus_Cxx

Test rework #13: ElDbus

Test rework #12: Eio

Test rework #11: Eina_Cxx

Test rework #10: Eina

mainly minor stuff: whitespaces, order of includes, ... Just don't fear the number
of changed files :-)

Test rework #9: Efreet

this efreet test suite needs more love later

Test rework #7: Eeze

eeze_suite.c has also been splitted

Test rework #6: Eet_Cxx

Test rework #6: Eet

This one is big: I had to split the huge eet_suite.c into separate test files
and it needs more review

Test rework #5: Edje

Test rework #4: Ector

Test rework #3: Ecore_Cxx

Test rework #2: Ecore_Con

Test rework #1: Ecore

Factorisation of the infra
make uniform all the tests

Evas out: Fix Eolian warnings.

This includes either migrating types to eolian, fixing namespace or
importing extra types.

Ecore con base: Fix Eolian warnings.

This includes either migrating types to eolian, fixing namespace or
importing extra types.

Evas ector buffer: Fix Eolian warnings.

This includes either migrating types to eolian, fixing namespace or
importing extra types.

Evas vg: Fix Eolian warnings.

This includes either migrating types to eolian, fixing namespace or
importing extra types.

Edje cc: Fix shadow warnings.

Efl vg: Use correct types in eolian file.

ecore-drm: Make ecore_drm_output_rotation_set fail properly

When trying to set a rotation on a given output, we would previously
always be returning EINA_TRUE. We should be returning EINA_FALSE when
the rotation_set fails.

@fix

Summary: ecore_drm_output_rotation_set should be returning EINA_FALSE when the output doesn't have a plane of requested type.

Test Plan:
1. call ecore_drm_output_rotation_set() with ECORE_DRM_PLANE_TYPE_CURSOR
2. If output doesn't have a plane of ECORE_DRM_PLANE_TYPE_CURSOR, the for statement does nothing. But return value is TRUE;

Reviewers: raster, stefan_schmidt, gwanglim, devilhorns, zmike

Subscribers: input.hacker, cedric, JHyun, ManMower, jpeg

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

Evas text: set NULL free'd pointers in evas_object_text_free()

Summary:
_render_pre() function could be called for an object which is
going to be deleted. According to state changes of the object,
text could be recalculated with free'd pointers. It caused an
invalid read and crash.
@fix

Test Plan:
1. Apply D1747.
2. Run elementary_test.
3. Put any character in elm_entry and change paragraph direction.
4. Put any character again.
5. It can cause a crash which is caused by invalid read in Evas Text.

Reviewers: herdsman, woohyun, tasn, raster

Subscribers: cedric, jpeg

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

Evas textblock: ifdef unused 'hyphens' code

The usage of the 'hyphens' array is only relevant when
"--enable-hyphen" is used.
This fixes the dead code addressed in
5b083ace84ca91496810cd1826c9469a6c741700.

Revert "Evas textblock: Properly ifdef hyphenation out."

This reverts commit 5b083ace84ca91496810cd1826c9469a6c741700.

The "--enable-hyphen" option refers to using the optional hyphenation
dictionaries. We support hyphenation via SHY-HYPHEN hints regardless of
this option.
The following commit will provide finer handling to address the issue in
the reverted one.

eldbus: add more debug prints for proxy and signal handler

eina: accept NULL in eina_array_free() without erroring

@fix

eina: add tests cases for eina_vector3.

Summary: Add tests for functions of vector3 in eina.

Reviewers: cedric, Hermet

Subscribers: jpeg

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

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

edje: add new part handling Canvas3D types to edje_load.c

Summary: Initiation of Evas Objects and creation of nodes for new part types in Edje.

Reviewers: cedric, raster, Hermet

Subscribers: jpeg, artem.popov

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

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

ecore_con: change Ecore.Con.Client to Efl.Network.Client.

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

Reviewers: cedric

Subscribers: jpeg

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

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

evas: optimize Canvas3D calculation of bounding space

Summary: Bounding box and sphere will calculate if user need callback of collision or data of box or data of sphere.

Reviewers: sri.hebbar, cedric

Subscribers: cedric, jpeg

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

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

edje: fix typo affecting our 32bits signal matching code.

Big thanks to Maxim who did a serious digging in this issue making it almost
a patch review.

T3125

@fix

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

Eolian eina_types: Add Eina.Matrix3 and start using it.

Ector gl renderer: Fix Eolian warnings.

This includes either migrating types to eolian, fixing namespace or
importing extra types.

Efl animator: Fix Eolian warnings.

This includes either migrating types to eolian, fixing namespace or
importing extra types.

Ector gl surface: Fix Eolian warnings.

This includes either migrating types to eolian, fixing namespace or
importing extra types.

Ector generic surface: Fix Eolian warnings.

This includes either migrating types to eolian, fixing namespace or
importing extra types.

Ector generic buffer: Fix Eolian warnings.

This includes either migrating types to eolian, fixing namespace or
importing extra types.

Ector cairo: define cairo_t as extern.

Ecore exe: Move more structs and enums to Eolian.

Ecore con dns: Fix eo file warnings.

ecore-x: Add EINA_SAFETY checks for missing display

xlib immediately crashes upon being passed a null DISPLAY object,
so every function in ecore-x should likely have safety checks such
as these.

@fix

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

ecore-x: Add EINA_SAFETY checks for missing display

xlib immediately crashes upon being passed a null DISPLAY object,
so every function in ecore-x should likely have safety checks such
as these.

@fix

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

ecore-x: Add EINA_SAFETY checks for missing display

xlib immediately crashes upon being passed a null DISPLAY object,
so every function in ecore-x should likely have safety checks such
as these.

@fix

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

Evas textblock: Disable hyphenation tests when hyphenation is disabled.

Evas textblock: Properly ifdef hyphenation out.

I found this thanks to coverity. There was some code that we were running
although hyphenation was actually off.

CID 1341062.

Evas textblock: Fix _dict_hyphen_load could return an uninitialized pointer

Summary:
If there are hyph_*.dic files except for requested language,
"dict" pointer could be return without initialized. It doesn't make any
warning messages when it is compiled. Normally, it is NULL implicitly.
But, it is good to set NULL explicitly for understanding code.

Test Plan: N/A

Reviewers: herdsman, tasn, woohyun

Subscribers: cedric, jpeg

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

ecore_imf/wayland: send cursor position before requesting to show

ecore_imf/wayland: Fix not to show IME in touching entry after hiding IME

ecore-x: Fix compiler warnings about return values

This is an "oopsie" fix from my previous commit wrt adding EINA_SAFETY
checks. I made an oopsie and didn't realize that these 2 functions
were expecting int returns.

@fix

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

ecore-x: Add EINA_SAFETY checks for missing display

xlib immediately crashes upon being passed a null DISPLAY object,
so every function in ecore-x should likely have safety checks such
as these.

@ref 210b3e6c622341d4366c27c4d9bc1086012efd28

@fix

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

Evas font: Use proper enum value for extrabold

Summary:
ULTRABOLD is identical to EXTRABOLD in freetype.
But, "extrabold" word is added for FC_WEIGHT_EXTRABOLD.
So, it has to be changed to use EXTRABOLD instead of
ULTRABOLD. It was mistake in my previous commit.
@fix

Test Plan: N/A

Reviewers: tasn, herdsman, woohyun

Subscribers: cedric, jpeg

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

efl: fix installation and compilation of efl_types.eot.h

Summary:
Fix Efl.h include problem of the installed header "efl_types.eot.h".

Move efl_types.eot to src/lib/efl/interfaces to compile and install at
the same place of other Eolian-generated headers.

Test Plan: compile and test expedit

Reviewers: felipealmeida, stefan_schmidt, tasn, cedric, q66, JackDanielZ

Subscribers: jpeg

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

eeze: add case for Graphics subsystems in eeze_udev_find_by_type

Add code to detect Graphics subsystem, i.e framebuffer devices
/dev/fbX in eeze. This will be used in Ecore-Fb to detect the
framebuffers present on the system.

@feature

Signed-off-by: Nicolas Aguirre <aguirre.nicolas@gmail.com>
Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

evas: include Eet.h where it's needed.

This patch is applied by OpenEmbedded project to let evas build
correctly in crosscompilation environnement.
cf: https://github.com/openembedded/meta-oe/blob/master/meta-efl/recipes-efl/efl/efl/0002-evas_3d-Add-Eet.h-includes.patch

@fix

Signed-off-by: Nicolas Aguirre <aguirre.nicolas@gmail.com>
Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

eolian: allow silencing of type errors in validation

This allows generators to silence type errors in validation in order
to reduce duplicate error messages when generating multiple files.
Also adjusted the C generator to only emit type errors when generating
Eo header files.

@feature

efl: Have Efl.Time outside of a beta guard

Since this is a essential type, this must not be guarded by a beta
symbol guard.

efl: Renamed Ecore.Time to Efl.Time and defined it

Moved the Ecore.Time @extern struct to Efl lib and defined it as
specified in C specification for struct tm. Thus, bindings can be
automatically generated for where struct tm is used.

ecore: move Ecore_Pos_Map to ecore_types.eot

Move Ecore_Pos_Map from Ecore_Common.h to ecore_types.eot.
Give it the namespaced Eolian name "Ecore_Pos_Map" to follow the
standards.

Update documentation to refer to Ecore_Pos_Map instead of its previous
enum definition "_Ecore_Pos_Map".

ecore: create ecore_types.eot and add Ecore.Time to it

Create the file ecore_types.eot to hold common types related with Ecore.

Add Ecore.Time as an external type to ecore_types.eot.
This type is intended to be a alias to struct tm (from time.h).
That way .eo files have a standard way to reference it.
Each language should manually bind it.

evas: check EINA_MEMPOOL env var when creating evas mempool allocators

this should really be a utility function already since all mempools in efl use
the exact same code...

eina: add test covering tiler rect add/del/add of same geometry regions

ref ec2ec3e9d13f64b5a4792717204deb59eb9d4caa

eina: invalidate last add/del rects when the opposite operation occurs

applying this optimization to prevent the same rectangle from being added
or removed repeatedly in succession would result in the rejecting of successive
operations of the same type when the other operation occurred in between.

as an example:

add(0, 0, 100, 100)
del(0, 0, 100, 100)
add(0, 0, 100, 100)

should yield (0, 0, 100, 100), not zero rects and a failure to add the
second rect

this fixes a serious issue in enlightenment where stacking three windows
on top of each other with the first and third windows having the same geometry
would result in the top window receiving no input geometry (oops)

@fix

evas-wayland-shm: Fix compiler warning about incompatible pointer type

This patch fixes some compiler warnings about assignment from
incompatible pointer type when getting image data from evas_cache. The
simple fix is to cast the return value to RGBA_Image.

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

edje: Remove unused variables

This patch just removes some unused variables that the compiler was
warning about

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

Ecore con local: properly clean up fd on error.

This wasn't done correctly in the previous commit. First of all, the
order of cleanup was wrong, the cleanup area should only be called if
failures occurred after the fd allocation, not before. Also, fd should
be reinitialised to -1 once we close the socket.

Ecore con: Close server socket fd on failure.

Summary:
Socket fd must be closed to avoid file discripter leak.
Programs can usually only open a limited number of file descriptors,
so if this happens a lot, it may turn into a problem.

@fix

Reviewers: raster, Hermet, wonsik, spacegrapher, cedric, jpeg, tasn

Reviewed By: tasn

Subscribers: cedric, alok25, yashu21985, singh.amitesh

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

edje_cc: allow omitting "name" keyword in color_classes.color_class block

Summary:
This allows developer to omit "name" keyword in color_classes.color_class
block in EDC.

Reviewers: cedric, jpeg

Reviewed By: jpeg

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

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

edje: add new part types to edje_cache

Summary: Add new part types (Mesh_Node, Camera, Light) to edje_cache

Reviewers: cedric, raster, Hermet

Subscribers: jpeg, artem.popov

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

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

ecore_con: changing Ecore.Con.Server to Efl.Network.Server.

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

Reviewers: cedric

Subscribers: jpeg

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

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

eina: add eina_vector3 as static inline.

Summary: Move evas_vec3 to eina_vector3 and add documentation.

Reviewers: stefan_schmidt, jpeg, cedric

Reviewed By: cedric

Subscribers: jpeg

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

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

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: raster, jpeg

Maniphest Tasks: T2513

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

edje: introduce color_tree (color_class inheritance)

Summary:
The "color_tree" block contains a list of one or more "node" blocks.
A "node" block consists of its own color class name and the list of child
color classes. At runtime, parent color class will be referred instead,
if child color class is set to part but its color values are not defined.

Reviewers: raster, Jaehyun_Cho, jpeg, cedric

Reviewed By: cedric

Subscribers: cedric, kimcinoo

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

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

edje: add test for omitting "name" keyword in color_classes.color_class block

Summary:
This test checks whether omitting "name" keyword in color_classes.color_class
block in EDC work normally.

Depends on D3598

Test Plan: make check

Reviewers: cedric, jpeg

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

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

evas: add native surface with TBM Surface for wayland_shm engine

Summary: @feature  add evas object image's native surface with TBM surface for wayland_shm

Test Plan: Making tbm test sample and testing on Tizen Device/Emulator

Reviewers: raster, cedric, spacegrapher, jpeg

Reviewed By: spacegrapher

Subscribers: mer.kim, JoogabYun, scholb.kim, dkdk

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

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

eo: tests callback add and del return value.

ecore_evas: refactorize code and make sub ecore evas exploration recusrive.

ecore_evas: improve debugging for Ecore_Evas animator.

edje: add back use of Ecore_Animator when the Edje object is created on an non Ecore_Evas canvas.

This is necessary for backward compatibility still I am thinking of displaying a warning for this
use case and request people to update there work and drop that feature in the future (In a year
maybe from now). Elementary doesn't need this as it depends on Ecore_Evas.

Evas textblock: Save memory space reducing unused hyphen dictionary loads

Summary:
Commonly, only few hyphenation dictionaries are used at a application.
So, loading all of dictionary files could cause waste of memory.
Evas textblock has to load hyphenation dictionaries only when it is
really needed.

Test Plan: N/A

Reviewers: woohyun, tasn, herdsman

Subscribers: cedric, jpeg

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

This should not have been pushed. Revert

Revert "ecore-drm: Add API functions for rotation support"

This reverts commit b8ceaf0d403ce640ed6a485a59a3bd5a7b06a260.

This should not have been pushed. Unsure what happened here, but
revert this

Revert "ecore-drm: Add opaque type for Plane support"

This reverts commit 94082f54e8ce7d47402ea5d8d57e3871f8fa98a4.

ecore-drm: Add API functions for rotation support

This adds 2 new API functions for getting supported rotations of an
output, and for setting rotation on an output

@feature

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

ecore-drm: Add opaque type for Plane support

This adds an opaque structure to represent a hardware plane for use in
rotation, setting cursor/overlay/primary plane content, etc

Signed-off-by: Chris Michael <cpmichael@osg.samsung.com>
ecore-drm: Add API functions for rotation support

This adds 2 new API functions for getting supported rotations from an
output, and for setting rotation on an output.

@feature

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

ecore-drm: Add internal function to create planes for an output

This patch adds an internal function that we use during output
creation in order to create the planes necessary and get the supported
rotation values for an output

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

ecore-drm: Add API function to set rotation on an output

This patch adds an API function to allow the Screen Setup dialog in
Enlightenment to support setting rotations on an output when running
under drm

@feature

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

ecore-drm: Add API function to get supported rotations from an output

This patch adds an API function so that we can get the supported
rotations from an output. This is needed so that the Screen Setup
dialog in Enlightenment can list the rotations supported for the user
to choose from while running using drm

@feature

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

ecore-drm: Add opaque structure for Ecore_Drm_Plane

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

Evas: Use proper language for harfbuzz shaping

Summary:
Evas Text, Textblock, Textgrid keeps own language information.
This language information could be vary from the result of setlocale().
Especially, Evas Textblock supports <lang> tag. The language could be
changed in the middle of text. All of these language has to be used
for harfbuzz shaping.
@fix

Test Plan: N/A

Reviewers: herdsman, raster, woohyun, tasn

Reviewed By: tasn

Subscribers: cedric, jpeg

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

Eo tests: Add tests for eo_init()/shutdown() cycles.

We needed to fix the issue and remove a workaround in the test suite
before we could add these tests. Now they are here and they test that
init/shutdown cycles work as expected.

Eo tests: remove hack to prevent eo_init()/shutdown() cycles.

We can now remove it thanks to the previous commit.

Eo: add support for initialising eo after it has been shut down.

Until now it wasn't allowed/possible to init (eo_init) eo after it has
been shut down (eo_shutdown). This commit fixes that, so now that is
fully legal to have as many init/shutdown cycles as you want.

There was a previous workaround for this issue:
e47edc250dc715a6b0f94be4b1cb81d32e9d97fe.

This should allow more flexibility when using the EFL in loadable
modules and in various other scenarios.

The problem is that the class_get() functions cache the previously
created class for efficiency, but the class is freed if eo is shut down,
so the cached pointer is actually invalid.
The solution to the problem was to maintain a generation count
(incremented every time we shut down eo), and compare that to a locally
saved version in class_get(). If they don't match, recreate the class,
as it has already been freed.

@feature

Revert "Eo: Prevent shutdown from actually doing anything"

This commit was a workaround to let us shutdown and then init eo without
any issues. It leaks and it's wrong. This will properly be fixed in the
next commit.

This reverts commit e47edc250dc715a6b0f94be4b1cb81d32e9d97fe.

ecore_imf/wayland: Fix IME hide -> show issue in case of focus-out and then focus-in

Change-Id: I1d3d7023dbd10b78f510380ff3c46087228f9180

ecore_imf/wayland: do not check input panel enable in show or hide

Input panel enable is already being checked in ecore_imf API.

ecore_imf/wayland: support to set prediction allow mode

ecore_imf/wayland: notify language changed