ximmodule: check whether pointer is NULL before accessing in xim_preedit_callback_set

ecore_imf_context_data_get () can return NULL, so it should be checked before using the result value.

This fixes CID 1135273

Merge branch 'withdrawn'

Implement withdrawn_set/unset on Wayland engines. It basically calls
ecore_evas_show/hide. Some fixes were done to show/hide on both EGL and
SHM backends too.

This set of patches should fix T155.

ecore_evas/wayland: Add a function to cleanup the frame callback.

This callback must be manually destroyed and removed on
ecore_evas_hide(), because it won't be delivered anymore after the
surface is destroyed. If the callback still exists, the engine will find
it and avoid doing a new redraw until it is finally called.

Maybe the correct thing to do is to keep this callback in the
Ecore_Wl_Window struct, and have some functions to set/unset it, so it
gets destroyed when the window is hidden. Or when the surface is
destroyed.

ecore_evas/wayland_egl: Unset the surface on window hide.

We must unset it, so the Evas engine knows that it has to do all the
setup to destroy the EGL surfaces associated with it. And in the case of
an ecore_evas_show, recreate everything again.

ecore_evas/wayland: Implement ecore_evas_withdrawn_set.

Just call ecore_evas_show/hide inside the function.

evas/wayland_shm: Don't use a global var to store the sent buffer.

When an Ecore_Evas is hidden, it will destroy the buffer swapper. When
it's shown again, it will try to attach a new buffer, that can be same
buffer. If that global var is still pointing to the old buffer, it can
match to it again and avoid sending a new buffer. So, just put this sent
buffer var in the buffer swapper, and it will get set to NULL when the
swapper is destroyed and created again.

This should fix an intermitent problem of ecore_evas_show() not always
working after an ecore_evas_hide() on the wayland_shm engine.

Update Korean translation for the efl1.8 release

Summary: Translation patch for the efl 1.8. of course, existing one has no problem, but some of sentence structure is enhanced.

Reviewers: seoz

CC: seoz, cedric
Differential Revision: https://phab.enlightenment.org/D360

NEON vectorization: added use of COLSAME define in map routine

Reviewers: raster

Reviewed By: raster

CC: cedric
Differential Revision: https://phab.enlightenment.org/D341

Revert "fix smart sma by allowing more recursion"

This reverts commit 1319733caeb811cd0d53b589b8a11f4229455ed3.

Let's revert this patch and lower the recursion number.
As we released efl 1.8, let's find the root causes of the problem and fix them.
I already told this revert to raster so I am not mean :)

Adding the cubic-bezier curve to edje transitions

Summary: Adding an option to use a cubic-bezier curve in edje transitions.

Reviewers: Sachiel, cedric, raster

Reviewed By: raster

CC: raster
Differential Revision: https://phab.enlightenment.org/D319

Eldbus: added function for getting bus unique name

Summary:
Hello guys,

We are now working on a accessibility support for elementary (ATSPI2) and we need following function to correctly register application.

Reviewers: cedric, raster, lucasdemarchi

Reviewed By: raster

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

edje_edit: Add getters and setters for fixed param

Reviewers: cedric, seoz, raster

Reviewed By: raster

CC: cedric
Differential Revision: https://phab.enlightenment.org/D339

FUCK - remove logic to silently add -release names to libtool builds

sneaky hidden m4 rule to ADD -release to shared lib names IF profile
!= dev profile. come on! why do that? seriously. this snuck in and was
undetected because i recompiled things against efl and thus things
linked against the new releasename libs. this requires an efl 1.8.1.
argh!

ecore ecore_x_e: check 0 and -1 for ecore_x_window_prop_card32_get() return value which mean fail.

- ecore_x_window_prop_card32_get() returns:
  -1 if the property was not successfully fetched.
  0 if the property exists but has no elements.

NEWS: added EFL 1.9.0 NEWS entry.

ChangeLog: added 1.8 release entry.

fix unchecked return value of stat in ecore_file_mv

stat() function may fail and return an error code

This fixes CID 1039697

evil: use the right ifdef.

v1.8.0

readme - make requirements list a bullet list

add some README dependency notes

eina - fix set-but-unused warning in eina_cow when valgrind off

release work - switch to release profile

this now maxes out max log level to 3 (same as dev otherwise)

evas event freezing is "wrong" for core events - turn it off

