edje: Fix double free scenario caused by static pointer.

Summary:
The result of evas_object_textblock_cursor_content_get() API has to be cleaned
by outside.  _edje_entry_cursor_content_get() is calling free() inside of the
function for handle the result using static pointer. But, the caller of
_edje_entry_cursor_content_get() is already handling the result using free().
It can cause double free problem.

The bigger issue is in elementary. See elm_entry_cursor_content_get() API's
document. The document advice developers to free the result when it is done.

@fix

Test Plan: N/A

Reviewers: tasn, raster, woohyun

Subscribers: cedric

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

ecore_pipe: add write fdset and except fdset.

ecore_pipe_wait can be run not only select but also _ecore_glib_select__locked.
if write fdset is null, it make problem in FD_ISSET.

@fix

ecore_wayland: add wayland session recovery listener

Register and wait for the uuid event issued by the compositor once a new
surface is created.

ecore_wayland: bind and destroy session_recovery interface correctly.

This was missing from the initial session recovery support patches. Bind
the interface so we can actually work with it on the client side and destroy
it at the end.

@fix

Eo base: print an error when an object with a parent is destructed.

This should not happen. Objects with parents must have their parents
unset before they reach refcount == 0. That's because the parent is the
one holding the refcount. This means that if we get to the destructor
(object is deleted) while a parent is still set, we have an error
scenario.

Revert "Revert "Eo base: Change parent_set to be an assignment of ref.""

Had to revert it until I pushed the changes to elm, which I had issues
with. Now the patch can safely go back in.

This reverts commit 37abea3831f62ac612b377bb48ce6c90e076dd25.

Revert "Eo base: Change parent_set to be an assignment of ref."

Damn, this breaks some things. Reverting until fixed.

This reverts commit 9c78ee0bf4125c095e85f7fcf9921586cda64a52.

Eo base: Change parent_set to be an assignment of ref.

After this change, parent_set assigns a ref, so for example:
  obj = eo_add(CLASS, parent); /* Ref is 1 */
  eo_do(obj, eo_parent_set(parent2)); /* Ref is 1 */
  eo_ref(obj); /* Ref is 2 */
  eo_do(obj, eo_parent_set(NULL)); /* Ref is 1, giving the ref to NULL */
  eo_do(obj, eo_parent_set(parent)); /* Ref is 1 */

This is following a discussion on the ML about commit
8689d54471aafdd7a5b5a27ce116bf2ab68c1042.

@feature

edje_cc_out: Fix edje_cc compile error with script on Windows.

On Windows, PATH_MAX is 260 and PATH_MAX is used as string buffer
size in edje_cc compile. This causes edje_cc compile error when the edc
file contains "script" keyword and the length of file paths is
relatively long.
To resolve this problem, change the string buffer size in edje_cc
compile.

@fix

Revert "Revert "ecore_win: fix the incorrect mouse cursor position.""

This reverts commit 3ce8860dab29ba6d0849ed1ecbbc72bbe0e11773.

Apply only to mouse wheel case. Button press/release wans't problem actually.
If I correct, this is caused because of different nature of window systems.
Anyway our Ecore_Event_Mouse values should keep consistency among the various systems.

Revert "ecore_win: fix the incorrect mouse cursor position."

This reverts commit 1f1542f82404fa2262526cc3eed7b2edc622c5ce.

coordinate has been corrupted. need to investigate more.

build: replace libsystemd-{daemon, journal} with libsystemd

Summary:
since all the libs got merged into libsystemd in 209, we can just check
for libsystemd

Reviewers: cedric

Subscribers: stefan_schmidt, morlenxus

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

ecore_win: fix the incorrect mouse cursor position.

Ecore_Event_Mouse_* x, y values are relative to the current window position
as well as the root x, y, values are relative to the root window.

previously, x,y is started from the root window and root x, y values are invalid.

fix them

@fix

small typo in docs, mounse is mouse

eldbus: add since tag for now public eldbus_message_signal_new API

Thanks go to vtorri for bringing this to my attention.

evas_3d: fix defines

Summary:
evas_3d: removed unnessecary defines

Evas_Real was allready defined.

The typedefs of the Eo types can be avoided by fixing the include order

Reviewers: cedric, stefan_schmidt, tasn

Reviewed By: stefan_schmidt, tasn

Subscribers: stefan_schmidt, cedric

Projects: #efl

Maniphest Tasks: T2658

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

evas/filter: Remove unused variable

lib/evas/filters/evas_filter_parser.c:861:14: warning: unused variable 'ok' [-Wunused-variable]

eldbus: remove now public eldbus_message_signal_new declaration from private.h

The declaration is now on eldbus_message as EAPI so we can safely remove it here.

eldbus: add eldbus_message_signal_new to public header

Test Plan: elementary tests

Reviewers: cedric

Subscribers: seoz, cedric

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

efl - jp2k loader - guard against openjpeg bug tha causes an abort

