Revert "recursively hide edje group objects on edje hide"

Unless you like your desktop to look something like:

http://www.enlightenment.org/ss/e-53834e0c0cb3c1.87889787.png

then this commit really needs reverting.

This reverts commit 385326b8e4d7c05794670773aad6ece236ea71d9.

update po's

recursively hide edje group objects on edje hide

this massively improves edje performance when using groups, which previously would continue calculating their parts even when their parent object was hidden

CPU usage in my test case went from 20-30% to 1%.

@fix

Eolian: fix generation when optimization is set.

When using -O2 or -O3, the Eina_Bool legacy_support (unsigned char) was
overriding the int eo_needed.

The result was a failure during options check:
Eo flag is not specified (use --eo). Aborting eo generation.

@fix

Eolian/Generator: add support to namespaces.

The C generators now support namespaces generation.

Eolian: add support of namespaces.

It is now possible to define a class in a .eo file as e.g:
Elm::Widgets::Button.
Elm and Widgets will be the namespaces of the class Button.

@feature

Eolian: remove unused structures.

These structs were belonging to a feature that has been removed cause
not considered as needed.

Eolian/Tests: add test for events.

Eolian: modify API to allow future scalability

Until now, the functions giving access to class information were taking
the class name as parameter.
Except the fact that we needed to search into a hash table for the internal
class structure, no flexibility is possible.

This change consists in modifying most of the APIs using the class name
with a new Eolian_Class type and adapt the code of the C and C++
generators accordingly.

Eolian/Tests: add test for overriding functions

Eolian/Tests: add test for consts

This tests checks the different 'const' possibilities in the .eo file
and their good parsing.

Eolian: add test for generator.

The test checks the dev code generator.

Eolian/Generator: add support for implementation source file.

By using -gi option, the generator appends the functions that are
present into the given eo file and missing into the developer file
(given via -o option as an in/out file).

@feature

Eolian/Tests: move parsing tests to a specific file

Eolian: add support for typedefs.

It includes parser updates, database fill and tests on basic and complex
types.

One can define types in this way:
type Evas_Coord: int; /* Simple type definition */
type List_Objects: Eina_List * <Eo *>;

@feature

release: Switch back to development mode

Open for 1.11 changes now.

release: v1.10.0

autotools: Fix eolianc-xx header distribution.

Summary:
Sorry, last patch added the headers to the correct relative
location but replaced dist with nodist which is incorrect. nodist is
for generated headers only.

Reviewers: cedric, raster

Reviewed By: raster

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

autotools: fix Eolian-Cxx installation.

