Evas masking: Fix GL masking with maps and stuff

This fixes an issue spotted after the previous fix.
Passing the Y-invert flag is necessary because in the usual
case a map is rendered with Y-invert (OpenGL coords vs. Evas coords)
but in case a map is rendered in an FBO (another map's surface)
then Y-invert must be unset.

Evas masking: Fix another issue with masking

Fixes rendering in the following case:
- Object with a map has a mask
- Object is child of smart object which also has a map (eg. transit)
--> Masking did not apply to the children before this patch.

NOTE: This works fine in SW but still didn't work in GL, see the
      following commit...

elua: correct eo inheritance behavior

Now we try to treat first inherit as a parent and everything else as a mixin.
If the first inherit is not a class (interface or mixin), every inherit is
treated as a mixin.

@fix

eldbus: fix memory leak.

Summary:

@fix

Reviewers: felipealmeida, larryolj, cedric

Reviewed By: cedric

Subscribers: cedric

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

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

Ecore_con: fix initialisation clean up and avoid memory leak.

Summary: When ecore_init failed or when failed to initialize ecore_con_log_domain, then evil_shutdown was not called. So cleaned up code and added the fix.

@fix

Reviewers: cedric

Subscribers: cedric

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

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

evil: prevent _evil_init_count from going below zero.

Summary: _evil_init_count should never go below zero. This can occur, if a developer mistakenly calls evil_shutdown before calling evil_init. So fixing the code so that it never goes below zero.

Reviewers: cedric

Subscribers: cedric

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

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

eldbus: production code inside Eina Safety Check assert.

Summary: Eina Safety Checks assert macros don't do anything (no-op) if EINA_SAFETY_CHECKS is undefined.

@fix

Reviewers: felipealmeida, larryolj, cedric

Reviewed By: cedric

Subscribers: cedric

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

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

evas: Evas_3D - fix bug with uninit Evas_Mat4 flag.

Reviewers: Hermet, raster, cedric

@fix

Reviewed By: cedric

Subscribers: cedric

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

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

edje: Fix typo in a documentation while reading it.

ecore-x: Port deprecated API and new API function to xcb

Summary: This ports the deprecated screensaver_suspend and new API
screensaver_suspend to work with xcb.

@fix

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

ecore-x: Deprecate old misspelled API function and add new one

Summary: This deprecates the old misspelled screensaver_suspend API
function and adds a new one.

@fix

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

ecore-x: Deprecate misspelled screensaver API function

Summary: This API function is misspelled, so after speaking with
Stefan, we've decided to deprecate this function add add a properly
spelled one.

@fix

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

Ecore_con: preventing _init_con_ssl_init_count to go below zero.

Summary: _init_con_ssl_init_count should not go below zero. This can occur if a developer mistakenly calls ssl shutdown before calling ssl init. So adding the check to prevent this.

Reviewers: cedric

Subscribers: cedric

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

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

eina: fix valgrind invalid read of size in eina_file_path_sanitize.

Summary:
Apparently eina_tmpstr_strlen counts the null character as well. This
doesn't follow how strlen works, as the latter excludes it from the count.
This resulted in mistreatment of the string in _eina_file_escape, with
tmp_str paths that had "../".

This fix will do for now, but it is advised that we avoid using
eina_tmpstr_strlen, to prevent such confusions in the future.

Test Plan:
The following lines will throw a valgrind 'invalid read of size 1' error
prior this fix:
  char *path = "home/mydir/../myfile";
  Eina_Tmpstr *tmp_str = eina_tmpstr_add(path);
  char *ret_path = eina_file_path_sanitize(path);

@fix

Reviewers: cedric, stefan_schmidt

Subscribers: tasn, cedric

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

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

edje: quick fix to guarantee that ed->path is always correctly set.

NOTE: It will be better to completly remove ed->path in favor of using ed->file->f,
but that would be way to intrusive for this release cycle.

ecore-x - undo api/abi break with input selecting before release

update po's

eet - fix endianess break added by cedric a year ago in image decoding

@fix

this fixes T1987 - the commit that broke it was:

commit 18d494489c5651aa152622a56579bbc1bda853cb
Author: Cedric BAIL <cedric.bail@samsung.com>
Date:   Fri Jan 3 16:20:53 2014 +0900

    eet: use eina_swap*() function instead of custom slower one.

Evas masking: Fix rendering of restacked masks of masks in a map

I know. This title does not explain anything. Whatever.

This fixes the following issue:
- Mask a genlist (big mask)
- Each item has an icon masked (small mask)
- Apply a map to the genlist
- Scrolling the genlist
--> The big mask still works but totally screws up the
    small icons with masks.

Note: Once again this patch only affects code paths where an
      object is a mask.

Evas masking: Fix some remaining issues with animations

Yeah, mixing maps and masks of masks in a genlist leads
to tons of amazing bugs :)

