Revert "ecore_x: remove dead code."

This reverts commit 7303fcee4de5364baa8e9362ffcaa4b00d00fd34.

acutally the code was intended for compile warning.

ecore_x: remove dead code.

remove ridiculous code.

ecore_x: free the allocated memory after used.

XFixesInvertRegion doesn't free the xbound argument inside.

@fix

ecore_con: remove dead code.

there won't reach with !wp because if wp is null,
the iteration will jump to next iteration.

efl - edje_cc - fix double close of embryo script fd's to tmp files

@fix

thanks to 김낙경 <nakkyong.kim@samsung.com> for pointing this out.
indeed fclose() closes the fd adopted by fdopen(). it doesn't dup it.
so we double-closed. it seems this never caused an issue until now.
this should fix it.

evas - fix whitespace in code

eina: prevent compile warning.

Inline function sometimes causes app compile warnings
which applies strict compile option.

fix typos in API doc

Edje Example: Fix wrong usage of evas_object_color_set API

Summary:
Evas only handles pre multiplied colors. So use r,g,b values lesser than alpha to prevent error message.

Signed-off-by: Yeshwanth Reddivari <r.yeshwanth@samsung.com>
Reviewers: sachin.dev, singh.amitesh

Reviewed By: singh.amitesh

Subscribers: cedric

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

edje_edit: delete gradient funcs prototope

This API was defined in the Edje_Edit.h but haven't implementation.
Given the fact that gradient is not supported and print 'SPANK'
error on load have sense to delete them.

Evas GL: Free safe extensions hash when engine shuts down

This fixes the memory leak.
Thanks @rajeev.jnnce for spotting this!

eolian/generator: support for more paragraph tags

evas_render: Fix invalid clip

Test case: elementary_test -to "Evas Map 3D"
The cube was clipped to its top-left corner.

What's really weird is that this code patch is for non-mapped
objects.

evas - sw async render - move pixel upload/swap to a thread

@feature

this should unload mainloop some more and have pixel upload now in a
thread - same as rendering. this eeems to work where i see it so let's
put this in and see with further testing.

eolian/generator: generation of doxygen @note and @warning tags

@feature

Evas GL shaders: Add mask 21, 12, 22 downsampling modes

This is a huge patch, but keep in mind that 4326 line changes
were automatically generated from the template GLSL files.

This also adds some minor refactors here and there to accomodate
for the new shaders, but nothing crazy.

"GL fast-path" downsampled masks should now look pretty good.

evas render2 - more work on fleshing it out a bit

eina_thread: non-linux fixes following latest changes

ecore - let the ecore_con_eet_*.c examples build.

When using the --enable-always-build-examples configure option, these two
examples need to be linked to eet.

@fix

map: don't update surface unencessarily.

smart object don't need to update map surface always
check the state more elaborately so minus one more surface drawing!

efl - set thread names for internally created threads for debugging

@feature

eina - thread naming - when naming thread, cut off at 15 chars

automatically cut off at 15 chars (and add 0 byte) to ensure we get a
name, but just truncate it if it is too long.

eina thread - add wrapper for setting thread name and fix affinity

added support to set thread names for debugging. this wraps the gnu
extension to set a thread name of up to 15 chars (16 inc nul byte).
eina_thread_name_set() is the new api.
@feature

in the process i found the autofoo wasnt enabling cpu affinity support
at all in reality, so i had to fix that at the same time.

evas: fix memory leak in evas_object_image_render_pre.

Summary:
Eina_Rectangles are removed from pixel_updates list without eina_rectangle_free during render pre stage.

@fix

Test Plan: check memory leak.

Reviewers: cedric, jypark

Subscribers: jypark, cedric

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

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

evas: CID 1323091

Summary:
@fix CID 1323091 for Coverity

Reviewers: cedric, Hermet, raster

Subscribers: cedric, artem.popov

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

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

ecore x vsync - add in exynos checks and move kernel v checks inside

this adds asmung exynos driver as a whiteliested drm vsync driver.
this means 3.0 needs to work so move kernel ver checks inot each driver.

@feature

Edje_cc: check value for text.source attribute.

Summary:
Add additional check for description.text.source
and description.text.text_source attributes in TEXT part.
For cases when as source uses non TEXT/TEXTBLOCK part
serialization process should be stopped. This will cause
segmentation fault on runtime. (see edje_text.c line 251)

Reviewers: Hermet, raster, cedric, jpeg

Subscribers: cedric

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

Signed-off-by: Jean-Philippe Andre <jp.andre@samsung.com>

evas/gl_common: Do not set minus value to unsigned int

