Coverage: Fix base directory.

The base compilation directory is the root of the repo, not src/.

All the relative paths are relative to there.

Eolian: Re-generate the ragel file, but this time from the right place.

Add --disable-gui.

This allows people to disable the building of anything GUI related.
In my case, it is used for servers.

I encourage anyone that think they can do a better patch to improve it,
as i dislike having to add all those AM_CONDITIONAL().

Maybe the macros should be improved.

Eolian/Generator: Fix Windows issue during string manipulation.

For Windows, it is simpler to replace all the \r\n with \n after having
read the file.

Eolian: rename internal files.

This files were needed to be renamed a long time ago :-)

Eolian: Fix Coverity issue.

In a certain case, the function was returning without releasing all the
resources.

CID 1220084

eina: removing more dead code looking for strange int.

CID 98377.

eina: removing more dead code that expect strange int.

CID 1039941.

eina: remove logically dead code.

I am wondering why this code was around, an useless variable and an if
that would never be taken...

CID 1039940.

eina: remove logically dead code.

Removing this code let me wonder what could be an int value that is not 0,
not < 0 and not > 0...

CID 1039938.

eina: let's check for returned value of realloc.

CID 1039689.

eina: remove logically dead code.

The for loop above would exit with at least length = 0, so < 0 is
just impossible.

CID 1039943.

eina: simplifie logic to not go with multiple level of callback for just getting the first entry in a hash.

CID 1039937.

eina: fix possible memory leak on failure in eina_tiler_equal.

CID 1216618, 1216615, 1216614.

eina_cxx: add documentation to the Eina C++ header files.

Summary:
Added documentation to almost all classes that are intended for direct use by the library users.
Marked classes/functions used by the library itself as @internal.
Modified the Doxyfile.in to enable the generation of documentation to the bindings.

Reviewers: felipealmeida, cedric, woohyun, smohanty, raster

CC: savio, cedric
Differential Revision: https://phab.enlightenment.org/D947

Signed-off-by: Cedric BAIL <c.bail@partner.samsung.com>

edje: Edje_Edit - add edje_edit_part_access functions.

Summary:
Add two main functions
- edje_edit_part_access_get
- edje_edit_part_access_set
Also support access into part's source code generation.

@feature

Reviewers: seoz, Hermet, raster, cedric

CC: reutskiy.v.v, cedric
Differential Revision: https://phab.enlightenment.org/D946

Signed-off-by: Cedric BAIL <c.bail@partner.samsung.com>

edje: Edje_Edit - add edje_edit_part_precide_is_inside functions.

Summary:
Add two main functions
- edje_edit_part_precise_is_inside_get
- edje_edit_part_precise_is_inside_set
Also support precise_is_inside into part's source code generation.

@feature

Reviewers: cedric, seoz, Hermet, raster

CC: reutskiy.v.v, cedric
Differential Revision: https://phab.enlightenment.org/D945

Signed-off-by: Cedric BAIL <c.bail@partner.samsung.com>

edje: Edje_Edit - add getters and setters for source2-source6 TEXTBLOCK properties

Reviewers: cedric, raster, seoz, Hermet

CC: reutskiy.v.v, cedric
Differential Revision: https://phab.enlightenment.org/D943

Signed-off-by: Cedric BAIL <c.bail@partner.samsung.com>

evas: Evas_3D - Revision frustum culling.

Summary:
Add API that checks OBB's points of node are inside frustum of camera.
Add example with frustum culling.

@feature

Reviewers: Hermet, cedric, raster

CC: cedric
Differential Revision: https://phab.enlightenment.org/D942

Signed-off-by: Cedric BAIL <c.bail@partner.samsung.com>

edje: Edje_Edit - add edje_edit_part_pointer_mode functions.

Summary:
Add two main functions
- edje_edit_part_pointer_mode_get
- edje_edit_part_pointer_mode_set

