evas: remove unused variable in evas_tangent_space_get.

Summary:
Firstly it was the main point of in CID1323089 from Coverity, but this variable is unused. So I've deleted it.

CID1323089

Reviewers: raster, Hermet, cedric

Reviewed By: cedric

Subscribers: cedric

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

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

emotion: add libvlc module

Summary:
Everything is implemented except visualization, mouse events and some
EMOTION_EVENT. Video can output RGBA, YUY2, YV12 or I420.

About the video sink: this emotion module use libvlc callbacks:
libvlc_video_set_format_callbacks and libvlc_video_set_callbacks. It may be
temporary. Indeed, an other solution is to add an Evas module inside vlc. But
this vlc module would need to link with emotion in order to use
_emotion_frame_new and _emotion_frame_resize private functions. I didn't
succeed to output a frame without these private functions: see
https://github.com/tguillem/vlc/commit/15daff4d3fdcd3ca7b485c1f8fabb8c1e1c4138c

List of /* FIXME */:

 - Visualization not implemented since there is no API (for now) in libvlc.

 - Mouse events not implemented since there is no API (for now) in libvlc.

 - Some EMOTION_EVENT are not handled.

 - SIGSEGV in evas_gl_common_texture_nv12_update with
   EVAS_COLORSPACE_YCBCR420NV12601_PL colorspace.

Subscribers: cedric

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

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

edje: remove unused variable and simplify function prototype.

edje_cache: remove duplicated call of _edje_file_coll_open

Summary:
_edje_file_coll_open will be executed after _edje_file_open is finished.
This duplicatated call will increase the reference counter and give failure
of _edje_cache_coll_unref.

@fix

Reviewers: cedric, raster, Hermet

Subscribers: cedric

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

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

ector: make it possible to have an unique id per Ector_Renderer.

This is going to be used by an Eina_Hash to deduplicate the number of Ector_Renderer.

ector: fix cairo backend to properly multiply gradient color.

@fix

evas: node that are used for a fill should not have a parent or they are going to be drawn.

@fix example for Efl VG.

evas: create ector surface per engine instance.

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

ector: cleanup default backend drawer.

ector: convert to freetype co-ordinate before storing path data in software backend.

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

evas: clip the clip rect to surface boundary in ector draw.

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

evas/gl_generic: set draw context in ector_renderer_draw

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

eolian: split tokens/keywords for cleaner indexing

ecore_x: fix the memory leak.

@fix

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