This commit removes x,y from the "mask" field in an object,
as they are duplicates of cur->geometry.{x,y} but were not
properly kept in sync.

This patch fixes a situation of:
- A genlist in a map
- Each item has an icon masked
- Scrolling the genlist
--> The masked items would not render properly before this
    patch.

Remaining known problem:
- Mask a genlist (big mask)
- Each item has an icon masked (small mask)
- Apply a map to the genlist
- Scrolling the genlist
--> The big mask still works but totally screws up the
    small icons with masks.

Note: These changes look scary just before the release
      but I would have to backport them to 1.13.x as they
      definitely are bug fixes. Also, they only concern
      code paths used exclusively by masking.

Evas masking: Fix masking of multiple objects in a map

All those masking bug fixes become harder to explain. But here goes:

 - Take a genlist, apply a mask to it (for example put everything
   in an elm_layout). Also play with various objects in the genlist.
 - Also apply a map to it (for instance, elm_transit zoom).

 --> Now some elements will be masked, some others will not,
     and some may even not render at all.

This patch restores a mask in the current drawing context, instead
of just unsetting it.

Evas masking: Forward previous mask where needed

In a situation where an object with mask of mask is in a map
(Yes! It can happen!) the masks would not get properly "multiplied".

Now the problem is that some objects still seem to bypass some
masks... Hmm...

Evas masking: Prevent drawing masks in the wrong surface

Masks should be actually rendered only to their specific target
surface.

ecore-drm should probably send the expected (reversed) scroll z value

ecore-wayland: Don't send key repeat events for keys which should not
get them

Summary: Some keys in certain keyboard layouts should not be repeated
so add a test which uses xkb to inform us if this key supports
repeating.

@fix

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

Updating serbian translation

ecore_con: remove code in examples that triggered warnings.

Summary:
When the examples were compiled they used to through the below warning
$ gcc -o ecore_con_client_simple_example ecore_con_client_simple_example.c `pkg-config --libs --cflags ecore ecore-con eina`
ecore_con_client_simple_example.c:11:0: warning: "EINA_UNUSED" redefined
 # define EINA_UNUSED
 ^
In file included from /usr/include/eina-1/Eina.h:215:0,
                 from /usr/include/ecore-1/Ecore.h:304,
                 from ecore_con_client_simple_example.c:5:
/usr/include/eina-1/eina/eina_types.h:112:0: note: this is the location of the previous definition
 #  define EINA_UNUSED __attribute__ ((__unused__))
 ^
So removed these defines as they are no longer required now.

Reviewers: cedric

Subscribers: cedric

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

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

release: Update NEWS and bump version for 1.13.0-beta3 release

Evas masking: Check only the variable we want to set

This is a left-over from a previous fix a few weeks ago.
The point of this "if" is just to avoid writing the COW value
if not needed.

For reference:
  commit f876cf31f8d6fd48f7602bd593c24c1a07138d8f
  Author: Jean-Philippe Andre <jp.andre@samsung.com>
  Date:   Tue Dec 23 18:57:45 2014 +0900
    Evas masking: Fix invalid geometry after mask redraw

evas callback: Fix typo in documentation of evas_object_event_callback_del_full

Reviewers: cedric, seoz

Subscribers: cedric, seoz

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

