Eo: Clean up windows code.

We were keeping a struct member that wasn't really needed on windows.
Since we already broke ABI, we can afford to clean this up.

Eo: Fix eo function name getter on windows.

Eo: rename EO_HOT_EVENT_DESCRIPTION to EO_EVENT_DESCRIPTION_HOT.

The old naming is inconsistent with the rest of the EFL. This fixes that.
Since we are already breaking ABI (and possibly API), we should fix this too.

Eo: Drop doc field from ops and events.

This hasn't been used for a while. Since we are going to break Eo a bit anyway
it's a good opportunity to drop this.

This may cause a slight performance issues with legacy events, such as
smart callbacks. This shouldn't really be a problem as we've migrated away from
them. If it does, we need to migrate the remaining parts. Only relevant
for callbacks that are added before the classes are created, which
shouldn't be possible except for smart, only for old evas callbacks.

ecore: fix typos in documentation

edje_edit: fix typo in error message

edje: fix typos in documentation

evas - protect against invalid obj id in legacy wrapper func

@fix

evas - lock render updates as this happesn to be accesed from threads

evas - software x11 - lock shmpool as now shm pool entities are threaded

recent changes moved shm buffer pushig to a thread and thus shm pool
has to be locked too

Eo: fix double callback deletion

Scenario:
- Same signal/function/data registered twice on e.g mouse_down
- On mouse_down, register mouse_move and mouse_up
- On mouse_up, unregister mouse_move

Result: mouse_move still invoked after mouse_up

Reason:
- When the mouse_move callback deletion is required, the cb is
flagged as deleted but is not freed as walking_list blocks.
- When the second (and same) has to be deleted, it will try to delete
the first again because the delete_me flag is not checked.

This patch fixes it by checking the delete_me flag when determining the
candidate.

@fix

ecore_evas-x11: unset withdrawn flag when showing the ecore evas

while the window map event seemed like a reasonable place to unset
the withdrawn state at the time, studies and further tests have proven
that the direct show callback is even more reasonable and effective

ref T2745

ecore_evas-x11: unset withdrawn state when window is mapped

according to ICCCM 4.1.4:
Newly created top-level windows are in the Withdrawn state.
Once the window has been provided with suitable properties,
the client is free to change its state...
...
Only the client can effect a transition into or out of the Withdrawn state

given that no external force can (according to spec) transition a
window out of the withdrawn state, this must be done at a reasonable
point. mapping the window seems like a reasonable point to me.

fix T2745
ref 5954289c6ce1cd55ff212428291604b981438439

@fix

Edje entry: Make cursors to null after free

Summary:
Edje entry: Make cursors to null after free

To prevent any illegal access of cursors
can lead to crash

@fix

Test Plan: NA

Reviewers: tasn

Reviewed By: tasn

Subscribers: shilpasingh, cedric

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

edje_adit: fix typos in header

Summary: @fix

Reviewers: cedric, raster, Hermet, reutskiy.v.v, FurryMyad

Reviewed By: FurryMyad

Subscribers: cedric

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

evas gl - optimize updates

@feature

this makes the gl engine by default not do bounding box, but instead
try and smartly merge nearby update regions. this means multiple
render passes IF your drivers support buffer age, but it seems to
actually help. in my test case on nvidia drivers which support buffer
age, i saw compositor cpu overhead drop by about 30%

evas image object - dont update just if image is dirty

marking images as dirty is only meant to have the pixle get callback
called IF rendered. you ADD update regions to get it to be called.
this is how it has always meant to work, but evas image objects where
adding updates just if dirty. this ends up with e as a compositor
redrawing ENTIRE eindows if even a single blinking cursor is blinking.
oh so bad.

@fix

fix coverity complaint (not real bug) - CID 1324882

coverity complained on the n <=- 0 return. it will never be <= 0 if
rects2 is non-null. this just should make coverity less noisy.

Evas smart: Add obj type safety check to callback_del.

Summary:
Evas_Smart_Data retrieved from eo_data_scope_get might be NULL
in case the input Evas_Object is not valid Evas_Object.
The smart data gets dereferenced and can cause a crash if its NULL
Hence return in case smart data is NULL.

@fix

Signed-off-by: Mrunal Sovani <mrunal.s@samsung.com>
Reviewers: raster, Hermet, tasn

Reviewed By: tasn

Subscribers: singh.amitesh, cedric, sachin.dev

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

eolian: enable doc dependency parsing code

eolian: no need to double-sanitize paths, only do it when needed

eolian: use fully sanitized path names everywhere

evas - make new exact regions round up to 16x16 to keep count sane

