eldbus: fix _eldbus_message_iter_arguments_vget

Summary:
there is a problem with _eldbus_message_iter_arguments_vget.

Assume: We have a array of type y, there are 3 elements in the array.

Lets say we are iterating with

printf("%c", dbus_message_iter_get_arg_type(iter));

and

dbus_message_iter_next(iter);

throuw the iter.

You will see that this will output yyy. As we are having 3 times v.
While the signature of the message iterator is y.

If you now call eldbus_message_iter_arguments_get(message, "y", &cont)
it will return false, with the errormessage in line 766, reason for this
is that the type of the signature iterator differs from the message
iterator. Because of the upper example.
So all in all: The signature given to the method has not to be equal to
the signature of the message iterator, it has to be equal to the
iterated signature, which cannot be found easily with the eldbus api.

The solution is to only iterate in the message iterator that long until
the signature iterator is at the end, if the type differs in this
region, give a error and return false. Otherwise return true.

The sad thing about this is that it is a behaviour break, but the
behaviour is borked, so its a fix.

I tested this over a week now, without any problem in efl/elm/e.

Reviewers: stefan_schmidt, zmike

Reviewed By: stefan_schmidt

Subscribers: zmike, ceolin, simotek, DaveMDS, cedric

Projects: #efl

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

Edje tests: Test inheritance of flag no_render

edje_cc: Fix inheritance of "no_render" flag

@fix

edje_cc: Add "render" keyword for lazEDC

Since @zmike also thought this could be useful, let's add it.

ector: don't leak cairo context.

eio: improve documentation.

This set of documentation update basically make it clearer that Eio
use Eina and do cross linking of function used by Eio to Eina. It
also copy some of the warning coming from Eina documentation into Eio.

eina: fix misleading documentation regarding enum values.

evas-wayland-shm: Reduce size of resize pool during resize events

Summary: We really don't need to be allocating a shm pool this large
during resize, so reduce the size of the pool.

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

ecore_ipc: fix typo in ecore_ipc_server_send() API doc

ecore_imf: ibus - remove logically dead code

fixes CID: 1261460

edje_cc: Add "norender;" as lazEDC for "no_render: 1;"

edje_cc: Allow PROXY as clipper (clip_to and desc.clip_to)

This arbitrary limitation can now be lifted since masking
is well supported (ie. clip_to can point to an IMAGE rather than
a RECT).

@feature

evas-gl-drm: Add support for 'smart' merge mode

Summary: This just adds support for 'smart' merge mode in the evas
gl_drm engine to keep in line with gl_x11 engine.

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

evas-wayland-egl: Add support for 'smart' merge mode

Summary: This just adds support for 'smart' merge mode in the wayland
egl engine to keep in line with the gl x11 engine

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

edje_cc: Check default state as well

Strangely only the other states were checked for errors. Errors
include:
- no name for state, or "description with missing state"
- invalid clip_to

Also improve the error message a bit

Edje: Fix big oopsie in previous commit

Edje: Turn SNAPSHOT objects into filled images for now

Without that, the image has no fill information. Fill properties
may need to be added to SNAPHOT parts but the default behaviour
should make sense. Before this patch you just get a black rectangle.

Considering how image filters currently work, marking snapshots
as filled by default is not the best solution (need padding_set(0)
to render nicely).

Evas filters: Fix handling of FILL for image filters

Note: Image filters still need a LOT of work to be usable.

Edje: Fix invalid loading of snapshot parts

Edje: Add test case for SNAPSHOT

This is mostly an edje_cc compilation test.

Edje: Add filter support to SNAPSHOT parts

This makes SNAPSHOT a part type on it own, combining the
common and filter subtypes.

This means it is now possible to set an evas filter on
a SNAPSHOT object, just like for TEXT, IMAGE and PROXY.

@feature

Edje tests: Add PROXY part to the filters test case

Only compilation is tested here.
Use edje_player to check the resulting edj if you wanna see what
happens.