Summary: grammar/* is now installed to the correct destination.

Reviewers: tasn, cedric

CC: JackDanielZ, felipealmeida, smohanty, cedric
Differential Revision: https://phab.enlightenment.org/D895

Signed-off-by: Cedric Bail <cedric.bail@free.fr>

efl: remove some warnings.

Summary:
Mainly from the examples but also from libunibreak and tests/eet.

I'm not sure if it's really worth to remove warnings from the examples
-- because it adds pedantic-ness to something supposed to be didatic,
but I leave for you guys to judge.

Reviewers: tasn, cedric

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

Signed-off-by: Cedric Bail <cedric.bail@free.fr>

eina-cxx: remove a warning.

Summary: When assert is disabled 'diff' variable was unused.

Reviewers: tasn, cedric

CC: JackDanielZ, smohanty, felipealmeida, cedric
Differential Revision: https://phab.enlightenment.org/D893

Signed-off-by: Cedric Bail <cedric.bail@free.fr>

autotools: disable automake portability warnings.

Summary:
Vanish with these:
src/Makefile_Eolian_Helper.am:15: warning: '%'-style pattern rules are a GNU make extension
src/Makefile.am:36:   'src/Makefile_Eolian.am' included from here
src/Makefile_Eolian.am:42:   'src/Makefile_Eolian_Helper.am' included from here
src/Makefile_Eolian_Helper.am:18: warning: '%'-style pattern rules are a GNU make extension
src/Makefile.am:36:   'src/Makefile_Eolian.am' included from here
src/Makefile_Eolian.am:42:   'src/Makefile_Eolian_Helper.am' included from here
src/Makefile_Eolian_Helper.am:21: warning: '%'-style pattern rules are a GNU make extension

Reviewers: tasn, cedric

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

Signed-off-by: Cedric Bail <cedric.bail@free.fr>

Updating czech translation

Eolian: fix class lookup during parsing.

If classes A and B are stored in a same file, when A inherits from B,
and A needs to be generated, the generator, even if B is parsed, will
not search the class in the database but will look for some b.eo.

This patch fixes that issue by checking the existence of the classes
into the database before looking for a corresponding file.

evas/text: fix abnormal ellipsis logic when harfbuzz is enabled.

Summary:
text_props.len indicates visual length from harfbuzz logic when it is available.
But, input pram "idx" is logical length. So, ellipsis works abnormally in some case.
@fix

Test Plan: D870

Reviewers: tasn, woohyun

CC: cedric, herdsman
Differential Revision: https://phab.enlightenment.org/D871

evas/text: add text ellipsis test case for ligatures special case.

Summary:
Some fonts has combination information for "ff".
When harfbuzz is enabled with the font, evas text ellipsis logic can be broken.

Reviewers: tasn, woohyun, cedric

Reviewed By: tasn

CC: cedric, herdsman
Differential Revision: https://phab.enlightenment.org/D870

Some comment/commit message improvements by TAsn.

Eo: Add function name to OP desc on Windows

Match function names when the API pointer is out of range.
Reviewed by TAsn and modified according to his comments :)

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

Win32: Fix invalid free in eio monitor

You can't just duplicate a string once and free it three times...

edje: Edje_Edit - fix wrong behaviour on draggable.event processing while deleting parts.

Summary:
the draggable.event field (in edje) is pointing on the id of the part it is assigned to.
The main problem is about deleting parts.
After part being deleted, all other parts (that are below the deleted part) have their id shifted,
but event doesnt shift.

For example,
1) draggable.event at some part pointing at part with id=7,
2) some other part is being deleted
3) part with id=7 have new id now (id=6)
4) draggable.event DOESN'T change it's pointer and still points at part with id=7 (which is now has id=6)
5) if there is no parts with id=7 anymore, then it could give you SEGFAULT someday (or not, depending on luck).

This commit contains next changes:
1) after deleting part, draggable.event also shifts (if required)
2) after deleting part that is being pointed with event by someone, it will remove all event points at it
@fix

Reviewers: cedric, raster, seoz

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

Signed-off-by: Cedric Bail <cedric.bail@free.fr>

Win32: Add configuration to cross-compile

Signed-off-by: Jean-Philippe Andre <jp.andre@samsung.com>

Win32: Fix ecore_evas engine search path

Engines are stored in a lib/ folder, while the main DLL files
are in the bin/ folder, so the engine would never be found.

A solution was to add the proper checkme file in the share
folder, but since this is necessary only for Windows, we
can simply use ../lib instead of using the full eina_prefix
detector.

Thanks vtorri for the review.

update po's

autotools: Fix Makefile_Ecore_Audio_Cxx.am.

Summary: This patch fixes T1246.

Reviewers: cedric, smohanty, stefan_schmidt, raster

Reviewed By: raster

CC: felipealmeida, cedric
Maniphest Tasks: T1246

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

Updating translations

 release: v1.10.0 beta3

evas/render - fix segfault because of null cow access.

issue ticket: T1252

@fixed

efreet: Fix mistakenly inverted behavior in URI handling

Fixes a mistake in commit 78c9a82a142f619ccd0957957351fe0e573de76c

The authority part would get placed in the struct member "path",
and vice versa.

Lesson learned: Don't rewrite your patch late at night after
arc has messed it up.

evas/gl_x11 - replace inlist counting to one api call.

efreet: Handle URIs in a more generic way

Summary: Fixes T1219

Reviewers: englebass

CC: cedric
Maniphest Tasks: T1219

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

update po's

Eolian/Lexer: fix support of const parameters.

The syntax to define a parameter of a property as const is as following:

set {
   data: const;
}

When no space was present between ':' and "const", a syntax error was
occurring.

Eo: Make it clearer that a NULL deref can't happen.

We check _current_table for NULL, and then populate it (it's a global)
through another function, but we don't really check it's not NULL before
using it, we just assume because of an indirect other variable.

This confused coveritiy, can confuse humans too, and in general risky
(if something changes).

CID 1039419

Eo callbacks: Fix a possible issue of corrupted callback list.

This happens when we delete more than one callback at the beginning
of the list (i.e 2 or more) in one go.

Eo: Don't write to pointer after free.

Fixes coverity: CID 1039898.

Revert "Eo: Fix deref after free."

Coverity found the issue, but it's an issue we've already fixed in the
past. I don't know how it got lost, but it seems like someone did a bad
merge. Probably when migrating to Eo2.

This reverts commit 831c20464d212fde8d6ee125342e90e5e64efc1e.

Eo: Fix deref after free.

In some rare cases it was possible for a pointer to be referenced after
it was already freed. This is now fixed thanks to coverity.

@fix

CID 1039898

edje_calc: Document some intended fall through cases in switch-case

We share code here for different types. Make a note and also marked them as
false positive in coverity.

CID 1039377, CID 1039376

ethumb: Memset whole contex in md5 implementation

I don't know what they tried here but if we consider the content
of ctx sensitive we should memset the whole thing.

CID 222382

ecore_imf: Account for terminating null byte on copy

strlen retruns the count with the terminating null byte but strncpy expects
length with it. Account for that.

ecore_imf: Avoid potential buffer overrun

We can't just blindly copy the full string into a 128 byte buffer. Adding
some check here to actually stay below the buffer limit or return.

CID 1193233

fix ecore_thread_global_data_wait to work when no data exists yet

@fix

this is based on feedback from wonil choi, where this didn't work
right. this also has an if check for the hash itself.

fix behavior of edje entry to match other toolkit entires

this fixes T1229 in the part about the cursor behavior when you press
updown/left/right when a selection is there. @fix

edje_calc: Fix invalid parameters during first calc

In particular, ellipsis is -1 by default in Evas, but at this
point (first layout calc), the parameters used for recalc are
incomplete and ellipsis would then be 0 by default (calloc).

As a consequence, Edje will call ellipsis_set(0) enabling
ellipsis even on objects that force "ellipsis: -1".

Solution: set all the parameters before entering text/tb calc.

I believe the other changes are only color and image padding
and should not affect recalc_single.

fix ecore_con_url using 100% cpu and just let fd handler do work

this fixes T1251 - @fix

evas/wayland_shm: unlink the REAL temporary file

Signed-off-by: U. Artie Eoff <ullysses.a.eoff@intel.com>

Eolian/Generator: fix the class description generation.

The class description was not used during the generation of the legacy
and Eo headers.

Evas filters: Fix NULL dereference (legacy_strdup)

CID 1210809
CID 1210813

+ remove comment

Evas filters: Fix NULL dereferences

CID 1210807
CID 1210808

Evas filters: Remove logically dead code

CID 1181889

edje/Edje_Edit: _edje_edit_real_part_add init field of Edje_Real_part

Summary:
To correcly display the added pats using the edje edit api, update
init the filds of Edje_Real_Part. Initialize the next filds: type and
typedata. It filds initialized accordingly to Edje_Part_Type value.

@fix

Reviewers: raster, cedric, seoz, Hermet

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

Signed-off-by: Cedric Bail <cedric.bail@free.fr>

edje_calc: Don't add padding twice to text objects

Text objects declared in Edje will see their padding added twice,
as the Evas_Object_Text itself contains the padding already.

This WILL break some EDC files. It's a bug nonetheless.

Should this be backported?

edje_text: Remove no-op pad_get

This call to pad_get is useless.

edje_calc: Update some disabled code so it can compile

Yup, that's basically a no-op but allows other devs to test
the old behaviour of that function as it's still there:
_edje_part_recalc_single_text

add soem class description to eo timers to test if it gets generated

de-elipsis edje

the word is "ellipsis". it's bad enough that we have released misspelled api, but we can at least spell things correctly internally.

edje_cc now prints helpful warning messages when it compiles broken text parts

edje_cc should throw errors when inheriting a part from the same part

release: v1.10.0 beta2

Revert "eolian-cxx: re-enable Makefile_Ecore_Audio_Cxx installation and tests."

This reverts commit 1e592d22b77f1e818e0bc4d1350abf59f146dcc2.

ecore-drm: Send proper keycode to ecore key event, and remove
erroneous variable.

@bugfix: Since later functions which process the ecore key event will
end up subtracting X's broken keycode values from This original
keycode value, we had better send the proper code with the offset
already added.

Signed-off-by: Chris Michael <cp.michael@samsung.com>

evas/evas3d: reset image properies if the image is used for scene.

eo ecore timer - improve documentation in the eo file

evas/examples - revised evas-3d-proxy

evas/evas3d: discard proxy textures list properly.

this fixes evas-3d-proxy example to be exited correctly.

evas/evas3d: fix proxy texture source set.

the appeneded texture should be object itself but not private data.

Compilation: align EOLIAN, EDJ with CC and others

Hate seeing those unaligned things:

  EOLIAN   blah
  EDJ   blah
  CC      blah
  CC      blah

OCD mode :)

Evas: Fix abort() when using ETC2 with alpha

Ecore evas: Fix crash after image_data_set

In some cases, the image has a format different from RGBA,
so image_data_set will be invalid as the new data is expected to
be RGBA. This happened with ETC2 images.

evas/evas3d: applied eo intensively and fix indentation.

ecore-x - fix setting protocol atoms for icccm for 64bit

@fix

evas: remove logically dead code.

CID 1211984.

Note that in error case, the calloc is either not passed or
returned NULL so free will never be called.

evas: fix uninitialized value usage in Evas_3D.

This code look suspiciously similar to src/lib/evas/canvas/evas_3d_mesh.c
line 682.

evas: move the type and enum of Evas_3D into the unstable part of the API.

I am concerned that we do duplicate the color format of a texture insteed
of reusing the colorspace enum from Evas (that already exist). I believe
this is a side effect of not using Evas_GL_Image for texture.

This let me think that I wont allow this API out of BETA until we reduce
duplication by using Evas_GL_Image as it seems it will also impact the
public API of Evas_3D.

evas: fix uninitialized value in Evas_3D.

CID 1211986.

evas: remove left over Evas_3D public/released API.

evas: fix use of uninitialized value in Evas_3D_Mesh.

evas: fix uninitialized value being used when the engine doesn't support Evas_3D.

CID 1211988

autotools: define @datarootdir@ explicitly to avoid config.status warnings.

Summary:
This patch removes warnings like this:

config.status: WARNING:  '/home/savio/src/efl/pc/eo.pc.in' seems to ignore the --datarootdir setting

Reviewers: cedric, JackDanielZ, smohanty

CC: felipealmeida, cedric
Differential Revision: https://phab.enlightenment.org/D843

Signed-off-by: Cedric Bail <cedric.bail@free.fr>

eolian-cxx: remove annoying compilation warnings and messages

Summary:
- remove an automake warning caused by a variable redefinition
- remove unnecessary verbosity during debug compilation

Reviewers: cedric, smohanty

CC: felipealmeida, cedric
Differential Revision: https://phab.enlightenment.org/D844

Signed-off-by: Cedric Bail <cedric.bail@free.fr>

redo lazEDC parsing a bit to fix desc.state omission

lazEDC missing keywords for scale/noscale

fix be9b27ac99dc4fcb49c340b8bdf4bfcbe4240e5f for edje_cc

do_params is set when expecting parameters, not req_params

lazEDC syntax update: imprecise -> noprecise

ecore_x_image_get() should not abort when shm creation fails

...also when shmat() fails

ecore-x should print an error when shmget fails

examples: fix eolian_cxx Makefile.am to work with --with-eolian-cxx.

Summary:
Includes Makefile_Eolian*_Helper.am and -- since
_EOLIAN_GEN_DEP and _EOLIAN_CXX_DEP can't be resolved from
src/examples -- redefine the generation rules locally.

Reviewers: cedric, stefan, stefan_schmidt

CC: felipealmeida, cedric
Differential Revision: https://phab.enlightenment.org/D836

Signed-off-by: Cedric Bail <cedric.bail@free.fr>

eina-cxx: add EFL_PTHREAD_CFLAGS to EINA_CXX_CFLAGS and EFL_PTHREAD_LIBS and -lm to examples

Summary:
EFL_PTHREAD_CFLAGS are needed for tests and users that use
efl::eina::thread's and EFL_PTHREAD_LIBS to eina_cxx, eo and evas examples.

Reviewers: cedric, stefan, stefan_schmidt

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

Signed-off-by: Cedric Bail <cedric.bail@free.fr>

eolian_cxx: pass eo_class by reference instead of by value.

Summary: This patch gets rid of a "defect" pointed out by Coverty Scan -- as per jackdanielz request.
CID 1211985.

Reviewers: cedric, smohanty, JackDanielZ

CC: felipealmeida, cedric
Differential Revision: https://phab.enlightenment.org/D840

Signed-off-by: Cedric Bail <cedric.bail@free.fr>

eolian-cxx: re-enable Makefile_Ecore_Audio_Cxx installation and tests.

Reviewers: cedric, stefan, stefan_schmidt, smohanty

CC: felipealmeida, cedric
Differential Revision: https://phab.enlightenment.org/D839

Signed-off-by: Cedric Bail <cedric.bail@free.fr>