make region count sane by rounding to 16x16

evas_canvas3d:Broken source file for generation 3D shaders

Summary:
Bug reproduce if delete evas_gl_3d_shaders.x and try re-build.
Source file include.shd for generation shader code consist m4 macro definitions.
After this revision https://phab.enlightenment.org/D3056 order of the quatation
marks in function pcf was broken.
@fix

Reviewers: cedric, Hermet

Reviewed By: Hermet

Subscribers: cedric

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

ecore_x_keygrab: fix memory leak

evas cutouts - optimize to use less cpu

this optimizes draw ctxt cutouts by skipping small ones and
remembering the last cutout added so it isn't double-added as well as
extending the minimum cutout array to 512 and going up in blocks of
512 instead of 128. also optimize the clipping code a bit more.

evas tiler update handler - move to region code to be accurate and fast

this move evas tiler that does update handling to use fully correct
regions using region.[xh]. this also removed old unused regionbuf code
and a bunch of commented out code no longer needed. much simpler now
and easier to maintain.

evas new region code - copy region size along with region math

eina: add example for eina_tmpstr.

Summary:
Example for eina_tmpstr added.

Example tests for eina_tmpstr_add_length, eina_tmpstr_len, eina_tmpstr_del and eina_tmpstr_strftime APIs

Signed-Off By: Shilpa Singh <shilpa.singh@samsung.com>

Reviewers: cedric

Subscribers: cedric

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

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

ecore_x: remove useless assignment.

Summary:
The assignment of NULL will have no effect on the caller. So removed that statment.

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

Subscribers: cedric

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

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

ecore_con: replacing strncat with strncpy.

Summary:
strcat will look for the null-terminator, interpret that as the end of the string, and append the new text there, overwriting the null-terminator in the process, and writing a new null-terminator at the end of the concatenation. buf is uninitialized, so it might start with NULL, or it might not have NULL anywhere within it. So this might produce undefined behaviour. So replaced with strncpy.
Signed-off-by: Srivardhan Hebbar <sri.hebbar@samsung.com>
Reviewers: cedric

Subscribers: cedric

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

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

ecore_exe: fix memory leak on realloc.

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

Reviewers: cedric

Subscribers: cedric

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

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

ecore_example: fix file name in the example's comment.

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

Reviewers: cedric, stefan_schmidt

Subscribers: cedric

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

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

ecore_con: preventing possible NULL pointer dereference.

Summary:
Checking if client is NULL before freeing is not of much use. If it is NULL, it would have crashed before when it was dereferenced in printf. So checking NULL before dereferencing.
Signed-off-by: Srivardhan Hebbar <sri.hebbar@samsung.com>
Reviewers: cedric

Subscribers: cedric

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

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

edje: fix internal colorclass field

Summary:
Its value is never modified directly and should not be modifiled by mistake
in future.

@fix

Reviewers: Hermet, raster, reutskiy.v.v, cedric

Reviewed By: cedric

Subscribers: cedric

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

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

eolian: preliminary doc dependency detection for external files

This doesn't quite work yet as path sanitization needs to be done for it to
work correctly. For now this code path is disabled and will be enabled once
all paths are correctly sanitized.

ecore_buffer: abort if wl_display_dispatch returns negative value

Summary:
If the connection between server and client has problems, wl_display_dispatch
an return negative value. i.e. when enlightenment has been killed accidently.
In this case, app can enter into infinite loop.

Signed-off-by: Boram Park <boram1288.park@samsung.com>
Reviewers: stefan_schmidt, zmike, gwanglim, raster, devilhorns

Subscribers: cedric

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

ecore_wl: abort if wl_display_dispatch returns negative value

Summary:
If the connection between server and client has problems, wl_display_dispatch
an return negative value. i.e. when enlightenment has been killed accidently.
In this case, app can enter into infinite loop.

Signed-off-by: Boram Park <boram1288.park@samsung.com>
Reviewers: stefan_schmidt, zmike, gwanglim, raster, devilhorns

Subscribers: cedric

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

ecore_evas_extn: remove old buffer in evas render post function.

Before, ipc_server_data(fd handler callback) is not runned between
evas_render_pre callback and evas_render_post callback.
but after async mode, hd handler can be called between render_pre and render_post.
we should remove buffer which can be used render thread, after render finished

edje_edit: remove a redundant API declaration.

edje_edit_state_font_get has twice declaration in same header file.

evas gl engines - remove TIMDBG code that is unused entirely

eldbus: add dbus_pending NULL check