evas/examples: just change file name.

[Evas: Evas_3D] Improve shadows example.

Reviewers: cedric, raster, Hermet

Subscribers: cedric

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

ecore-drm: Trap for failure to dup stdin

Summary: If we fail to duplicate stdin, we should write out an error
and exit appropriately.

@fix

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

ecore-drm: Fix formatting

NB: No functional changes, just formatting

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

Revert "eina: eina_file - fix "invalid read of size 1" in valgrind"

This reverts commit f52f5628916f44a6b8ce1fdd3b9cc40f6c997eda.

This is reverted because it breaks eina_file_path_sanitize when using
  "/../" in paths, for example:
eina_file_path_sanitize("/home/../mydir/myfile")
  returns: "/mydir/myfili"

What invalid read size does this fix? Why was no test case specified?
Anyway, this change affects too much code to leave it in like this.

edje: edje_entry - remove dead code

Edje: edje_edit - abort Recursive Reference in edje_edit_part_source_set

It is unable to do recursive reference such as:
>   Having group A with GROUP part that has group B as source.
>   Having group B with GROUP part that has group A as source.
Here we have a loop that is not allowed by edje_cc, so edje_edit also need
to check this case.

@fix

ecore-wl should maybe clear its repeat timer for keyboard on window free

eina: eina_file - fix "invalid read of size 1" in valgrind

@fix

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

eina: fix tests suites for Windows to have a predictable result.

@fix

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

ecore_win32: don't allocate Ecore_Event_Key events with NULL keys

If "keyname" and "key" are NULL an Ecore_Event_Key is allocated
throught calloc. strlen() is called on these variables, which crashes
the program. As it is not correct to return a key event for NULL
keys to ecore, it is preferrable to return NULL and exit the function
before the key event is allocated, which fixes the crash in the same
time.

@fix

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

eina: use uintptr_t for Eina_Thread storage to be the same size for the different architectures.

This should not be an ABI regression on the platform where it work (Unix). It is on Windows,
but the API was unusable at all without this change, so this is a necessary breakage to
unbreak things.

@fix

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

check: fix tests suites on Windows

Disable timeout because we cannot fork on Windows.

@fix

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

eolian: fix to pass make check on windows

@fix

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

eo: fix to pass make check on windows

@fix

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

autotools: fix make check build error on Windows

Windows does not use ECORE_X.

@fix

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

eio: Win32 make GetOverlappedResult function non-blocking

@fix

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

evil: get right Windows environment variables for home path.

@fix

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

evil: add windows compatible langinfo.

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

Edje: edje_cc - abort recursive Reference that is made by GROUP parts

Summary:
It is easy to create edj collections that aren't working at all and
edje_cc easily allows user to do that.
For example:
>   Having group A with GROUP part that has group B as source.
>   Having group B with GROUP part that has group A as source.
In this case edje_cc compile source code perfectly, but if user try to load
this edje as layout or use together with edje_edit it will cause unexpacted
and wrong behaviour.

@fix

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

Reviewed By: cedric

Subscribers: cedric

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

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

evil: fix SEGFAULT in strcasestr

Summary:
there was an unsigned int underflow.

@fix

Test Plan: strcasestr("a", "bbb");

Reviewers: cedric, raster, Hermet, seoz

Subscribers: cedric, reutskiy.v.v

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

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

eolian: fix unary expression parsing (correctly consume tokens)

Also disable typed expression validation for now. The interfaces are not
finished yet so some things will fail validation because of unexistent
type definitions.

Fixes T2064.

@fix

evil: fix initialisation of the socket library.

@fix

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

Set the name for table, box items

If box or table items haven't name in the edc we generate a new
name by mask "item_0xFFFFFFFFFFFFFFFF". Item name important for
Edje_Edit.

@fix

embryo_cc_sc2.c: prevent memory corruption.

Summary:
Modified code to following coverity result.
(overflow the memory access.)

@fix

Reviewers: raster, Hermet

Reviewed By: Hermet

Subscribers: cedric

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

edje: Edje_Edit - fix setting NULL into string pointer