Also support pointer mode into part's source code generation.
EVAS_OBJECT_POINTER_MODE_AUTOGRAB is default value,
so generation won't add "AUTOGRAB" into source code if this type of
pointer mode is set.

@feature

Reviewers: cedric, seoz, Hermet, raster

Reviewed By: cedric

CC: reutskiy.v.v, cedric
Differential Revision: https://phab.enlightenment.org/D941

Signed-off-by: Cedric BAIL <c.bail@partner.samsung.com>

edje: Edje_Edit - add edje_edit_part_copy function that copy parts within whole group.

Summary:
Add ability to copy whole part and create same part but with new name).
This feature also require to slightly remake function edje_edit_state_copy to make
it able to copy state from one part to different one (only for part_copy usage).

@feature

Reviewers: cedric, seoz, raster, Hermet

CC: reutskiy.v.v, cedric
Differential Revision: https://phab.enlightenment.org/D939

Signed-off-by: Cedric BAIL <c.bail@partner.samsung.com>

gitignore: let's ignore generated binary.

edje: Edje_Edit - fix edje_edit_part_source_set

Summary:
Fixed edje_edit_part_source_set so that it does not destroy TEXTBLOCK.
Changed type check to switch-case to stress allowed types.

@fix

Reviewers: cedric, raster, seoz, Hermet

Reviewed By: cedric

CC: reutskiy.v.v, cedric
Differential Revision: https://phab.enlightenment.org/D931

Signed-off-by: Cedric BAIL <c.bail@partner.samsung.com>

C++ bindings: Generate them as part of the build, not pre build.

There's no reason why it should be forced in with BUILT_SOURCES.
It should be part of the normal build and dependency based (nothing should depend
on it anyway).

This patch fixes that.

Evas textblock: Fix crash when cutting ligatures with formats.

"f<color=#f00>i</color>f" could cause textblock to crash. It doesn't
crash anymore. It doesn't render the colours correctly either, but at
least this is the first step.

This is the start of fixing T1308

@bugfix

evas: check that we don't use a NULL pointer on wrong frame number.

CID 1220083

evas: Examples - fix Evsa3D Eo classnames.

Summary: Renamed EVAS_OBJ_.+_CLASS to EVAS_.+_CLASS so that "make examples" works.

@fix

Reviewers: cedric, tasn, Hermet

CC: felipealmeida, smohanty, raster, cedric
Differential Revision: https://phab.enlightenment.org/D935

Signed-off-by: Cedric BAIL <c.bail@partner.samsung.com>

edje: Edje_Edit - add sounds list accessor.

Summary:
make "get" functions for block "sounds" which return
a sounds or tones list (Eina_List *) off the given edje object.
@feature

Reviewers: raster, cedric, seoz, Hermet

Reviewed By: cedric

CC: reutskiy.v.v, cedric
Differential Revision: https://phab.enlightenment.org/D932

Signed-off-by: Cedric BAIL <c.bail@partner.samsung.com>

edje: Edje_Edit - add new feature edje_edit_image_compression_set.

Summary:
Set compression type for the given image. Has object being edited,
the name of the image and Edje_Edit_Image_Comp(EDJE_EDIT_IMAGE_COMP_RAW,
EDJE_EDIT_IMAGE_COMP_USER, EDJE_EDIT_IMAGE_COMP_COMP,
EDJE_EDIT_IMAGE_COMP_LOSSY[_ETC1]) as in parametrs.
Return EINA_TRUE if successful, EINA_FALSE - otherwise.

Reviewers: cedric, raster, seoz, Hermet

Reviewed By: cedric

CC: reutskiy.v.v, cedric
Differential Revision: https://phab.enlightenment.org/D929

Signed-off-by: Cedric BAIL <c.bail@partner.samsung.com>

Fix curl dynamic loading on windows

With win-builds.org curl dll name is libcurl-4.dll. Try to load this
correct filename first

Fix gcc warning about uninitialized variable.

Eolian CXX: Remove unneeded Eo.h include.

