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...

Evas masking: Fix rendering of masked objects with maps

evas: fix signed/unsigned warning.

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

evas: fix to load BMP file compressed with RLE.

Summary:
If the file size of RLE compressed image is bigger than original image,
BMP loader doesn't work as well.
@fix

Reviewers: Hermet, cedric

Subscribers: cedric

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

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

evas: add new test for bug in BMP loader.

evas: fix tests to actually test image !

elocation: fix typos.

Efreet: do not get confused by path ending with / in XDG_* vars

This @fix the parsing of dirs from the xdg env vars. Now always remove
the leading / char from the paths.

This was causing lots of trouble on my system, where XDG_DATA_DIRS is:
/usr/local/share/enlightenment:/usr/local/share:/usr/local/share/:/usr/share/

At first /usr/local/share was added 2 times in the list, one with the / and one
witout, causing a double lookup for each file.

Secondly the icon cache was totally unusable as the cached paths ended up
as: /usr/share//icons/Mint-X/places/32/folder.svg. The double / in there
was making the cache lookup to fail and anways return the biggest icon
available. Causing a big system slowdown whe searching for icons.

As a bonus the function now use eina_str_split instead of the custom splitting
code that require a bad special handling for the last item.

autotools: remove some Windows CE bits.

We actually do not support Windows CE and don't plan to. Let's remove that
from our configure.

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

ecore-wayland: Don't send a mouse_up on grab release unless we
actually have a grabbed button

Summary: If we don't have an actual grabbed mouse button, then don't
send a mouse up event. Caught this while adding e_grabinput support to
Enlightenment.

@fix

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

Eo add: beef up error reporting.

In some cases object ceration would fail without an error,
this is bad and should not happen.

Thanks to cedric for reporting.

evas/map: make an internal function to static.

no need this function set external.

Evas masking: And another case of bad mask rendering

A masked object just can't be considered as opaque. As such,
it can't generate an obscure area. This fixes some super
rare rendering bug.

Evas masking: Fix blending of some border images

ee engines should not crash when re-setting the same title/name_class

@fix

Evas masking: Simplify ref/unref code

Those masks are always render surfaces so we should
treat them as such. Plus, there are proper functions
to ref/unref GL images.

Evas masking: Aaaand fix another series of memleaks

Don't inc-ref more than we can dec-ref.

Evas masking: Fix another memory leak

When the mask is hidden, free its surface.
This will fix leaks in, say, a genlist full of masked
objects.

Evas masking: Fix potential GPU memory leak

This should free the mask surface properly, like we do
in software.

Evas masking: Fix potential crash

This adjusts the clip to match the mask image data.
To be fair, I'm not sure in which situation those two
don't match well. This patch fixes a rare crash.

warning--

evas: do not compute things twice.

Fix CID 1265599, 1265597, 1265598 and 1265596.

eet: properly handle failure and success case by not leaking memory.

CID 1265601.

Evas masking: Fix major memory leak

The memory usage graph was going up and to the right!
I was told this is always a good thing!

... maybe not this time :)

Hopefully I didn't forget a case. An intense session of
genlist scrolling with masks all over the place and masks
of masks didn't show any glitch, crash or memory leak.

Evas masking: Try to reduce memory footprint a little

Move some mask object pointers around to spare a few
bytes of memory.

Fixes T2025.

ecore-wl needs to also set xdg props again on show

fix T1819

ecore: lower priority of some intrusive logs.

Summary:
After fixed logging to journald there is lots of prints like
Jan 15 16:04:21 localhost system_server[1017]: enter idling=1 fds=1 events=0 timers=0 (next=17.14) idlers=0

Jan 15 16:04:21 localhost system_server[1017]: leave, timeout = 0

Change-Id: Id26e37a55ddaad36094308ac7fd09767cf659c04
Signed-off-by: Vasyl Vavrychuk <vasyl.vavrychuk@globallogic.com>
Reviewers: cedric

Reviewed By: cedric

Subscribers: cedric

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

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

evas: reduce potential memory hole in Evas_Object_Protected_State.

The main difference between 1.12 and 1.13 memory foot print is actually
related to this two pointer to mask. I am wondering if there is not an
issue here also has we do have a duplicated pointer. We have prev_mask
and mask in both cur and prev state of an Evas_Object, but only mask
and prev_mask from the cur state seems to be accessed.

If we can remove two pointers from those state, we should have a decent
win in expedite benchmark. Hopefully 300KB to win there (Close to half
the additional cost in memory).

evas: reorder structure to avoid structural hole.

edje: edje_pick - fix double free eina hash iterator.

Summary:
move freeing iterator into block where it was created.
@fix

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

Reviewed By: cedric

Subscribers: cedric

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

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

Revert "ecore-drm: Don't init tty unless we are Not using systemd"

This reverts commit 28e4901d041e087f505c19bc9733ef22d31addf0.

This broke vt switching in drm so revert (for now) until we get the
systemd vt and setup handlers

ecore-drm: Don't init tty unless we are Not using systemd

Summary: It turns out we don't need to handle open & setup of the tty
unless we are Not using systemd. This fixes an issue during
enlightenment restart where we are unable to reopen the tty.

@fix

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

ecore-drm: Reset output cursor and crtc mode on shutdown

Summary: When we shutdown outputs, we should reset the hardware cursor
state and the saved crtc mode in order to reset things back to normal.

@fix

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

ecore-drm: Restore tty keyboard mode on restore of tty

Summary: When we are shutting down a tty, restore the old keyboard
mode that we retain from tty_setup.

@fix

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

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

Revert "eet: fix memory leak in eet_cipher"

This reverts commit d25d56758c735cdf5760278f8a97e9c6b8ec7355.

That patch was leading to double. Thanks ami for finding it.

examples/evas: Add evas_3d_shadows binary to .gitignore

ecore-wayland: Don't crash if we have no input->cursor_name

Summary: If we are using a touchscreen, there will be no pointer and
thus no cursor_name to use. Trap for that case by checking if
input->cursor_name is valid.

@fix

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

examples/eina_inarray: Remove unused variables.

Evas masking: Fix precise_is_inside for masked smart objects

The flag should be set on the mask itself.
Checking for (x,y) being inside the mask can be an expensive operation,
so further optimization will be required.

ecore-evas-x should check itself before it wrecks itself when re-setting the same name/class

edje: Edje_Edit - fix memory leak in edje_edit_image_usage_list_get

Summary:
freeing iterator when necessary

@fix

Reviewers: Hermet, seoz, cedric

Reviewed By: cedric

Subscribers: cedric, reutskiy.v.v

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

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

eina mmap safety handler - fix on systems without some sibgus types

this fixes 615456aae8c309d0e82c6b549d17a7109554a4f6 to compile on
systems without MCERR sigbus types.

eet - one place endianess check wasn't explicit - cleaner code if it is

this MAY be a bugfix, but i can't see how after checking the code, but
to keep it conistent and less likely to grow a bug... do this.

eet - boyscouting - clean up whitespaces

eet - image decode - fix robustness of image decode from eet file

there are possible security implications by not checking values of
size fields to see if they are within the data range AND are not 0 or
negative. so do this.

@fix