Some of edje_edit functions allows to set up NULL instead of string,
but by using _edje_if_string_free() function it should set variable into NULL,
so previous data won't appear anymore.

@fix

evas: fix some clipping issues before/after Evas_Map animations

Summary:
This is an attempt at fixing:
 - T1767: The ultimate evil map & clip bug

Force recalculation and re-propagation of clipper geometry
after or just before a map is applied (only when transiting
between map enabled and map disabled).

I realized that doing clip_unset+clip_set in the E widget
code would fix the issue, but this is not a solution that
makes a lot of sense.

Unfortunately I have no idea about the side effects of this
patch, especially in terms of performance.

Fixes T1767 and maybe T1630.

Test Plan:
Open PackageKit popup in E, check the animations
and that clipping works fine both during, before and after
the animations.

Reviewers: raster, cedric

Reviewed By: cedric

Subscribers: cedric, Hermet

Maniphest Tasks: T1767

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

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

evas-image-loaders: Fix compiler warning of comparison between signed
and unsigned values

@fix

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

evas-software: Fix compile errors when using xcb

Summary: This fixes compile issues when building evas software engine
using xcb. When calling xcb_outbuf_free, the Render_Engine has no 'ob'
field, so remove that call. Also fix call to
render_engine_software_generic_init using the proper function
parameters.

@fix

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

evas-software: Fix software engine to compile using xcb

Summary: Fix several compile issues with software engine when compiled
against xcb. This fixes an improper call to
evas_software_xcb_outbuf_flush with improper parameters, and adds a
missing EINA_UNUSED to that function.

@fix

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

evas-software: Fix call to evas_software_xcb_outbuf_setup function

Summary: Seems we were missing a comma here and this was causing a
compile break.

@fix

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

ecore-xcb: Add missing EINA_UNUSED for function paramater

Summary: Depth is not used in the ecore_x_window_full_new function, so
add missing EINA_UNUSED

@fix

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

ecore-xcb: Fix ecore_x_window_full_new function to compile again

Summary: Let's use the proper variable names when assigning return
mask values.

NB: How did all this even get Into git with someone checking it and
compiling it ??

@fix

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

ecore-xcb: Fix function formatting

Summary: No functional changes, just formatting

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

ecore-xcb: Fix compiler build break

Summary: Use proper variable name when when trying to free selections
data

@fix

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

ecore-xcb: Fix unused paramater compiler warning

@fix

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

ecore-x: Fix build error in xcb

Summary: If we do not get the property we are looking for here, then
reset the Proper variable to 0. This fixes a build break.

@fix

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

ecore-x: Use proper XCB_ATOM_CARDINAL in call to window_property_get

Summary: XA_CARDINAL is defined for xlib, XCB_ATOM_CARDINAL is defined
for xcb, so let's use the proper one and fix a build break.

@fix

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

ecore-x: Remove unused variable in backlight_available code

Summary: We never actually do anything with the _backlight variable
here, so just remove it

@fix

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

ecore-x: Add missing EINA_UNUSED for function paramater

@fix

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

ecore-x: Fix calls to disable dpms in xcb

Summary: We already have ecore_x_dpms code which handles this
properly, so call it here.

@fix

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

elocation: Hide also generated headers behind the BETA API define

evas: Add since tag for new evas_textblock_cursor_range_simple_geometry_get() API

ecore_x: Add since tag for new ecore_x_input_select() API.

ecore_input_evas: Add since tage for new ecore_event_evas_axis_update() API

ecore: Add since tag for new ecore_thread_wait() API

Added in this cycle but no since tag was given.

Evas render: Remove useless code

While invesigating some clip & map issues, I found some very
strange piece of code:
{
  tmp = a;
  a = c;
  a = tmp;
}

This actually comes from a very old code refactoring where a
line in-between was removed:

   tobj = obj->cur.map_parent;
   obj->cur.map_parent = obj->cur.clipper->cur.map_parent;
 - evas_object_clip_recalc(obj);
   obj->cur.map_parent = tobj;

Adding this line back there doesn't seem to do anything anyways.
So, let's just remove useless code.