so... if you load a non-jp2k file using openjpeg, you can get an abort
deep inside the openjpeg library that we can't do anything about. we
set all error handlers but literally the openjpeg code has ab assert
there that causes this bug. it shouldn't and newer opengjpeg libs have
it removed, but 1.5.2 has it and this causes an untrappable crash.
this is simply bad behavior in openjpeg not allowing it to be used
safely to loade image files. the relevant backtrace:

    w=w@entry=0x7fffffffb548, h=h@entry=0x7fffffffb54c,
    alpha=alpha@entry=0x7fffffffb556 "", map=map@entry=0x7fff29ac2000,
    length=<optimized out>, error=error@entry=0x7fffffffb5bc,
    opts=<optimized out>)
    at modules/evas/image_loaders/jp2k/evas_image_load_jp2k.c:111

the relevant code in openjpeg:

int cio_numbytesleft(opj_cio_t *cio) {
  assert((cio->end - cio->bp) >= 0);
        return cio->end - cio->bp;
}

so that assert is triggered. and nothing can be done about it which is
pretty poor.

so an upgrade of openjpeg should fix this as in newer versions have
dropped the assert line in that function, but until poeople have that from
their distro, this adds magic number checks for file headers that avoids
using openjpeg if it's not "apparently" a jp2k file. this does not
stop a corrupt file or a maliciously designed file still causing this
problem, but it does just result in an abort() and isnn't seemingly an
overflow isse that can be exploted, so if you still suffer, find a way to
upgrade openjpeg to 2.x. until then... this reduces inadvertent damage.

@fix

eina: fix coverity issue CID 1317154.

ethum: fix compilation on Windows due to missing visibility setting.

evil: add missing since

Eo base: clean up parent removal in destructor.

This cleans up the changes in 8689d54471aafdd7a5b5a27ce116bf2ab68c1042.
This commit reduces code duplication, and fixes broken handling of
overridden parent_set().

ecore_con: fix test to properly send the right amount of data (including the '\0').

eina: use new eina_file_path_join API

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

eina: add API to join pahs, as well as path separators

eina_str_join() is used a lot to contatenate paths, but the
separator should be '\' on Windows. So add 2 API and 2 defines for
more cross platform code

@feature

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

ector: don't forget to reset cairo matrix to identity.

efl: fix Efl.Gfx.Shape.Dup to use const parameter.

efl: prevent segv while interpolating empty shape.

evas: improve simple VG example to interpolate a full VG tree.

eina: add test for decomposition and composition of matrix.

eina: add eina_matrix4_quaternion_get and eina_quaternion_matrix4_get.

Implementation taken from pseudo code at :
http://www.w3.org/TR/css3-transforms/#decomposing-a-3d-matrix

eina: add eina_matrix4_multiply and eina_matrix4_identity function.

eina: add eina_matrix4_transpose.

eina: add tests for eina_matrix4_inverse.

eina: add eina_matrix4_inverse.

eina: test eina_matrix4_normalized.

eina: add eina_matrix4_normalized.

eina: add eina_matrix4_determinant.

eina: our 3x3 matrix are 2D matrix, not 3D rotation matrix.

evas: add logic to duplicate recursively an Efl_VG_Node tree.

evas: allow parent during dup to be an Evas_Object_VG.

evas: add support for interpolating Efl_VG_Node tree.

evas: add an unique name for each Efl.VG.Base object and make it possible to find it from parent.

Evil: fix compilation when MSVC is not available.

localtime_s is not defined in msvcrt.dll but rather is defined in
Microsoft libc when Visual Studio or other stuff is installed.

Issue introduced in:024812c1a76286991f292c3191936778ec219ff8

Fixes T2681

@fix

Evas filters: Fix build for Jenkins gcc_x32

Disable bit32 library if it's not available.
We should probably either ship it or disable it altogether
for the filters. Hmm.

Add experimental implementation of custom animator ticks

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

evas: properly fix unref of ector renderer.

Async rendering doesn't have a main loop cleanup function. The only one
being called is in the rendering thread. I wrongly assumed in my previous
patch that render_post on an object was called after the async render was
done which is obviously not the case as pointed by Subhransu. This patch
now wait for the async rendering to be done.

eo - use ren not xref for children to not waste memory

optimization

xrefs keep lists of objects references. children are already in a list.
why keep both? lots of extra memory used for no value when debug is on
(pretty much most of the time).

evas: fix win32 build warning.

include evas_private.h so that
internal ector eo apis have affected with symbol visibilty declarations.

ector: freetype -> default software

ecore_imf_example: remove unnecessary code to set client window

evil: fix warnings.

correct data size for 32/64 compatibility.

evil: fix incorrect function call.

_localtime64_s() requires _time64_t as one argument but here we passes time_s.
Proper api is localtime_s().

eo - destruction - ensure child is removed from parent child list

this follows on from cbc1a217bfc8b5c6dd94f0448f19245c43eb05e0 as this
code was correct, but was then causing bugs due to children staying in
their parent lists. this should never have happened and this is really
bad. this fixes this and ensures children on destruction are gone from
their parent lists.

@fix