Summary:
cur_tex* are unsigned int, so minus value should not be set.
Also, add cur_texa, cur_texm here.

Test Plan: Local tests

Reviewers: jpeg

Subscribers: mer.kim, wonsik, cedric

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

eet_example: Fixing memory leak.

Summary:
calloc is done 2times, so there is a memory leak. So fixing that.

Signed-off-by: Srivardhan Hebbar <sri.hebbar@samsung.com>
Reviewers: cedric, stefan_schmidt

Subscribers: cedric

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

ecore_evas_convert: make sure we add all needed flags and deps for the build

We got a report where ecore_evas_convert linked against some old libs
installed in the system instead of the recent ones in-tree. While I was not
able to reproduce this the flags and deps for ecore_evas_convert have been
missing in any case and could be the culprit.

Ref T2716

ecore_evas_extn: Fix server rendering after restart

Summary:
The server can render, only after the server get the OP_SHOW from the client.
However, if the server relaunches while client is running, the server cannot get the OP_SHOW.
In this case, the client should send the OP_SHOW, when the server is added.

Test Plan: Relaunch a server especially the indicator, while client is running.

Reviewers: raster, cedric, Hermet, woohyun, jaehwan, jypark, jpeg

Subscribers: cedric, seoz

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

Signed-off-by: Jean-Philippe Andre <jp.andre@samsung.com>

evas examples: avoid shadowing a global variable in a function

evas: gl_generic: fix leak if we return dst and not newdst

In case we still retrun dst we should make sure we free up the newly
allocated newdst.

CID: 1323090

ecore_example: Fixing NULL deference.

Summary:
pd was dereferenced and then checked for NULL. If it was NULL, the program would crash much before. So fixed that.

Signed-off-by: Srivardhan Hebbar <sri.hebbar@samsung.com>
Reviewers: cedric, stefan_schmidt

Subscribers: cedric

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

ecore_example: Fixing memory leak.

Summary:
Fixed a memory leak in this example.

Signed-off-by: Srivardhan Hebbar <sri.hebbar@samsung.com>
Reviewers: cedric, stefan_schmidt

Subscribers: cedric

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

evil: remove executable bit from memcpy assembler file.

Hopefully the last offender we have in tree with this.

ecore_example: Fixing NULL dereference.

Summary:
cd was dereferenced and then was checked for NULL. If cd is NULL, it would crash before itself. So checking for NULL before dereferencing cd.

Signed-off-by: Srivardhan Hebbar <sri.hebbar@samsung.com>
Reviewers: cedric, stefan_schmidt

Subscribers: cedric

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

ecore_example: Cleanup of ecore_compose_get_example.c

Summary:
Relating to D3010. This patch is code cleanup. Have done dos2unix. After this will update D3010 bug.

Signed-off-by: Srivardhan Hebbar <sri.hebbar@samsung.com>
Reviewers: cedric, stefan_schmidt

Subscribers: cedric

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

build: add efl as internal dep for eio

Thanks to Adrien Nader for the report and fix.

update mailmap

edje_cc: remove executable bit from source file

ecore: win32: remove executable bit from source file

ector: software: remove executable bit from source files

This really shows what parts are developed under windows. While it does not
matter where you develop please make sure that you do not introduce CRLF
endings and normal source files as executables.

ector: software: convert dos to unix line endings

ecore: cocoa: convert dos to unix line endings

evas: tests: convert man_mesh.obj from dos to unix line endings

Remove TODO-eo2 file.

This should never have gotten in.

Evas: Move REND_DBG to evas_render.c

Unlike what the comment claimed, it is not being used anywhere outside
this file.

evas: Use malloc instead of calloc in context_dup

Since we call memcpy right after, no need for calloc

evas_render: Fix some clipping issues inside proxy

Inside a proxy, clipping information might be wrong since the
source object may be at a different position than within
the proxy. If source_clip is not set, then we need to discard
all clips that are outside the proxy context.

So we just propagate the clip information inside the current
draw context, and even recurse from clipper to clipper to
find the final state of clipping.

Map and proxies and others (who said masks?) should definitely
rely more on the same model.

This code is not a mess. At all. You gotta love evas_render.

evas_render: print out context clip with REND_DBG

evas_render: simplify masking and clipping in general

Use context_dup to inherit from previous contexts in a clean
manner. This removes the need for restoring the previous
clip information.

Plus, this commit removes lines of code so it must be good, right?

evas_render: Cosmetic changes in evas_render_mapped

Use ENFN, ENDT macros to simplify code.
I believe it is more concise and readable like this.

evas: Add internal context_dup function

This will simplify some code related to clipping and masking.

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.