Merging 'devs/q66/eolian_experimental'

This gets in several changes - such as new eolian API for deduplication
of names, support for eot files and processing of classes and typedefs
in sequence.

elua: use the new dedup API in lualian

eolian_cxx: use the new eolian dedup API

eolian: new API: eolian_function_full_c_name_get - reduces duplication of code

eolian: free the iterator

eolian_cxx/generator: support for eot files

elua: update lualian against latest changes

eolian: introducing support for .eot files

.eot files are similar to .eo, but they can't contain classes; all .eot files found in include paths
are parsed and information is added into the database. They're there for typedefs and eventually also
enums, structs and constants.

@feature

eolian: process classes and typedefs in order

evas/drm: Remove obsolete tty open function

We use ecore_drm for this already. Remove this non-used version.

evas/drm: Make use of ecore_drm for opening DRM device

We use this functionality already from ecore_drm. The evas version does
not even use udev to acquire the device which means we could not support
hotplugging. The only missing feature was the capability check for
DUMB_BUFFER which I added to ecore_drm now.

ecore_drm: Make struct Ecore_Drm_Device available

Allow the struct to be used by other parts and not only ecore_drm.
This is needed to use ecore_drm in the evas drm engine.

ecore_drm: Test drm device for DUMB BUFFER capability

Without this we can't do software rendering on the DRM device.

ecore_drm: Use CLOEXEC option when openning drm device

ecore_drm: Iterate over all available connectors not just the first

Maybe this was here for some debugging? It does not make any sense though
to loop over all connectors but break out of the loop after the first run.

lz4: fix shadow issue

@fix

ecore_win32: discard WM_MOUSEMOVE message if it has the same mouse coordinates than the previous one. See link in commit for an explanation

ecore_win32: activate an Ecore window when raising it

ecore_win32: fix the raise of an Ecore Window overlapped by another window when clicking on the title bar

ecore_win32: simplify logic

To avoid flickering, we have to manage WM_ERASEBKGND. This message
is sent because CS_HREDRAW and CS_VREDRAW are used when registering
the window class. So just omit CS_HREDRAW and CS_VREDRAW.

ecore_win32: fix typo.

evas: fix jpeg loader rotation by metadata (exif)

Summary:
90 or 270 degree rotation is not working properly
width should be regarded as height, and vice versa.

if this patch and D1082 were commited, rotation from metadata will be working properly by using evas_object_image_load_orientation_set()

@fix

Test Plan: add image object and invoke evas_object_image_load_orientation_set() -> load file with orientation metadata -> check whether image is rotated properly or not

Reviewers: raster, cedric, jpeg

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

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

jpeg loader: exif parsing bug fix

Summary:
problem: orientation information was not come out properly

byte align value was wrong. 0x4949 is II not MM

refer to http://www.media.mit.edu/pia/Research/deepview/exif.html

@fix

Reviewers: raster, cedric, jpeg

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

jpeg laoder: exif orientation test added

Summary:
currently, normal orientation tests are only added.
I'm going to add flipped orientation tests as well after I put related code in jpeg loader (currently it's not supported)

Reviewers: raster, cedric, jpeg

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

lz4: fix possible security issue.

See :
- http://fastcompression.blogspot.fr/2014/06/lets-move-on.html
- http://www.openwall.com/lists/oss-security/2014/06/26/25

@fix

windows: make those warning explicit about what they imply.

evas/gl - ... removed white trailing.

evas/gl: fix indentation.

evas/gl: improve the rendering quality.

To avoid texture bleeding in the texture atlas,
we adjust texture uv point as much as a half uv point.
Especially, this improves the rendering quality when the image has the border
area.

Unless apply this patch,
You might find the rendering result is different with software backened,
if the image has the borders.

In the software backened,
the border line was clear but the gl wasn't.
because the border line was interpolated so the rendering result was not the one we expected.

@fix

evas: fix warning and remove useless code.

evas: read fullscreen flags from initialized value.

evas: fix warning in DDraw backend.

evas: silent some warning during Windows backend build.

ecore_file: this parameter is indeed not used.

ecore_con: just a reminder to look at later.

ecore_win32: another forgotten read that is unecessary in current setup.

ecore_win32: silent warning by reading the returned value in a useful way.

eina: silence warning of unused parameter on Windows.

eina: use the right function prototype on Windows.

ecore_evas: remove warning from Windows backend.

Edje C++: Fix Edje.hh generation to update correctly.

edje: textblock style loading changed

Summary:
Keeping "font=", "font_size=" and "text_class" in tag->value fixes
style loading issues (after saving edj file with edje_edit_save method).