this turns off event freezing for SOME event types for eveas canvas
events (was was intended to begin with). this fixes T577

evas - pre/post render events are for canvas not object

the eo event naming was wrong for pre/post render event callbacks.
they are for the canvas as a whole, not per object

fix freetype header includes to be correct where ft 2.5.1 breaks builds

this should fix T591

fix eina_cow issues. Closes T581, T527

ecore/ecore_file: NULL-check for input parameter

Add check by NULL for input parameter 'path' in method ecore_file_mkpath

Reviewers: seoz, Hermet

CC: cedric
Differential Revision: https://phab.enlightenment.org/D349

eina: formatting

efreet: add async menu test

efreet: improve locking

Do find and ref inside lock, so no one can ref-- after we find desktop

ecore/wayland: Use touch_focus for touch_based events.

Also call the mouse related events with BTN_LEFT instead of 0, otherwise
the correct button won't be assigned to the event (button 1).

ecore/wayland: Send a mouse up event when the implicit grab starts.

The previous workaround used to send a mouse up event to the
application, using the pointer enter callback to determine when an
implicit grab has finished. This was "simulating" a mouse up event when
the surface move or resize finished.

However, this doesn't work for touch-based move. The pointer enter isn't
emitted, because the wl_touch interface doesn't have the concept of a
pointer.

Changing this code to make it more similar to what Weston's toytoolkit
does, the mouse up event is sent as soon as the move or resize grab
starts. After that, the compositor takes care of the move/resize almost
entirely.

Should fix T468.

ecore/wayland: Get the touch up event position from the down_info.

down_info is a struct that stores some information about the current
pressed touch events. It should be used for that specific touch point,
instead of the generic input info, when sending a mouse_up event.

ecore/wayland: Remove list of touch points from input structure.

We already have the "Ecore_Wl_Mouse_Down_Info" list, that contains this
information.

eldbus: Fix crash caused when the object of an monitored proxy is deleted

This happen because proxy was already freed and we try print some information
about the proxy in error message.

This fix: https://phab.enlightenment.org/T543

Revert "workaround for T543"

This reverts commit b663b5b8c7232d38ba257f99490cfa22814d7970.

readme update - mention systemd / udev for eeze deps

v1.8.0-beta2

Fix some typos/grammar issues in new README

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

update images.mk

readme - update readme for efl 1.8

lots of extra documentation and information now in the README

evas - fix gl egl textrue from pixmap support y invert query extension

Instead of hardcoding yinvert=1 we should query for
EGL_NOK_texture_from_pixmap extension and if it's present
use EGL_Y_INVERTED_NOK value

evas/cserve2: Fix interrupt on select in edje_cc

Summary:
Block SIGCHLD during select().
This fixes a bug with edje_cc when EVAS_CSERVE2=1: Fixes T464.

select() used to return prematurately with EINTR because the
app received some unexpected signals. In particular SIGCHLD
is received when a child terminates, but this is not a reason
to cancel the image load.

In theory, all blocked signals in pselect() should be pending
until pselect returns, so any SIGCHLD should still trigger
the app's signal handler.

Reviewers: cedric

CC: raster, cedric
Maniphest Tasks: T464

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

evas: roundup the step size of an array to fit in one page of memory.

This make an array stay in 4K instead of 20K before. It should reduce a little
bit our peak memory usage without slowing down things to much.

eina: fix rounding logic to avoid making the array to short.

I have no idea how the previous formula was supposed to work at all, but
this one is the same as our alignof code to make sure we do allocate to
the really nearest size and don't do over allocation. Additionnaly it works.

ecore_evas: fix wrong check in _ecore_evas_buffer_msg_send

'if (!child_ee)' implies that "child_ee" is equal to NULL, but
child_ee is used in the nest of if statement.

This fixes CID 1039431

xim immodule: fix dereference before NULL check issue in client_window_set

The variable 'info' can be NULL, therefore it should be checked before accessing.

This fixes CID 1039640

ibusimmodule: check whether pointer is NULL or not before accessing.

The pointer variable str can be NULL, therefore we should check whether it is NULL or not
before accessing.

This fixes CID 1132621, 1132622, 1132623

doc: add missing images

The edje-nested.{png,eps} files were missing in the doc tree. I
regenerated them using the sample code from tutorial_edje_nested.html

evas - code refactoring.

put the frequently used compare prior to the other.

