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

Eee image: fix macros to be surrounded with do while().

Macros should be surrounded with do {} while() statements,
otherwise there can be issues, like the macro not requiring a semicolon
after the statement or etc.

@fix

Evas GL common: Remove unused filter_prog

Evas GL common: Add shader ID as debug info in the pipe

Simplifies code readability a bit and helps for debugging (with GDB).

Evas masking: Fix evas map masking in GL

While this may look like a feature, it's definitely a fix.

ecore-drm: Re-enable setting K_OFF keyboard mode

Summary: Now that things are shaping up, we can re-enable the K_OFF
tty mode which will help with buffering input, etc.

@fix

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

ecore-wayland: Fix issue with Elementary DnD test not releasing mouse

Summary: When we do an input_ungrab, we should be sending a mouse up
event so that apps/elm/etc know that the mouse has been released. This
fixes an issue in the Elm Features DnD test report by Daniel Zaoui

@fix

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

ecore-wayland: Don't set move cursor unless we actually start a drag

Summary: We should not be setting a 'move' cursor unless we actually
make the call to start a drag

@fix

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

ecore-evas-drm: Init outputs before inputs so that input devices can
have a pointer to outputs

Summary: In order to limit pointer motion, we need to assign an Output
to each Input device. In order to accomplish that, we need to
initialize Outputs first

@fix

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

ecore-drm: Limit mouse movement to within the output size

Summary: This patch adds support to limit the mouse from moving
outside the output dimensions, and fixes coordinate transformation
calls to use the Output from the Input device

@fix

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

ecore-drm: Remove call to unused function

Summary: This just removes the call to the no longer used/needed
function to update the input device's view of the output.

@fix

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

ecore-drm: Remove unused function

Summary: Since we now have a reference to the actual output inside the
input device, we no longer need this function to update the Input
device's view of the output size.

@fix

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

ecore-drm: Add actual Outut to the Input device structure

Summary: This adds an actual Ecore_Drm_Output pointer to the
Ecore_Drm_Evdev structure so we can check the output size to limit
mouse corrdinates.

@fix

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

eina: fix eina_log with journald.

Summary:
Nothing was printed to journald before because sd_journal_send_with_location
expects file and line to be of the NAME=VALUE form.

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

Reviewed By: cedric

Subscribers: cedric

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

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

eina - mmap safety handler - fix to only handle sigbus's from io

@fix

this fixes the eina mmap safety handler to only  handle what comes
from actual i/o errors (not actual corrupt/hw memory faults or invalid
objects), and if it's not a sigbus eina mmap should handle then pass
it onto the previous handler before eina mmap took over.

ecore_cocoa: Depend on ecore_cocoa_window.h file for library build.

Without this missing dependency the file would not show up in the
tarball and the build on MAC OSX would fail.

Thanks a lot to Leif Middelschulte for testing and the report!

edje: Remove dead code

Summary: This removes the commented out dead code block from
9cdb8aaf9bec83a0e0879c04f66408f6326fa2b2

@fix

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

Eo: use int for _eo_init_count intsead of Eina_Bool

Eo base class: Fix compliation.

@inout also used to affect the type generated.
Compile check even the simplest changes.

Eo base: Remove @inout usage.

First step towards deprecation of @inout.

configure: Makre sure the tarballs are marked as alpha1

release: Update NEWS and bump version for 1.13.0 release

Eo tests: Fix bad free in eo_test_value

When running eo_test_suite we get an memory error: "double free or corruption".
That error arises because we try to free an Eina_Value value in eo_test_value
that doesn't need to freed.
I switched the eina_value_free to wina_value_flush, the proper way of
releasing it.

@fix

efl - edje_cc - fix default state checks in previous commit

this fixes 0cb33a46758bd1f66653e97d7ad027a9529b1279
@fix

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

this fixes T1926
@fix

Ecore: add EAPI to retrieve Ecore_Evas parent of a buffer.

Eo: add function name to the eo_do pre and post hooks.

This is useful to determine the Eolian class/function.

ecore_imf/scim: Fix logically dead code in scim module

CID 1261449

@fix

edje: Fix Coverity CID1261437

Summary: Coverity reports idential code for different branches here.
Not sure WHY that was done, but I commented out the existing if
statement for posterity.

@fix

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

evas: Fix unintentional integer overflow

Summary: This fixes CID1261436: unintentional integer overflow

@fix

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

ecore_evas/wayland_egl: Set alpha of ecore_evas object if parent alpha is set

Summary:
Resolved FIXME ecore_evas_wayland_egl_new_internal to set alpha if parent alpha is set

@fix

Signed-off-by: kabeer khan <kabeer.khan@samsung.com>
Reviewers: devilhorns