Reviewers: cedric, raster, seoz, Hermet, tasn

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

@fix

evas/png - better way to support png grayscale with transparency format.

suggested by cedric.

ecore-evas: Fix formatting and remove whitespace

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

ecore-evas: Fix cursor position getting reset to 0,0 when using
ecore_evas_cursor_object_set

NB: Previously, if we mad calls to ecore_evas_cursor_object_set to update the
mouse cursor hotspot, the mouse cursor would be repositioned at 0,0 on
the canvas due to x & y being set to 0,0. We fix that here by fetching
the current mouse position Regardless if we are changing the object or
not (ie: perhaps we are just updating the hotspot and not the object)

@fix

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

evas/gl - fix to render grayscale with transparency format in gl backened.

In case of png, grayscale with transparency format (transparency doesn't mean the png has alpha channel)
gl doesn't prepare that format render.
In this case, set it argb8888 to convert the data in the png loader.

@fix

ecore-evas: Fix some doxy grammar and spelling

NB: Rewrite the doxy paragraph for ecore_evas_object_cursor_set.
Previous documentation there was a bit confusing in how it read.

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

Documentation: adding missed documentation to edcref (map.color param)

Summary:
There is param (of the map block in part's description) that allows user to
set a color for specified vertex/point of that part.
That param was undocumented so actually no one know that this param is exist and can be used in edj file.

Two main fixes:
- adding documentation for map.color param in edc reference.
- adding .edc source code example for seeing how map.color param works.

@fix

Reviewers: cedric, Hermet, raster, seoz

Reviewed By: cedric

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

eina-tiler: Fix invalid read on eina tiler

Summary:
Fix invalid read on eina tiler reported by valgrind.
This revision will prevent access to data that was gained from eina iterator, after free of eina_iterator.

Test Plan:
1. Build enlightenment on devs/devilhorns/e_comp_wl branch with efl applyied this patch.
2. Run enlightenment with valgrind options.
3. build enlightenment with this patch
4. Run any wayland app on enlightenment
5. There will be no more invalid read message by valgrind.

Reviewers: cedric, devilhorns, raster, gwanglim, zmike

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

eina: fix build on windows.

Add necessary include for proper compilation.

@fix

Revert "eolian: process classes and typedefs in order (preparation for later potential extension)"

This reverts commit 7c28de061693fdc9a9409039692a4298bfb587c6.

Revert "eolian: generalized node append"

This reverts commit 98d25243aa053422307afab579ed59eb29f5b05f.

Reason: accidentally got into master... I'm tired.

eolian: generalized node append

eolian: process classes and typedefs in order (preparation for later potential extension)

elua: eo_prefix fix + add method name deduplication to lua generator

elua: eo_prefix_get will always return a valid value

edje: Edje_Edit - add edje_edit_sound_tone_add() function that allows user to add new tone to collection

Summary:
New function provides the ability to add new tones to currently loaded collection.
It takes the name that will define new tone in collection and its frequency as parameters.

@feature

Reviewers: cedric, Hermet, seoz, raster

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

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

edje: Edje_Edit - add functions that provide the ability to set and get source for proxy parts state

Summary:
This commit contains two new functions that provide the ability to set and get
source property of proxy parts state:
edje_edit_state_proxy_source_get() and edje_edit_state_proxy_source_set().
Also it contains the implementation of printing source data on
edc code generation.

@feature

Reviewers: cedric, Hermet, seoz, raster, reutskiy.v.v

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

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

edje: Edje_Edit - add missed iterator and object deletion in _delete_play_actions() function

Summary:
this commit adds missed eina_iterator_free() and evas_object_del() calls
in _delete_play_actions() functions.

@feature

Reviewers: cedric, Hermet, seoz, raster, reutskiy.v.v, tasn

Reviewed By: reutskiy.v.v

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

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

edje: Edje_Edit - add edje_edit_part_item_del.

Summary:
Delete exist item from part. This part should have BOX or TABLE type.
In parametr - name of exist item to delete it from BOX or TABLE.

@feature

Reviewers: cedric, Hermet, seoz, raster, reutskiy.v.v

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

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

edje: Edje_Edit - add API for missing map properties

Summary:
added getters & setters for
collections.group.parts.part.description.perspective.zplane
collections.group.parts.part.description.perspective.focal
collections.group.parts.part.description.map.rotation: x, y, z
collections.group.parts.part.description.map.smooth
collections.group.parts.part.description.map.alpha
collections.group.parts.part.description.map.perspective_on
collections.group.parts.part.description.map.backface_cull

@feature

Reviewers: cedric, raster, seoz, Hermet

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

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

ecore_win23: oops forgotten path.

ecore: fix ecore main loop on Windows when number of objects is greater that MAXIMUM_WAIT_OBJECTS

@fix

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

ecore_win32: fix key events on Windows.

@fix

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

evas/gl: eeeek. i missed removing one unnecessary checking

evas/gl: increase message level and avoid unnecessary checking

the checking is introduced previous my commit 5f52f5c9e5758f52f2bc9e4b7220c8c61076f4ea.

Evas ETC2: Remove some useless operations

No need to write out alpha in a RGBA color when only
the RGB values are used by the distance op.

Also, add a comment on the byte order. Maybe I'm wrong
but I believe the operations are fine wrt. byte order :)

evas/gl: don't go forward if the format is not supported.

evas/gl: abort is so cruel. print error and let it don't die there.

Evas ETC2: Remove some C99/GCC extensions code

There is still some C99 code in the file in the form of
  for (int k = 0; ...)

If there's a strong requirement not to use this form, I'll
change it, otherwise I find this specific code style more
readable (k is local to this iteration).

This patch and the previous one even give a ~10% speedup
on the encoding time. Sweet :)