fix eina_array_remove to actually realloc down in size to remove  bloat

eina_array_remove() didnt ever realloc down unless we went to 0
members. this wasn't very good as you'd expect the array to be reduced
in size if enough items were removed. not only that the old code was
stupid and ALWAYS malloc()ed a new array of the exact same size and
copied items over in the most complex way possible, then freed the old
one. this would have added overhead wherever used (evas_render) that
should not have been there.

this is based on the idea in a patch from
Viacheslav Lvov <v.lvov@samsung.com>, but this is a re-do of it
entirely, reducing the codebase massively even compared to the patch
and making it much simpler to read, maintain, actually reduce memory
and cut overhead.

ecore_ipc: prevent a double free.

According to clang static analyzer it is possible to find a path where
buf and svr->buf are pointing to the same array, better be safe than sorry.
Arguably this code could be more readable if it was using Eina_Binbuf.

evas: this value is never read after that point, simplify logic.

This has been spotted by clang static analyzer.

evas: simplify logic for better readability.

This also should solve some complain from clang static analyzer.

edje: make sure there is an initial value to avoid garbage out.

Detected by clang static analyzer.

ecore_con: remove unecessary code.

edje: correctly setup PATH and install Edje file as they are needed by examples.

This will fix T61.

evas/line - fix the line position set problem.

If the line goes to the 0, 0 then it won't be updated. Now it's fixed.

ecore ecore_x_e: initialize local variable to avoid having wrong value.

Thanks to devilhorns!

recursive monitoring and scanning fix for icons and desktop files

this fixes T580 ... or SHOULD fix it. there is recursion detection
code now and it properly follows symlinks and dirs. it also properly
updates the file monitor tree for both icons and desktops and it only
monitors dirs, not files (as a dir picks up changes to child data).
tested and it seems not to recurse into self-referencing symlinks
(once it detects the loop) and detects changes nicely in all my tests.

ecore_con: fix unitialized use of buffer from strncpy.

Fix CID 1039725.

ecore_con: fix non initialized use of buf with strncpy.

This fix CID 1039723.

evas: add missing define for some target.

Revert "Revert "evas: allow fuzziness on the texture format returned by GL.""

This reverts commit 87385b05c3117aa8d46fd4029bfdeadf3444a7b9 that is necessary for desktop target.

evas/cserve2: Update image loaders list

evas_image_load.c's list was updated to match the generic
loaders, in 38dd405712f5306fdb4b30.

The list used by cserve should be the same. Actually, there
should be a common function instead of code duplication here.

edje_entry: fix out-ouf-bounds issue in _edje_entry_imf_event_preedit_changed_cb

The maximum index of tagname is 7, and preedit_type_size is equal to 8.
Checking "attr->preedit_type <= preedit_type_size" implies that the value of "attr->preedit_type" may be up to 8

This patch fixes CID 1039308

evas evas_render: removed unnecessary semicolon from a macro.

This fixes coverity CID 1132631.

ecore ibus_imcontext: print the value of cursor_pos only when cursor_pos
is not null.

This fixes coverity CID 1132624.

ecore scim_imcontext: do not check null for ic. it was already checked
by EINA_SAFETY_ON_NULL_RETURN.

This fixes coverity CID 1132628.

emotion emotion_test: free allocated data when the dependent object is
deleted.

This fixes coverity CID 1099709.

ecore_evas/wayland: Update comments inside rotation code.

ecore_evas/wayland: Fix non-resize rotation.

Fix phab T392.

Notice that it should reopen T359, as it wasn't really fixed, but a
rotation with resize was being used when a non-resized rotation was
requested. The cause of the "protruding surfaces" is likely the fact
that Elementary is setting the opaque regions manually, instead of
leaving it to Ecore_Evas. This must be fixed either inside Elementary
itself, or adding the "surface extents" (shadow/non-visible surface
parts) info to Ecore_Evas too.

Revert "evas: allow fuzziness on the texture format returned by GL."

This reverts commit 2b5b3438e82a9a1bb4086864660942d3c1ddc113.

Breaks gl engines when using GLES + EGL.

Eo: Fixed eo_manual_free to always return a value.

I wonder how come this wasn't triggered in 64bit and only in 32.

Evas tests: Add canvas free with ref tests.

This should check test for the issue fixed in:
bb4a19b5ad6a8fc6720154700aabfa32f4b71244
8bba92fbca7c721a7582b098698404e491f44941

