Revert "eina: rely on GNU make to find file at their correct location."

This reverts commit e26fcbb1dc588c5130e477bf832c59386b1c1951.

This reverts one of three parts of the try to get sub directory
compilation back into eina. It breaks our distcheck though and I
talked to Cedric about it and he prefers to revert these as we might
need to go another route to bring this functionality back. Details
will come to the mailing list.

eolian: validate eo/legacy prefix to be [a-z_][a-z0-9_]*

build: do not use DEBUG when running with coverage

Do not force this on everybody who runs with tests=coverage. It makes stdout
impossible to follow as well as overruns the tests log file. If one wants to
run with -DDEBUG it can be set from your own CFLAGS.

build: fix linking with coverage enabled after osx fix

After aca6fc8c3611b5e7d4faaed61e52dedf057d7219 there have been problems when
linking when having coverage enabled.
AC_SEARCH_LIBS was used wrongly here. You need function name first and lib to
link against as second argument. The symbol name also has wto leading
underscores for me.
/usr/lib/gcc/x86_64-pc-linux-gnu/5.3.0/../../../../x86_64-pc-linux-gnu/bin/ld generated: undefined
reference to `__gcov_init'

tests: get rid of Eo_Event2 in cxx tests

Hopefully the last occurance of Eo_Event2 in our code base. Fixes a build break.

cxx bindings: get rid of Eo_Event2

This follows the change in EO and fixes some build breaks.

build: fix type in m4 macro

This macro changes the LDFLAGCS not CFLAGS.

Eo event cb: Update documentation.

Eo: Update documentation according to the event cb changes.

Eo: get rid of Eo_Event2.

The hack is still there, but much cleaner now.

ecore: Create Promises

Add a promise object that allows Eolian interface to include promises
as a way to have asynchronous value return and composibility.

The usage is like this in a .eo file:

class Foo {
   methods {
      bar {
         params {
            promise: Promise<int>;
         }
      }
   }
}

Which will create the following API interface:

void foo_bar(Ecore_Promise** promise);

and the equivalent declaration for implementation.

However, the API function will instantiate the Promise for the
user and the implementer of the class.

eina: Add Eina_Iterator implementation for C-array

This iterator is convenient when you already have a C-Array and you
need to pass this array to a function receiving an Eina_Iterator.

int array[] = {1, 2, 3, 4};
int* array2[] = {&array[0], &array[1], &array[2], &array[3], NULL};

Eina_Iterator* iterator = eina_carray_iterator_new((void**)array);

eina: fix typos in doxygen

ecore: better resources use in the glib integration

Summary:
When glib support is enabled (HAVE_GLIB), _ecore_glib_init()
was always reserving resources. However, its counterpart may not
be called when:
- glib is not always integrated and
- when a user didn't explicitly required the integration.

Calling _ecore_glib_init() within the request code will cause the
resources to be reserved only when the integration with glib is
required and furthermore guarantees that resources always have a
chance to be released.

Reviewers: cedric, raster

Subscribers: cedric, jpeg

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

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

Evas: fix camera node adding to scene

Summary: While making new changes after rewieving D3710 we met this bug again, removing eo_unref is the best way to fix it because _eo_ref_replace from D3021 makes nothing special.

Reviewers: cedric, raster, Hermet

Subscribers: jpeg, artem.popov

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

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

evas: fix preload thread deadlock issue

Summary: Wrong condition makes deadlock issue.

Reviewers: jpeg, jypark

Subscribers: cedric

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

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

edje: return default description depends on RTL when description doesn't exist

Summary:
If not found edje part description, edje just set default description in spite of RTL status.
This adds to call function for getting the correct description as RTL status.

Reviewers: raster, Hermet, cedric

Subscribers: minkyu, sju27, cedric, jpeg

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

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

edje_embryo: add box attributes support to set_state_val()

Summary: Add box attributes support to set_state_val().

Reviewers: raster, cedric

Reviewed By: cedric

Subscribers: jpeg

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

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

tests: disable loading of the scim module as this can causes hangs of the tests

With scim installed we can run into hangs of the test suite when the ecore_imf
scim module tries to connect to the scim on the system. This has happened again
and again on different installations and made the test suite really fragile.

We would need to make sure that scim is configured on the host before we could
run this test. It might be a candidate for skipped tests where we check if the
env has all we need to run the test and if not skip it. We don not have all the
needed pieces in place for this so the best we can do to make the test runs less
fragile is disabling scim module loading for now.

Eo tests: Add tests for null (invalid) array callbacks.

Eo: unmark Eo_Class as deprecated.

It's not deprecated, it's actually a useful alias.

Eo: Remove more deprecated functions.

Eo: remove the long deprecated eo_data_get.

It has been deprecated for a while, and now it's time to actually stop
using it.

Eo tests: Test more cases with legacy events.

Eo legacy events: remove pointless string copies.

This was there because the old code modified the string. It is no
longer needed now that we just stringshare it.

Eo tests: Increase coverage for children iterator tests.

Eo: Remove useless safety checks.

Those can never happen, ever.

Eo tests: add more eo_data_get tests.

Eo: fix oversight for key_data

Eo: set object as const for key_data_get

Due to Eo4 changes, const is now important for this function.

build: finally enable branch coverage in our lcov-check target

This has been a long standing issue and I finally figured out the details to
get this working. Since we started with coverage there always have been some
problems to get branch coverage work (problems with older gcc versions, lcov
not taking them into account, etc)
The last detail that made me go nuts was that in my lcov version (1.10) there
is a bug which leads to geninfo not applying the config file and thus not
enabling the branch coverage like I defined in the config. I added the
--rc option to work around this case.

In my local run I get this now from lcov-check:
Overall coverage rate:
  lines......: 35.5% (65814 of 185169 lines)
  functions..: 44.6% (7661 of 17195 functions)
  branches...: 22.7% (31492 of 138942 branches)

So we have 22.7% branch coverage right now.

The vivid followers of my QA mails will also see the difference in numbers for
line and function coverage if one comapres my local results and the one on
Jenkins. This is another long standing issue and I need to figure out these
details next. :)

eina: rely on GNU make to find file at their correct location.

Eo: Fix typo of SUPER_TAG_SHIFT.

This commit fixes commit fc88037977dcc39dfd6d817c522cce01f5bfa024

eina: reintroduce additional support for eina sub directory compilation.

This is still experimental and quite hacky. Let me know if you face issue.
It should just work with make -C src/lib/eina normally...

eina: split Makefile with files/headers in preparation for additional per sub directory compilation

eolian-cxx: Fix Eolian C++ constructing methods with new eo_add

Also fix Eolian C++ examples using the new "eo_super".

eolian-cxx: Partial fix for generation without eo_do

eolian: new type system API

The Eolian type system API has been overhauled to properly separate declarations
(i.e. struct/enum/alias declarations) and usage. This should simplify writing
generators (as it makes it clear what is what) as well as make it easier to
maintain.

@feature

eolian_cxx: Fix C++ compilation with new Eolian_Type_Type values

eolian: type documentation and other cleanups

eolian: completely clean up the type system

eolian: clean up some temporary hashes

eolian: separate type_to_str for decls

eolian: remove database_type_print (unused)

eolian: remove most of the old type APIs

eolian: more old type api removals

eolian: remove retrieval funcs for type-style enums/aliases/structs

elua: update eolian bindings to new typedecl api

eolian: initial conversion of C gen and tests to new type APIs

eolian: add api to deal with typedecl free funcs

eolian: add some missing typedecl APIs

eolian: add matching APIs to get typedecls by file

eolian: add wrappers for most of typedecl APIs

eolian: remove unneeded hashes

eolian: actually generate typedecls as necessary

eolian: free the hashes (prevent leaks)

eolian: add typedecl delete func and init hashes correctly

eolian: rename typedef to typedecl to clear name confusion

eolian: add a structure + hashes representing type decls

eolian: always define the first enum value for consistency

eolian: add typedef enumeration for type definitions

ecore-input: Fix spelling errors in comments

This small patch just fixes up some spelling errors in comments. No
functional changes.

Reviewers: zmike, devilhorns

Reviewed By: devilhorns

Subscribers: cedric, jpeg

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

Fix more wrong migration to Eo4.

The if wasn't using {} so one of the statements was out of the scope
of the condition.
There was some misindented code.

Edje calc: Fix wrong migration to Eo4.

The if wasn't using {} so one of the statements was executed
unconditionally.

Eo examples: remove another reference to the eo examples.

Thanks to _ami_ for reporting.

Doc: Fix compilation after massive eo_do patch

Eo: Migrate to the new syntax (Eo4) and adjust the EFL

This is a merge commit for all of the relevant changes.

The syntax is described in: https://phab.enlightenment.org/w/eo/

Summary:
eo_do(obj, a_set(1)) -> a_set(obj, 1)
eo_do_super(obj, CLASS, a_set(1)) -> a_set(eo_super(obj, CLASS), 1)

eo_do_*_ret() set of functions are no longer needed.

This is the first step, the next step would be to also fix up eo_add()
which currently still uses the old syntax and is not 100% portable.

Manually fix extra eo_do calls that were added after I migrated

Examples: Remove eo examples.

Those were always bad, inaccurate and outdated, and are not really needed
because you should be using Eolian, and not Eo directly.

Fix migration script mistakes and compilation warnings.

Mostly unused vars following the removal of eo_do_ret().
However, there are some cases where the migration script got some things
wrong, and I had to manually fix them.

Eo tests: Update tests according to the new syntax

These tests don't use eolian, but are plain Eo, so they need manual
updating.

Automatic migration to Eo4.

I just ran my script (email to follow) to migrate all of the EFL
automatically. This commit is *only* the automatic conversion, so it can
be easily reverted and re-run.

Some automatic migration and some manual adjustments to calc and text.

These file needed some manual adjustments in addition to the automatic
migration, that's why these are separate from the previous and next
commits, so I can easily know there are additional changes to these, and
it wasn't just the script.

Manually semi-migrate files in preparation for the script

The migration scripts breaks with some weird cases, here I manually
migrated some parts, and just removed the eo_do from others without
actually migrating (so I could deal with that later).

Eo: Migrate to the new syntax (Eo 4).

The syntax is described in: https://phab.enlightenment.org/w/eo/

Summary:
eo_do(obj, a_set(1)) -> a_set(obj, 1)
eo_do_super(obj, CLASS, a_set(1)) -> a_set(eo_super(obj, CLASS), 1)

eo_do_*_ret() set of functions are no longer needed.

This is the first step, the next step would be to also fix up eo_add()
which currently still uses the old syntax and is not 100% portable.

@feature

eolian generator: @class methods take const object

eolian: fix generation tests

eolian: generate correct const first param on methods where applicable

eolian generator: use new call syntax

eolian generator: add obj as first param to header method prototypes

ecore_cocoa: add function attributes

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

autotools: fix code coverage on OSX

OSX uses clang as a default compiler instead of gcc.
Gcov does not exist for clang, but clang has an alternative which
requires additional compilier and linker flags.

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

ecore_cocoa: add missing documentation

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

ecore_cocoa: move Ecore_Cocoa_Cursor.h in Ecore_Cocoa.h

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

ecore_cocoa: full documentation of Ecore_Cocoa.h

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

ecore_cocoa: remove Ecore_Cocoa_Event_Window

Create specific structures for each event:
- Ecore_Cocoa_Event_Window_Focused
- Ecore_Cocoa_Event_Window_Unfocused
- Ecore_Cocoa_Event_Window_Destroy

They are currently hold the same data, but this will allow not to break
the event protocol when future extensions will be needed.

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

ecore_cocoa: rename width and height parameters

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

ecore_cocoa: better naming of events

Lost and got focused have been renamed FOCUSED and UNFOCUSED to mirror
the focus API in Elementary.

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

ecore_cocoa: replace Ecore_Cocoa_Window_Id by Ecore_Cocoa_Object

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

ecore_cocoa: unify API with Ecore_Evas

- Sizes are signed integers
- Width is named w
- Height is named h

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

ecore_cocoa: introduce Ecore_Cocoa_Object

This type is used as a bridge between objective-c objects (which are
ALWAYS pointed to) and the C interface.
Ecore_Cocoa_Object* is a less ugly substitute for void*.

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

ecore_cocoa: use Eina_Bool instead of int

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

ecore_cocoa: implement and const-correct window size step getter

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

ecore_cocoa: implement and const-correct window size max getter

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

ecore_cocoa: implement and const-correct window min size getter

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

ecore_cocoa: implement and const-correct window size getter

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

ecore_cocoa: implement and const-correct window geometry getter

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

tests cxx: remove superfluous output to stdout

These outputs are not giving any more information besides what we already get:

Ecore C++ headers compilePASS: tests/ecore_cxx/cxx_compile_test

(Ignoring the problem with the newline) The test name tells it all and we are
just filling the log.

build: fix problems with coverage check build where some files are not found

Switch to use a lcov config file which geninfo_auto_base and remove hard coding
the base dir  to src/lib. geninfo_auto_base is designed for a use case like
ours where we have several base dirs (lib, bin, tests, ...) and it detects them
automatically. This fixes failures in a coverage run where the file is simply
looked for in the wrong directory.

build: enhance generated html output from coverage report

We do not handle branch coverage so remove that part of the output. On the
other hand at a legend to allow people to understand the color meanings.