Evas filters: Fix char buffer size

Thanks Coverity.
Fixes CID 1316684

Edje_Entry: check selection before the cursor position change.

Summary:
change position of the main textblock cursor
in depends of the selection is present. Change cursor position to
the start or end of selection only when selection is present.
@fix

Test Plan:
Press "Up" and "Down" arrow keys on selected and normal text.
For selected text: the entry cursor should be placed one line
   above or below selection block (in depends from pressed button).
For normal text: the entry cursor should be placed one line
   above or below of the current cursor position.

Reviewers: tasn, Hermet, herdsman

Subscribers: cedric

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

evas: be more pedantic in case of error in Evas_VG_Node.

eio: limit race condition during shutdown.

Evas language: fix script run code

For script runs that start with an UNKNOWN character, the whole
run was mistakenly identified as script type UNKNOWN.

Also, refactored code a bit for readability.

Fixes T2670.
@fix

evas: no need to get the data if it is not the right type of object.

ector: fix naming and forcing this inline function to be static.

evas: fix ref/unref of ector renderer to always happen in the main loop.

ector: remove depricated file ector_blend_private.h

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

ector: add SSE2 support for gradient filling in software backend.

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

ector: add NEON support for composition function in software backend.

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

ector: add sse2 support for composition function in software backend.

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

ector: refactored software drawing backend to use composition function.

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

ector: add alpha field to gradient data.

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

ector: add dash stroking feature in software backend.

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

ector: refactored shape generation in software backend.

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

ector: remove wrong implementation of singleton in software_surface class.

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

ector: fix memory leak in software backend when using shape.

Mixin destructor are not called, so we need to explicitely call it.

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

ector: add bounding box info in RLE data for software backend.

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

ector: now software rasterizer allocates memory on stack to work in multi threading env.

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

evas: delete the ector_renderer when evas_vg_node gets deleted.

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

Evas GL: Debug APIs should also restore context

As discussed briefly with @spacegrapher

Evas GL: Also generate debug functions for GLES 2

Evas GL: Add EVGL_FUNC_BEGIN() to GLES 3 functions as well

This call ensures that the context is current (context restore).

Evas GL: Simplify previous commit

Also generate the debug functions for GLES 3

Evas GL: Restore current context to evas gl context when the backend has taken over

Summary:
When Evas GL apis are called outside of on pixels callback,
evas gl backend context may have been made current, and Evas GL will
render into a wrong context.
So here we provide context restore mechanism of keeping track of
currently bound context and calling make current when needed.
@feature

Test Plan: Run Evas GL test cases

Reviewers: jpeg, cedric

Subscribers: mythri, mer.kim, wonsik, cedric

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

Evas filters: Fix proxy buffer size in Lua

In order to do that, avoid creating multiple Buffer instances
when pointing to the same proxy source. This fixes buffer.width
and buffer.height in Lua.

Evas filters: Fix massive memleak with async sw render

Oooops, the flag stolen meant that we don't hold any reference on
this buffer anymore, which meant we should not increase the refcount
here!

@fix

Evas: Add debug env var EVAS_IMAGE_NO_MMAP

Looking for image buffer memory leaks with Valgrind is impossible
when all images are mmaped. This is intended as a DEBUG environment
variable only.

Evas filters: Fix for Lua 5.3 (with --enable-lua-old)

Tested with LuaJIT, Lua 5.2 and Lua 5.3.

@fix

eo - fix bug vector when unparenting of an eo obj may affect others

it is possible that a destructor/parent_set override or function could
go modifying the children list of objects in a parent, this the
EINA_LIST_FREE may actually miss objects in the process since within
the "free" func the list may have been altered etc.

@fix

efl debug infra - add more event types for ability to check extended inf

efl debug - add the ability to begin/end states not just call ranges

eina: change default log level at which we display backtrace.

We mostly use in our code base ERR in conjunction with system related error. This
doesn't require any information from who did call that function to get debugged and
is creating way to much noise with little value.

evas_image: finish doc conversion

evas_image: convert more docs

evas_object,evas_image: partial doc conversion

edje: set file permission back to 644

ecore-wl: do not nul terminate drops

the application can do this based on mime types. we should not be
nul terminating incremental drop data

@fix

emotion: Fix typos in doxygen

Summary: This patch fixes some typos in the documentation of Emotion

@fix

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

ecore-x: add property member to Ecore_X_Event_Selection_Notify

@feature

evas render 2 - tiled rouding of updates to keep number regions down

tile to 16x16 (we can cahnge) to keep number of update rects
down/minimal with render2

evas - handle case where layer->evas is null - don't crash

there seems to be a corner case where obj->layer->evas is null for an
object. i think during shutdown of a canvas.

@fix

efreet - fix efreetd kill and start with clean cache case blank icons

if you kill efreetd ANd delete all the caches, the restart of efreetd
will lose all icons until an app re-registeres icon extensions and it
can scan all icons .. and then app has to actually get the right
upodate events and do the update properly when this happens. this
fixes that scenario

@fix