Evas object: Set manual_free to false once obj is detached from canvas.

This means unref will really free the object instead of just let it
linger in memory once the object is unrefed.
This is also correct because once detached from the canvas, Evas has
nothing to do with the object and it shouldn't need the manual free
anymore.

Eo: Make eo_manual_free() return a success flag.

eo_manual_free() can fail in some cases, and it is useful for users of
this API to know about it in order to decide what to do.

Evas layer: Fix deletion of canvas with still-refed objects.

This fixes the infinite loop while deleting canvases with still
referenced objects.
The canvas should not assume it can get rid of all of the objects. Some
objects might be referenced. I also added a printf to let the user know
about it.

This fixes T565.

evas - fix the memory corruption introduced by evas fonts_zero

Eo: Add an error message when old parent is wrong.

This is an addition to raster's fix in
55d8156414012e718a7e7adb3f9cd33e19744dc1
(eo - base class protection against NULL old_parent_pd)

ChangeLog: fixed formatting of Cedric's last changelog.

Ecore animator: delete animater's internal timer in _end_tick() in all cases.

When the ecore_animator_source_set() is called with different sources repeatedly, sometimes internal timer is not deleted and this leads animator misbehavior.
Especially when the source is changed from ECORE_ANIMATOR_SOURCE_TIMER to ECORE_ANIMATOR_SOURCE_CUSTOM before the SOURCE_TIMER's internal timer is deleted, this problem occurs.
In this case, even though _end_tick() is called in ecore_animator_source_set(), the SOURCE_TIMER's timer is not deleted because the source is already changed to CUSTOM.
So we should delete the internal timer in _end_tick() in all cases.

eo - base class protection against NULL old_parent_pd

bug T569 still shows that we have cases where, during e shutdown, we
still get eo_data_scope_get() return NULL for a parent object.
whatever this is, segfaulting is much worse than protecting and
marching on. so protect

evas: allow fuzziness on the texture format returned by GL.

evas - stop over redraw of map content with smart object.

clip object would always stay in a changed state. nobody was walking them to reset.
Now we do reset there state as soon as we do render them.

fix clang warning in _edje_entry_focus_in_cb

Ecore con dns test: Fix test hangs.

This should fix the test hangs on Jenkins. Fixed them for me.
The problem was, that the timeout was not handled correctly.
The server object was deleted, but the mainloop was not stopped.

Evas textblock: Fixed wrapping of lines ending with whites.

Whites at the end of lines ending with whites should not be cut, but
should be wrapped (there's no legal line break there).

Thanks to Shilpa Singh for reporting.

Evas textblock: Added a test for wraps ending with whites.

This tests for word wraps of lines ending with whites.
Those whites should not be cut, they should be wrapped as well.

evas - gl engine - async texture upload - protect null tex handle access

this stops a very blantant segv when accessing async->tex when it is
NULL on _evas_gl_preload_main_loop_wakeup().

ecore_file: Print more informative message when inotify_add_watch was failed.

inotify_add_watch can be failed because of several reasons.

Fix keyrepeat going crazy ;)

Previously, if you were hold down shift for 1-2 seconds and then press
a key, you would get superfluous key repeats (even tho you released
the printable key). This was because the "key repeat" code was not
checking for the same key before (re)starting the repeat timer.

This fixes the repeating key issue by checking if the key pressed is
different than the one already pressed. If so, it will (re)start the
timer. If it is not different, then the timer is already running and
we don't need to do anything.

Fixes T552

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

Fix ecore wayland key events to fill in Ecore_Event_Key with X11-like
values.

Previously, the keyname and key fields of the Ecore_Event_Key
structure were being filled in with the capitalized version of the
key. This is due to xkb_keysym_get_name always returning keys with the
modifier applied. There is no actual function in xkbcommon to Not do
this :/ so we have to manually check if Shift is pressed, and if so
then we need to convert the key to lowercase.

Fixes T550

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

evas - code refactoring.

merge duplicated(almost) functions to one.

1.8.0-beta1

efl - fixed ugly indentation.

evas - fix proxy render to update the proxies recursively.

If a proxy has proxies, all the chained proxies should be updated recursively.

fix smart sma by allowing more recursion

this ups the recursion allowance to 4096. it solves T494

evas - update extension/module loader list to match generic loaders list