Reviewed By: devilhorns

Subscribers: cedric

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

examples/elocation: Remove unused config.h include

It is not needed here.

examples/evas-3d-blending: Remove unused global variable.

This got shadowed in one of the functions and is not used at all.

examples/evas-3d-shadows: Avoid shadowing a global variable.

examples/evas-3d-frustum: Remove unused variables.

elocation: Mark elocation as BETA API for the upcoming 1.13 release

It is not ready as a public API yet. It needs to run on GeoClue1
and GeoClue2 which will definitely bring API changes as some parts
are tied to mcuh to what GeoClue1 offers right now.

elua: convert some eolian binding API results to numbers

This might fix some bugs and is the right thing to do anyway.

ecore-evas-wayland: Port fix of min/max size calculations to the
configure event handler

Summary: This patch ports the fix for windows without a min/max size
being set over to the configure event handler (which was also not
taking into account the fact that Some windows may Not have a min/max
property set on them.

@fix

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

edje: update the edc reference

Add a list of global signals that edje provide.

tests/ecore_thread: Disable verbose logging with printf

If we are going to test for 10000000 messages we should not print out for each
of them to stdout. The log file for ecore_suite will catch them all. It ended
up being 199MB on my machine just filled with these debug statements.

I left it in in case someone needs it for debugging but we should not have it
on by default.

eo: Fix bad addressing in _eo_classes array

The header.id was masked before using it as index in the _eo_classes
array and was not unmasked when used.
It hasn't caused segfault (by sheer luck) but was wrong.

@fix

ecore_evas_x: Free XCounter values leak.

Summary:
Whenever creating a window via elm_win_add(), save netwm_sync_counter values to edata->netwm_sync_counter
by ecore_x_sync_counter_new().
However, never free the values when a window has been destroyed.
So this patch is for free the values in _ecore_evas_x_free().

@fix

Reviewers: cedric, raster, Hermet, devilhorns

Reviewed By: devilhorns

Subscribers: cedric, seoz

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

ecore_imf: Add null check condition in ecore_imf module

Summary: Fixed null pointer dereference issue in ecore_imf module

@fix

Signed-off-by: vivek <vivek.ellur@samsung.com>
Reviewers: devilhorns

Reviewed By: devilhorns

Subscribers: raster, cedric

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

eina: all Eina_Stringshare printf variant now return empty string as expected.

Previously Eina_Stringshare printf variant where returning NULL with ("%s", ""). This
is highly disturbing as you would have gotten "(null)", if you asked for ("s", NULL).
To clarify and make things logical, we now return "".

eina: fix memory leaks in eina_stringshare_printf/vprintf functions.

Summary: @Fix

Test Plan:
  CK_FORK=no valgrind --leak-check=full {UNIT_TEST_PROGRAM} "String Share"

Reviewers: cedric, seoz, Hermet, raster

Subscribers: cedric, seoz

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

evas:fix markup text length issue

Summary:
There was no conversion to the double quotation mark in the evas_textblock_text_utf8_to_markup function.
The price of the text coming out to API and text coming out to Textblock was different as a result.
As a result, Two text lengths came out differently.
So, I added the exceptional treatment part in the evas_textblock_text_utf8_to_markup function.

@fix

Subscribers: cedric

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

[Evas, Edje] Adding of new structures

Summary:
There will be several methods to set orientation in edc, so we have decided to make one big vector,
the main reason is that we use quaternion by default, but look_at, for example, is given as 6 coordinates.

Reviewers: Hermet, cedric, raster

Subscribers: cedric

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

ecore: fix compilation, missing return value in ecore_thread_wait

@fix ecore_thread_wait return EINA_TRUE if execution is over or thread is NULL

ecore-evas-wayland: Fix min/max size calculations

Summary: This fixes an issue for windows which do not set a min or max
size in the properties. This was discovered when running Enlightenment
in a Wayland-Only scenario, and trying to bring up the settings panel
which would cause an endless loop in calculating the proper window
size due to min/max not being set.

@fix

NB: Thanks to Mike for the help in tracing this ! :)

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

ecore-wayland: Simplify opaque and input region handling.

Summary: We can make opaque and input region handling simplier if we
just use one opaque & input region per window. Previous code would
always create a new region, set it to the surface, then destroy it.
This code works much nicer in that it hammers the protocol with less
region create/destroy calls.

@fix

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

ecore-evas-wayland: Fix formatting

Summary: no functional changes, just formatting

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

eio: forcefully wait on shutdown for all thread to stop.

This should avoid potential crash during shutdown while some Eio thread
were still running. We are still not blocking for more than 30s, so if
an IO is blocked on a dead device, you should be fine.