evas_render: moar debug (REND_DBG)

Evas masking: Fix potential invalid access to mask image

After clip_image_get, the old mask may be replaced by a new one,
and unref'ed, but it is later on set back as the context mask image.
Maybe it's possible that there was 0 reference and the image
got freed in between.

No idea how to test this.

@fix

Evas masking: Make sure to check alloc before freeing old image

Okay, I'm being paranoid and this can't possibly happen (calloc
fail? yeaaaah) but this makes sure we don't return NULL after
freeing the original image.

Evas: Remove useless include

config.h is already included in the private headers, and should be
included first anyways.

eolian: remove support for old style docs completely

eolian: disable parsing of old style docs

evas canvas: finish doc conversion

eio: unify deletion of eina_file_ls's iterator.

Due to a bug with eio_file_stat_ls(), a previous patch
(73a23401ef007cd80297df6546484f3e8a065d2c) was written.

This patch change the way we delete returned eina iterator
from eina_file_ls in _eio_file_eina_ls_heavy() from a separate
thread, into eio_async_end() from the main thread.

To clarify the code, and avoid a useless call to
eina_iterator_free(), this patch also moves the deletion
of the same eina iterator from _eio_file_heavy() into
eio_async_end() as well.

ecore_psl1ght: Removed unused variable.

Summary: Signed-off-by: Srivardhan Hebbar <sri.hebbar@samsung.com>

Reviewers: cedric, stefan_schmidt

Subscribers: cedric

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

ecore-con-url- split out the curl dleopener and resolver and headers

this splits out out "manual" dlopen (eina_module_load) of curl into
its own .c file and special header out of ecore_con_url.c to tidy up
that code a bit and isolate our curl magic loading/handling

evas canvas: some doc conversions

evas_canvas: prepare for doc conversion + partial conversion

efl_vg_base: fix wrong @since (was pushed after 1.15)

efl_vg_base: convert docs

evas object: finish doc conversion

evas object: more doc conversion

evas object: partial doc conversion

evas - fix texture context bind reset in gl common using wrong texture

evas gl common was simply resetting to the wrong texture id in the gl
context struct - it was using pipe[0] not state.current. why i don't
know. i know i wrote the pipe[0] code many years ago - really don';t
know. it may have been a transitional piece of code that just happened
t6o work 99% of the time that never got fixe when i added pipes.

@fix

evas:textgrid: correctly check malloc failures

evas:textgrid: rename text_props to text_props_index

evas clip: add warning msg for wrong scenario.

[Evas] Counting of tangent space (fix)

Summary: It is counting here, so we can avoid using glsl functions dFdx, dFdy.

Reviewers: cedric, raster, Hermet

Subscribers: Oleksander, cedric, artem.popov

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

edje: remove un-used edje_container code

The code have been ifdef'ed out for a along time. I checked back until
the merge of our libs before the 1.8 release. Maybe even before that.

If you have a use case for this code fell free to revert but also enable
the code and make sure it works when doing so.

eina tests: disabling eina_test_matrix_quaternion which fails for over a week

This fails on nightly_efl_gcc_x86 (32bit) for over a week now blocking all other
nightly builds and tests. Inside the test case there is already something
disabled which meantions problems with rounding. MAybe the same problem here.

Reported as T2701 to keep track of it an enable again once fixed.

ref T2701

evas - gl common - make yuv gl textures double-bufffered like yuy2

this pulls yuv in line with yuy2 textures and dobule buffers them.
this is a workaround some driver bugs where either the driver doesnt
block and wait for the gpu to finish with a texture when updating OR
it does wait, and when it does it blocks and spins using cpu.

ecore win32: increase command buffer size.

PATH_MAX is not enough for command.
Even in this case, the buffer is not being used only for path.

ecore win32: modifiy to better meaningful exception.

eina: cpu - fix a typo in eina_cpu_count API doc

emile jpeg: more descriptive comment.

emile: fix can not parsing the EXIF info of the jpeg file which has IFD offset

Summary:
IFD offset of jpeg is not fixed.
But emile support only 0x8 on now

Reviewers: jypark, cedric, Hermet

Reviewed By: Hermet

Subscribers: cedric

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

evas_canvas3d: Recalculation coordinates for pick object from scene

Summary:
Change recalculation coordinate for pick to depend on size of viewport, not from
scene data.
Due to not correct working pick in case resize view window.
@fix

Reviewers: cedric, Hermet

Reviewed By: Hermet

Subscribers: cedric

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

Eina rectangle: Check for valid rectangle before taking intersection

Summary:
Check for valid rectangle

If any dst or src rectangle has zero width or height,
intersection should not return true.

@fix

Test Plan: Added test cases

Reviewers: cedric, herdsman, Hermet

Reviewed By: Hermet

Subscribers: shilpasingh, cedric

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

eet - improve eet file storage when encoding by aligning data

this forces data when written to be aligned to 8 bytes. this makes
mapping keys directly far better as from now on you can assume all
data is aligned to 8 bytes allowing you, if you want, to use eet files
as caches for raw binary data you mmap directly and just use (not
portable though).

@feat

Evas textblock: fix case of own_closer in style_set

Looks like it was assumed that an fnode->orig_format always ends with a
'/' character if the fnode is an own_closer.
The problem is that a paragraph separator ("ps" and "br" - the latter in
legacy newline mode) is also an own_closer, but might not have '/' at the
end, so decrementing the length is wrong.

This fixes T2654. The example markup had "br" read as "b", which led to
a mismatch with the "font_weight=Bold" tag. Coincidentally, "ps" was not
affected as there was no matching "p" in the style.

@fix

evas/gl_common: add new header files to build to make them available for dist

ecore_wayland: only add listener if recovery interface was binded correctly

In the case the bidn fails we could crash here by accessing it.

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.