For the record (legacy evas):
  commit e1f6f3c5f239dfd95a307949acd5f98831c0c3c0
  Date:   Fri Aug 17 06:16:04 2012 +0000
  evas/render - code refactoring.
  SVN revision: 75351

edje_cc must error when first provided state description is not "default" 0.0

correctly @fix T1926 without breaking existing apps/syntax

Revert "edje - edje_cc - fix wrong state lists where default is not the first"

This reverts commit 0cb33a46758bd1f66653e97d7ad027a9529b1279.

broke existing edc syntax. SPANK SPANK SPANK!!

Revert "efl - edje_cc - fix default state checks in previous commit"

This reverts commit 82014cad2c11b40e455e0578d22f09b04ebd1e5c.

eet: add warning about security to be sure people don't use this function in the wrong context.

Merge branch 'devs/devilhorns/ecore_drm'

This branch fixes several issues with ecore_drm:

In particular, it address the issue of not properly restoring the tty
on shutdown, saves the current tty keyboard mode and properly restores
that on shutdown, we now get the VT from systemd (if available) and
properly setup the VT.

@fix

ecore-drm: Remove session parameter for session take/release functions

The functions to take or release a session no longer accept session
name as a parameter, so fix calls to those functions

@fix

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

ecore-drm: Remove unused function parameter

The session take/release functions no longer need the session name
paramater, so remove that.

@fix

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

ecore-drm: Remove session parameter for internal function

The session take/release functions no longer require the name of the
session, so remove the function parameter

@fix

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

ecore-evas-drm: Fix improper shutdown sequence

Summary: As the launcher code now uses some fields from the
Ecore_Drm_Device, we need to free the device After the launcher has
been disconnected.

@fix

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

ecore-drm: Remove logind header file

Summary: These functions have been moved to the private header file
now, and thus this file is no longer needed.

@fix

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

ecore-drm: Remove logind header file from build order

Summary: Since we removed the need for the logind header file, let's
remove that from the build order.

@fix

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

ecore-drm: Move session and vt fields to store inside the
Ecore_Drm_Device

Summary: This moves storing the session name and VT fields inside the
device structure. This also adds a field so we can store the keyboard
mode of the tty, and restore it on close.

@fix

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

ecore-drm: Cleanup private header file

Summary: This moves the logind function prototypes to the private
header and thus removes the need for a separate logind header file.

@fix

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

ecore-drm: Cleanup launcher code

Summary: This moves the VT switch handling code to the tty file, and
sets the proper flags on the VT when opened.

@fix

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

ecore-drm: Fix various issues with logind code

Summary: This fixes a few issues with the logind code such as: Getting
the VT from systemd (if available), Properly setup the VT using proper
'open' flags, adding a 'restore' function to reset the tty properly,
and handle take/release session.

@fix

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

ecore-drm: Cleanup tty code and handle vt signal trapping

Summary: This cleans up the tty code a bit, and makes it the
responsibility of the tty code to handle VT switching signals

@fix

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

ecore-drm: Default device->session to NULL

Summary: This sets the default session to NULL when a device is
created. Also a formatting fix

@fix

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

ecore-drm: Store session name inside dev structure and don't
take/release session control inside dbus functions

Summary: These changes move the storage of session name to inside the
Ecore_Drm_Device structure and remove calls to take/release session
control to outside of the dbus code

@fix

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

ecore: Remove potentially useless 'else' branches

Summary: We actually don't need an else branch here if we don't have
USE_G_MAIN_LOOP

@fix

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

ecore: Remove blank line

Summary: No functional changes, just removing an empty line

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

release: Update NEWS and bump version for 1.13.0-beta2 release

embryo: prevent useless assignment.

Summary: In the if case 's1[]19 = 0' state looks meaningless.

Reviewers: raster, Hermet, cedric

Reviewed By: cedric

Subscribers: cedric

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

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

Evas masking: Fix masking of mapped sub-objects

Some complex examples of masking with mapped smart objects
would fail miserably, rendering the object without any mask,
and/or showing the mask itself somewhere in white color...