Evas ETC2: Fix horrible warnings in etc2_encoder

So I guess always compiling with debug flags and no
optimizations isn't the best idea as some really bad warnings
can be hidden. Thanks raster for the notice.

Evas map: Add missing EINA_MAGIC check

eolian_cxx: support API deduplication like the main generator

eolian/generator: public api func name deduplication, spank tasn if it breaks

eolian: less code repeating

eolian: more cleanups

eolian: some less repeated code

eolian: small cleanup

eolian: mark all methods/properties in interfaces as virtual by default and disallow everything but class.(con|de)structor in their implements

eolian: simplify the parser a bit (unify 3 funcs into one)

Evas/Eolian: Eolian'ized Evas_Object events.

Summary:
The event descriptions of Evas_Object were hardcoded instead
of declared in evas_object.eo and thus no binding could be aware of
their existence.

Note: Evas_Canvas events were left as is.

Reviewers: cedric, tasn, q66, JackDanielZ

Reviewed By: q66

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

tests/eolian: fix for the new namespace syntax

eolian: fix end-of-file checking, move namespaces from :: to .

evil: actually commit the right fix.

autotools: remove forgotten BUILD_SAVER_JPEG

evas: fix invalid read reported in eina_rectangle_pool_free by valgrind.

Summary:
This fixes an invalid read reported by valgrind in eina_rectangle_pool_free
Reference: T1360

Signed-off-by: Sanjay Nirankari <sanjay.n1@partner.samsung.com>
Signed-off-by: Rajeev Ranjan <rajeev.r@samsung.com>
Reviewers: cedric, raster

Reviewed By: cedric

CC: sreedeep.m, sanjay, cedric
Differential Revision: https://phab.enlightenment.org/D1069

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

evas: fix build of jpeg saver on Windows.

eina: don't leak ressource due to the use of CreateThread on Windows.

Eina-Cxx: Added native type constructors and examples.

Summary:
This patch allows us to construct efl::eina:: types from
their respective C types.

Reviewers: cedric, felipealmeida, q66

Reviewed By: q66

CC: smohanty, woohyun, raster, cedric
Differential Revision: https://phab.enlightenment.org/D1068

eolian: remove semicolon termination of classes

Eolian-Cxx: Corrected the syntax of the .eo examples and added missing includes.

Summary: Just updating the .eo's to match the new syntax.

Reviewers: cedric, q66, tasn, JackDanielZ

Reviewed By: q66

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

evas: fix crash when preloading Evas_Object_Image.

This was a left over from the Eina_Rectangle patch improvement from Rajeev,
thanks Chris for spotting it.

eolian: fix a few casts

Introducing new lexer/parser for Eolian

This set of changes introduces a new, handwritten lexer+parser in Eolian. This should
allow for easier maintenance as well as easier feature additions. Unlike the previous
effort, this separates lexer and parser part (but they still run at the same stage).

In long term some more APIs should get exposed (for IDEs and other tools). The parser
is also stricter and more conformant to the actual format so several issues with
existing Eo files were caught previously.

Merges branch 'devs/q66/eolian_new_parser'

@feature

eolian: allow omitting of () in class with no inherits, fix up tests

eolian: stricten the allowed characters in values, and values starting with @ must be keywords - we no longer need all the arbitrary characters beacuse of lex_balanced and lex_until

eolian: fix double free

eolian: lex_until and lex_balanced

eolian: fix error handling and memory leaks

eolian: simpler type parsing