Eo.h depends on things that are generated by normal Eolian, and because Eolian C++
is generating code pre-build (WHY THE FUCK???), the dependencies are missing.

This removes the stupid include, but a more proper fix would be to move eolian_cxx
to be part of the build process and not pre-build.

Eo base: Start generating eo_base from the .eo file.

Efl: Update code to use the new class names generated by eolian.

Eolian cxx: Fix it to use the right class names.

This is following the class name change in normal eolian.

Eolian: Use class name and not prefix in the class macro.

So for example a class called Eo_Base with prefix: eo, used to be
generated to EO_CLASS instead of EO_BASE_CLASS.

Eo: Fix and use the abstract class .eo file.

Until now it was just there, but never generated or used.

Eolian: Fix bad behavior when the .eo is incorrect

In this case, the section 'implements' contains bad information about the
function to override. If the first (at least) function is correct, it
will never fail but use the last correct information retrieved from the
database.
The patch fixes it by checking the result of the database function
eolian_implement_information_get.

@fix

Eolian: Fix class look up.

If a class (e.g Evas_Object) is expected as a base class but is not
present in the existing expected file (e.g evas_object.eo), it will not
fail.
This patch fixes it by checking the existence of the class into the
database after the parsing of the file.

@fix

Dont install c++ headers of bindings when --disable-c++11 is used.

Eo base: rename event_freeze_get to event_freeze_count_get.

This is needed because of a possible clash between the method event_freeze
and the property event_freeze with bindings.

eina/file - revert 18be4c50d9990c82ed9ce4269b7820e61f772699

I realized that's not a bug in eina but user usage was inccorect.
revert the patch

@fix

evas/blend: oops, missed a variable declaration.

build error by this commit.

aede29e2d4cd49c370b5225196fd267a712d3593

evas/blend - disable neon code.

Current neon code is a little buggy, the blend won't be done correctly.
so leave the c code depends on the compiler optimization.

Eo: Fix docs of functions that have changed in eo2.

Fix unused var warning in emotion.

Fix unused var warning.

Fix unused var warning

Fix unused var warning.

Fix warning with undefined ntohl().

Fix warnings with sig_atomic_t being a long on FreeBSD.

Eo: Add eo_finalize. A func that's called at the end of eo_add.

This function lets you hook at the end of eo_add and override it for a
class. This is essentially the first step towards killing custom
constructors. Instead of having a custom constructor, you should just
do:
eo_add(CLASS, parent, a_set(3), b_set("eou"));
eo_constructor is called at the beginning for pre-init things.
eo_finalize is called at the end, for actually finalizing and doing
things. This cleans up the API and possibly saves a lot of things that
would have been stupid and slow in the past, like loading an elm widget
with an existing theme, and then changing the theme.

** This breaks Eo ABI, please recompile elementary and everything else that
creates eo objects.

@feature

Add prototype of evas_3d_mesh_aabb_add_to_frame for evas_3d_mesh_loader_md2.c

Fix warning with -Wincompatible-pointer-types-discards-qualifiers

Fix unused param warning when building on non linux.

Remove warning by adding eina_cpu.h which expose eina_cpu_page_size()

Removed double inclusion of eina_thread.h

Fix warnings in eina_thread.c

Fix warnings on calls to iconv, on freebsd.

Fix unused var warning when not building on linux.

evas/evas_3d: fix implicit declaration function call.

Thanks JackDaniel for reporting.

Evas: Update for giflib 5.1 api change

Summary: Slight change in the file close function.

Reviewers: jpeg

CC: cedric
Differential Revision: https://phab.enlightenment.org/D916

Evas/Textblock: add support for ellipsis values

Summary:
This enables textblock to support more values other than 1.0.
For 0 <= ellipsis < 1.0, it splits the text such that it fits the
textblock's width. The ellipsis is relatively position according to the
ellipsis value, and characters are removed also relatively.
For example, a value of 0.5 will position the ellipsis right in the
center of the textblock's width, while removing characters equally right
and left from the center.