Summary:
When we call dbus_connection_send_with_reply, the dbus_pending
can be NULL. In this case, the next call dbus_pending_call_set_notify
will cause application crash.
We should check if dbus_pending is NULL before calling dbus API.

@fix

Reviewers: cedric

Subscribers: englebass, cedric, seoz

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

edje: return immediately when deleting a color class if no users exist

Ector software gradient: Fix cast from integer to pointer of wrong type.

When casting pointer<->int, always use uintptr_t, not just int.

@fix

Edje_entry: emit "cursor,changed,manual" for Home/End/PgUp/PgDown.

Summary:
The keys Home/End/PgUp/PgDown are changes the entry cursor
position in the same way as arrow keys. For unified behavior
callback "cursor,changed,manual", added emitting signal for
those keys events.

Reviewers: tasn, cedric, herdsman, thiepha

Subscribers: cedric

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

evas sw generic - comment NULL methods so we know what they should be

edje: edje_edit - add getters/setters for a container item spans

This functions designed that make the edje edit API's more
homogeneous. The all edje attribute setters and getters
treated only ony attribute, besides container API's

edje: edje_edit - add getters/setters for a container item position

This functions designed that make the edje edit API's more
homogeneous. The all edje attribute setters and getters
treated only ony attribute, besides container API's

edje: edje_edit - add getters/setters for container align

This functions designed that make the edje edit API's more
homogeneous. The all edje attribute setters and getters
treated only ony attribute, besides container API's.

edje: edje_edit - add getters/setters for container padding

This functions designed that make the edje edit API's more
homogeneous. The all edje attribute setters and getters
treated only ony attribute, besides container API's.

edje: edje_edit - add getters/setters for min container size

This functions designed that make the edje edit API's more
homogeneous. The all edje attribute setters and getters
treated only ony attribute, besides container API's.

TODO:
 * separate edje_edit_state_container_padding_get/set
 * separate edje_edit_state_container_align_set/set
 * separate edje_edit_part_item_position_get/set
 * separate edje_edit_part_item_span_get/set

eio: add kevent backend.

Summary: Implement basic kqueue/kevent backend for eio. When it comes to tracking directory changes, this backend falls back to the polling one.

Test Plan: Ran Enlightenment for several days and some other EFL apps without any issue.

Reviewers: cedric

Reviewed By: cedric

Subscribers: cedric

Projects: #e_on_freebsd, #efl

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

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

ecore_x: fixing memory leak on realloc.

Summary:
If realloc fails, it returns NULL. Then whatever the memory the ignore_list was pointing to would be leaked. So freeing it now.

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

Reviewed By: cedric

Subscribers: cedric

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

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

eldbus: fix broken fallback interface.

Summary:
Previous implementation assumed that fallback interface path should be changed
on every method call. This is generally not needed since real request
path can be obtained directly from dbus message passed as method call
paremeter.

This patch reverts this behaviour and additionally fix broken Property and Introspect
interface handling of fallback interface.

Reviewers: raster, lucasdemarchi, cedric

Subscribers: cedric, seoz

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

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

eina_tmpstr: add eina_tmpstr_strftime

Summary:

@feature

Test Plan:
eina_tmpstr_strftime API can be used to create a temporary string
which is updated with strftime output

eina_tmpstr_steal can be used to get actual string set in eina_tmpstr

Reviewers: cedric

Subscribers: rajeshps, cedric, govi

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

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

edje_edit: remove save from group_del and group_copy

Summary:
edje_edit_save_all includes source code generation that is way more expensive
than save itself. User should be able to save manually with or without sources

Reviewers: cedric, Hermet, seoz, reutskiy.v.v

Subscribers: cedric

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

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

emile: fix can not check ifd offset of jpeg in MM(little endian) format

IFD offset is 4 byte.
But just one byte is checked for it in previous patch.

Reviewers: Hermet, jypark, cedric

Reviewed By: cedric

Subscribers: cedric

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

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

evas_canvas3d: sets for shadow rendering

Summary:
Add possibility change quality and offsets for shadow.
Add mesh API's to sets size, step and bias constant.
Add scene API to sets depth values.

Reviewers: Hermet, cedric

Reviewed By: cedric

Subscribers: cedric

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

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

emotion: lock pending events and pending objects

Summary:
_emotion_pending_ecore_begin and _emotion_pending_ecore_end are not called from
the same thread. Indeed, _emotion_pending_ecore_begin is called from gstreamer
callbacks, and _emotion_pending_ecore_end is called from the mainloop.

Reviewers: cedric

Reviewed By: cedric

Subscribers: cedric

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

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

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>