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.

Evas filters: Fix meaning of data 'execute' flag

The doc said it would behave like "value = data" but actually
the data part was executed directly. This should fix that.

Evas filters: Mark object as changed on data_set

This fixes the new test case.

Evas filter: Add new test case for state and data set

Edje calc: Simplify code readability with eo_do

This removes an absolutely crazy use of eo_do where all calls
to the efl_gfx_filter functions where factorized in an unreadable
manner. Hopefully eo_do will disappear soon.

Evas filters: Use fail_if instead of fail in test case

This will give more accurate information in case of failure.

Evas text: Add legacy filter APIs

Those should now be considered stable, even if their internals
may change. Also, these APIs are in Tizen so adding these will
help merging Tizen EFL and upstream.

Evas filters: Fix bad stringshare usage

Make sure the filter uses properly initialized state names and
don't call stringshare_del on a static string.

Efl.Gfx.Filter: Rework APIs and mark as stable

- Remove @beta flags,
- Update @since to match stabilization,
- Change methods to properties with keys,
- Use eo_prefix and add filter_ prefix to all properties since
  they use very generic names,

The filter API stays under Efl.Gfx since there are other kinds of
filters, and this one is the particular "graphical filter" or
"effect" API.

The EO API mostly not change from an application point of view,
except for "source_get" which now returns a string directly. Also,
state and data can now be queried.

efl - fix threadqueue spinlock destruction on openbsd - doesn't like it

so the spinlock on the threadqueue block pool it taken on shutdownn,
while the block pool is freed up then its is destroyed, but openbsd
very much doesnt like this and returns an error, so release the lock
before destroying it.

@fix

edje entry: fix gcc warnings about possibly uninitialized variables

this fixes warnings from gcc specifically:

lib/edje/edje_entry.c: In function ‘_edje_entry_imf_cursor_info_set’:
lib/edje/edje_entry.c:4104:4: warning: ‘dir’ may be used uninitialized
in this function [-Wmaybe-uninitialized]
    ecore_imf_context_bidi_direction_set(en->imf_context,
(Ecore_IMF_BiDi_Direction)dir);
    ^
    lib/edje/edje_entry.c:4099:24: note: ‘dir’ was declared here
        Evas_BiDi_Direction dir;
                                ^
                                lib/edje/edje_entry.c:4103:4: warning:
‘ch’ may be used uninitialized in this function [-Wmaybe-uninitialized]
    ecore_imf_context_cursor_location_set(en->imf_context, cx, cy, cw,
ch);
    ^
    lib/edje/edje_entry.c:4098:27: note: ‘ch’ was declared here
        Evas_Coord cx, cy, cw, ch;
                                   ^
                                   lib/edje/edje_entry.c:4103:4:
warning: ‘cw’ may be used uninitialized in this function
[-Wmaybe-uninitialized]
    ecore_imf_context_cursor_location_set(en->imf_context, cx, cy, cw,
ch);
    ^
    lib/edje/edje_entry.c:4098:23: note: ‘cw’ was declared here
        Evas_Coord cx, cy, cw, ch;
                               ^
                               lib/edje/edje_entry.c:4103:4: warning:
‘cy’ may be used uninitialized in this function [-Wmaybe-uninitialized]
    ecore_imf_context_cursor_location_set(en->imf_context, cx, cy, cw,
ch);
    ^
    lib/edje/edje_entry.c:4098:19: note: ‘cy’ was declared here
        Evas_Coord cx, cy, cw, ch;
                           ^
                           lib/edje/edje_entry.c:4103:4: warning:
‘cx’ may be used uninitialized in this function [-Wmaybe-uninitialized]
    ecore_imf_context_cursor_location_set(en->imf_context, cx, cy, cw,
ch);
    ^
    lib/edje/edje_entry.c:4098:15: note: ‘cx’ was declared here
        Evas_Coord cx, cy, cw, ch;
                       ^
                       lib/edje/edje_entry.c: In function
‘_edje_part_move_cb’:
lib/edje/edje_entry.c:4104:4: warning: ‘dir’ may be used uninitialized
in this function [-Wmaybe-uninitialized]
    ecore_imf_context_bidi_direction_set(en->imf_context,
(Ecore_IMF_BiDi_Direction)dir);
    ^
    lib/edje/edje_entry.c:4099:24: note: ‘dir’ was declared here
        Evas_BiDi_Direction dir;
                                ^
                                lib/edje/edje_entry.c:4103:4: warning:
‘ch’ may be used uninitialized in this function [-Wmaybe-uninitialized]
    ecore_imf_context_cursor_location_set(en->imf_context, cx, cy, cw,
ch);
    ^
    lib/edje/edje_entry.c:4098:27: note: ‘ch’ was declared here
        Evas_Coord cx, cy, cw, ch;
                                   ^
                                   lib/edje/edje_entry.c:4103:4:
warning: ‘cw’ may be used uninitialized in this function
[-Wmaybe-uninitialized]
    ecore_imf_context_cursor_location_set(en->imf_context, cx, cy, cw,
ch);
    ^
    lib/edje/edje_entry.c:4098:23: note: ‘cw’ was declared here
        Evas_Coord cx, cy, cw, ch;
                               ^
                               lib/edje/edje_entry.c:4103:4: warning:
‘cy’ may be used uninitialized in this function [-Wmaybe-uninitialized]
    ecore_imf_context_cursor_location_set(en->imf_context, cx, cy, cw,
ch);
    ^
    lib/edje/edje_entry.c:4098:19: note: ‘cy’ was declared here
        Evas_Coord cx, cy, cw, ch;
                           ^
                           lib/edje/edje_entry.c:4103:4: warning:
‘cx’ may be used uninitialized in this function [-Wmaybe-uninitialized]
    ecore_imf_context_cursor_location_set(en->imf_context, cx, cy, cw,
ch);
    ^
    lib/edje/edje_entry.c:4098:15: note: ‘cx’ was declared here
        Evas_Coord cx, cy, cw, ch;
                       ^

and the likes...

edje epp - fix gcc warning about possible unused var usage

gcc now is complaining about out ancient cpp code possibly using
newlines as undefined. this should keep this warning quiet - there
isnt a real performance issue here.

bin/edje/epp/cpplib.c: In function ‘cpp_get_token’:
bin/edje/epp/cpplib.c:4602:15: warning: ‘newlines’ may be used
uninitialized in this function [-Wmaybe-uninitialized]
       else if (newlines > 0)

@fix

evas render2 - move empty region data into const land so it is in RO mem

this should ensure these constants are in read-only memory if
possible iof they are marked as const as well as static

evas - render2 - silence new gcc warnings about possibly freeing static

This one is new:

In file included from lib/evas/canvas/render2/evas_render2.c:5:0:
In function ‘_region_break.isra.5’,
    inlined from ‘region_add’ at
lib/evas/canvas/render2/region.c:847:41:
lib/evas/canvas/render2/region.c:107:62: warning: attempt to free a
non-heap object ‘_region_brokendata’ [-Wfree-nonheap-object]
 #define FREE_DATA(reg) if ((reg)->data && (reg)->data->size)
free((reg)->data)
                                                              ^
                                                              lib/evas/canvas/render2/region.c:184:4:
note: in expansion of macro ‘FREE_DATA’
    FREE_DATA(region);

While it won't actually free is because if using brokendata the size
is 0 and it'll skip it, add in a check to see if region->data is the
brokendata static