Basic approach to this feature was to do some work before the layout
process. We calculate the expected total width of the items, and by how
much we exceed from the textblock's width. Afterwards is it just some
careful work to set the boundaries of the width we want to cut, and
deciding which characters we need to removed to fulfill this requirement.
The rest is splitting the text and visually-removing the part we
need to cut.
This is all handled before any logical lines are created, so the
_layout_par function remains almost intact. A designated _ellip_prev_it
field in the Paragraph struct instructs after which item we place the
ellipsis item, if at all.

Note that we keep the fast path for ellipsis 1.0, as heavier work needs
to be done for the other values.

Added tests to evas_suite for a range of ellipsis values.

Also, multiline is unsupported at the moment.
@feature

Test Plan: Anything that uses Evas Textblock (single-line, please)

Reviewers: tasn, id213sin, raster

CC: cedric, JackDanielZ, raster
Differential Revision: https://phab.enlightenment.org/D905

Eolian: modify 'implement' API parameters.

The function eolian_implement_information_get was returning strings for
the class and the function. It was written in this way at the beginning
because it was not needed to verify the correctness of the class and
the function.
Now that we have the namespace feature, this function must check it,
meaning that the class and the function are now known.
So we can return them instead of returning the strings.

The generators had to find the class from the classname. It is no more
needed.

The C++ generator has been adapted to this new API.

Win64: Fix a bunch of warnings

Fix invalid casts.
Use printf("%z") where appropriate.
Fix unused variables warnings.

Thanks vtorri for the patch.

@fix

Evas: Remove invalid cast and fix warning/bug on windows

On Windows, long is 32 even on Win64.
Thanks Vincent for spotting this.

@fix

Eo legacy events: Add missing addition to hash table.

This rendered the whole thing useless.

Found thanks to coverity.
CID 1216616
CID 1216617

evas loader - redo gif fix so it doesn't break everything

this fixes 96f9353f4c951dd5dd41a687e272ffe1c28bb9ba so it works right
@fix

Eolian: modify generators to use internal variables environment.

Eolian: add the super invocation for developer code generation.

Now the eo_do_super is invoked for constructors overridding in a
inheriting class.

Eolian: Add an internal environment for class variables.

To reduce the invocations to strings conversions, we store the
classname, the Eo prefix in upper and lower cases in global variables.
The problem comes when we have to handle overriding functions. A lot of
conflicts between class base and class inheriting can happen.

The chosen solution is to create independent environments storing the
converted strings.

Eolian: fix test.

Constructor has to be defined in the constructors section and not as a
method.

Evas_3d: fix warnings.

Revert "Evas: gif decoding bug fix"

This reverts commit 96f9353f4c951dd5dd41a687e272ffe1c28bb9ba.

This breaks gif playing BAAAADLY!

ecore-drm: convert compose string into utf8

Summary:
Key event string encoding was LATIN1 instead of UTF-8.
This unifies the handling with the x backend.

Reviewers: raster

Reviewed By: raster

CC: cedric
Differential Revision: https://phab.enlightenment.org/D912

Eo/evas events: Fix support of mixing legacy smart events and eo events.

This merge removes the old hash table and a few other things that were
wrong in the old way of doing things. This should also make things a bit
faster with legacy callbacks.