Edje: Fix IMAGE filters

Some invalid logic made filters work only in a rare situation.

@fix

Edje: Add support for filters to PROXY parts

Same syntax as for IMAGE and TEXT: description.filter

Edje: Fix filters for IMAGE parts inherited states

Edje: Move internal struct filter around (refactor)

doc/eldbus: fix parameter in document

Summary:
The allow_initial_call apprears two times in
eldbus_name_owner_changed_callback_add api.

@fix

Reviewers: seoz, Hermet

Reviewed By: Hermet

Subscribers: cedric, seoz

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

po: update po files

evas: include new header file in build to fix distcheck

Introduced in 72b13c3ebb0b5f117f3602871ec2842d4baac786 the new header was
not deployed and thus not available in the tarball.

evas_canvas3d: Unification algorithm generation shadow

Summary:
Change type of texdepth from GL_R(doesn't support with GLES) to GL_RGBA.
Don't use useless additional framebuffers for shadows.
Use same shader code for generation shadow map texture.
Turn always using software alpha-test.
It reduce and simplify code in general.

Reviewers: cedric, Hermet

Subscribers: cedric

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

edje_cc: Fix TEXT part validation for aliased parts

Not 100% sure of this fix but it "works". This is just compile-time
safety checking code after all, so it won't impact any existing
application.

This fixes commit d4444bf7a0e368097bec555. Tested with empc.

Thanks @zmike for the report and test case.

evas: Add defines for GL_COLOR_EXT, GL_DEPTH_EXT, GL_STENCIL_EXT

evas/gl_common: Fix memory leak for font glyph extension image

Evas_GL_Image created for font glyphs in evas_common_font_rgba_draw
is sometimes freed after Evas_Engine_GL_Context is freed.
Since gc is already freed, pt_unref returns and leaves pt behind.

eolian-cxx: Fix inheritance from C++ classes after Eo ABI breakage

Fixed removal of op field from Eo_Op_Description and removed
unnecessary use of internal Eo API to implement the Eo API function
for theh constructor of the inherited class.

evas: move all GL safe define to a common header and use it.

evas: use void * instead of struct wl_* to avoid compilation issue on system without Wayland extention.

edje: fix a memory leak while iterating used color class.

ecore-audio: defer deletion of pulseaudio timer until after event processing

==27523== Invalid write of size 8
==27523==    at 0x9E855F5: _ecore_time_wrapper (ecore_audio_pulse_ml.c:132)
==27523==    by 0x647E5CF: _ecore_call_task_cb (ecore_private.h:336)
==27523==    by 0x647FB8B: _ecore_timer_expired_call (ecore_timer.c:733)
==27523==    by 0x647F9EE: _ecore_timer_expired_timers_call (ecore_timer.c:686)
==27523==    by 0x647B4CE: _ecore_main_loop_iterate_internal (ecore_main.c:1814)
==27523==    by 0x647998E: ecore_main_loop_begin (ecore_main.c:983)
==27523==    by 0x4E4F676: elm_run (elm_main.c:1099)
==27523==    by 0x12801B: elm_main (test.c:1010)
==27523==    by 0x1280C4: main (test.c:1021)
==27523==  Address 0x20537208 is 8 bytes inside a block of size 56 free'd
==27523==    at 0x4A07D6A: free (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so)
==27523==    by 0x9E858ED: _ecore_pa_time_free (ecore_audio_pulse_ml.c:204)
==27523==    by 0x108DB350: free_events (socket-client.c:109)
==27523==    by 0x108DBA53: do_call (socket-client.c:157)
==27523==    by 0x9E855F0: _ecore_time_wrapper (ecore_audio_pulse_ml.c:131)
==27523==    by 0x647E5CF: _ecore_call_task_cb (ecore_private.h:336)
==27523==    by 0x647FB8B: _ecore_timer_expired_call (ecore_timer.c:733)
==27523==    by 0x647F9EE: _ecore_timer_expired_timers_call (ecore_timer.c:686)
==27523==    by 0x647B4CE: _ecore_main_loop_iterate_internal (ecore_main.c:1814)
==27523==    by 0x647998E: ecore_main_loop_begin (ecore_main.c:983)
==27523==    by 0x4E4F676: elm_run (elm_main.c:1099)
==27523==    by 0x12801B: elm_main (test.c:1010)
==27523==    by 0x1280C4: main (test.c:1021)

@fix

Ecore win32: Fix compilation.

Broke in commit 015040d13f1340529cf31ee73d6b25987c23a670.
Please compile check before pushing.

ecore_win32: fix the mouse out behavior.

Current win32 let the mouse out regardless of mouse down status.
This is not quitely same with the x system so widget behaviors were not properly working.
Make the widget behaviors about mouse-out same to x window system
to keep the same behaviors on all window system always.

@fix

ecore_win32: code refactoring.

replace to ECORE_CALLBACK_PASS_ON from 1 for readability.

Eo: Clean up windows code a bit more.

Merge more parts of it with the non-windows code.

po: Add Vietnamese translation

Summary: Add translation for Vietnamese.

Reviewers: seoz, stefan_schmidt, Hermet

Subscribers: cedric, seoz

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

upower: free version string splits

@fix

eolian generator: refactoring of file reads

eolian generator: no need to replace newlines on win32 (lexer can deal with it)

ecore_buffer: fix typos in documentation

Note by TAsn: I accidentally reverted this commit, bringing it back.

Eo: add a regression test for broken callback comparison.

This bug was fixed in d889da6b12e6123073edb54d766c9af1fc6db822, but it was too
late at night to start hacking on extracting a regression test and adding it to
the suite, so adding it now.

This test makes sure that we only fallback to string comparison with legacy
events.

ecore_buffer: fix typos in documentation

evas: fix typos in documentation

Eo legacy events: Fix broken callback comparison

Commit 37f84b7e966372384e2dfe5d191a6f907a17962e introduced a few changes
to the callback matching mechanism that made it so sometimes callbacks
would be triggered for the wrong events. The problem was there because
of the support for legacy events that forces to do string comparison
instead of the usual pointer comparison. We should only do string
comparison when we are certain one of the callbacks is a legacy
generated one.

Regression tests will follow tomorrow. Way too late here for that.

Thanks to cedric for reporting.

Eo: reduce memory usage across applications.

As described by Carsten in his email to edev ML titled:
"[E-devel] eo stability - i think we need to postpone that"
with the switch to Eo2 we significantly increased our usage of RW memory
pages, and thus significantly increased our memory usage when running
multiple applications.

The problem was that during the migration to Eo2 the op id cache and the
op description arrays were merged, causing the op description arrays to
no longer be RO. This patch enables users of Eo (mainly Eolian) to
declare those arrays as const (RO) again, saving that memory.

There might be performance implications with this patch. I had to remove
the op desc array sorting, and I used a hash table for the lookup. I
think the op desc sorting doesn't really affect performance because that
array is seldom accessed and is usually pretty short. The hash table
is not a problem either, because it's  behind the scenes, so it can be
changed to a more efficient data structure if the hash table is not good
enough. The hash table itself is also rarely accessed, so it's mostly
about memory.

Please keep an eye for any bugs, performance or excessive memory usage.
I believe this should be better on all fronts.

This commit *BREAKS ABI*.

@fix

eolian tests: strip trailing newlines for ref file compare

ecore_con: fix typo in documentation

eldbus: fix typos in documentation

eio: fix typo in documentation

emotion: fix typo in documentation

eet: fix typo in documentation

Eolian generator: remove sentinel generation

Eo: Fix examples and benchmarks according to recent Eo changes.

Eo: Remove EO_SENTINEL.

This was never really needed because we always had the count.
Removing this now because we are already breaking API and ABI.

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