@fix (no backport, as it's too intrusive)

Eo legacy events: Add legacy events tests.

These tests test mixing legacy events with eo events.

Eo: Support mixing legacy and non-legacy callbacks.

Eo/evas events: Move legacy events dynamic generator to eo.

warning--

[Evas/evas_3d_scene] Propagate events.

Summary: Add 3 new APIs function for full work of the picking nodes.

Reviewers: Hermet, cedric, raster

CC: cedric
Differential Revision: https://phab.enlightenment.org/D889

C++11 explicit conversion from accessor to bollean

Summary:
Replaced the "safe bool idiom" by explicit conversion to bollean,
in order to follow the improved conventions of the C++11.

Reviewers: felipealmeida, cedric, woohyun, smohanty, raster

Reviewed By: raster

CC: savio, cedric
Differential Revision: https://phab.enlightenment.org/D903

Correction for the macro EINA_CXX_DOM_LOG

Summary: The previous definition of EINA_CXX_DOM_LOG could cause a problem if used inside an one-line "if" statement followed by an "else". Changed the top "if" by a "for" statement to avoid this problem.

Reviewers: felipealmeida, cedric, woohyun, smohanty, raster

Reviewed By: raster

CC: savio, cedric
Differential Revision: https://phab.enlightenment.org/D902

Fixed rbegin/rend implementation on the Eina C++ ranges.

Summary:
Fixed the rbegin/rend functions returning switched reverse iterators.
Added code to some unit tests for comparing elements in reverse order, in interest of
checking the correctness of the rbegin/rend functions.

Reviewers: felipealmeida, cedric, woohyun, smohanty, raster

Reviewed By: felipealmeida

CC: savio, cedric
Differential Revision: https://phab.enlightenment.org/D901

eina-tiler:add union, subtract, intersection, equal apis for tilers

Summary:
Support union, subtract, intersection, equal(comparison) between tilers.
@feature

Test Plan: Test with added test case(src/tests/eina/eina_test_tiler.c) and the example(src/examples/eina/eina_tiler_02.c)

Reviewers: gwanglim, devilhorns, raster, zmike, cedric

CC: cedric
Differential Revision: https://phab.enlightenment.org/D880

Eolian: fix issue for Windows.

On Windows, '\' are used to separate the names of a path. It was not
supported.
This patch fixes that. Thanks to Vincent Torri for catching it (and good
luck in Windows ;-))

gl_x11/evas_engine: Prevent dereference before NULL check

Summary: re->win pointer was not compared with NULL pointer before re->win was referenced.

Reviewers: Hermet

Reviewed By: Hermet

CC: seoz, cedric
Differential Revision: https://phab.enlightenment.org/D910

evas_gl_rectangle: Remove unnecessary conditional expression

Summary: The comparison gc->dc with NULL is not necessary. So the unnecessary conditional expression is removed.

Reviewers: Hermet

CC: seoz, cedric
Differential Revision: https://phab.enlightenment.org/D909

evas/textblock - apply efl coding style.

evas/textblock - removed unnecessary null check.

It should be definitely valid.
It's meaningless checking the value there.

evas/textblock - null check format.

logically it's insane that it doesn't check the null there but it checks right next line.

Evas rg_etc2: Fix Windows build

Use unsigned char/int instead of stdint in the header file.

evas_draw_main: Remove unnecessary conditional expression

Summary: The comparison dc with NULL is not necessary. So remove the unnecessary conditional expression.

Reviewers: Hermet

Reviewed By: Hermet

CC: seoz, cedric
Differential Revision: https://phab.enlightenment.org/D908

evas_font_dir: Prevent the use after free

Summary: Pointer fd is used after being deallocated.

Reviewers: Hermet

Reviewed By: Hermet

CC: seoz, cedric
Differential Revision: https://phab.enlightenment.org/D906

Eolian test: Fix other compilation warnings

Eolian test: Fix test on hardened gentoo

Invalid snprintf, why declare MAX_PATH and then use PATH_MAX?

Evas rg_etc1: Fix build break (with DEBUG)

Ooops, bad code merge here.
This breaks the nightly builds.

evas/evas3d - fixed incorrect usage in evas-3d-aabb example.

Revert "ecore/ecore_event - logically this check is redundant."

This reverts commit df60e63bf9968a4528577762a7da262466ac7683.

seems this patch is wrong.

Evas textblock: Fix wrong advance calculation when bidi is disabled.

The last_it was not updated correctly with bidi disabled.

Thanks to Daniel Hirt for spotting this.

@fix