Evas textblock: Fix selection with bidi text.

This is a regression introduced in
548e548632333f5ef533079a7aaa54aadd20ef8d.

This is really bad, and essentially broke selection geometry for bidi
text. Very serious.

The problematic code assumed that the range comparison for the items
assumed the item marked with 1 is always logically before the item marked
with 2, which is just not true.

Evas textblock: Added a test for range geometry with bidi text.

fix evas_tiler bug (makes evas get stuck in an infinite loop there in certain scenarios)

efl: fixed documentation typos.

Edje entry: Fix memory corruption (of by one in allocation).

This fixes T867.

evas_render: avoid unnecessary assignment.

Edje entry: Fix anchor href parsing.

It should now correctly work with arguments surrounded with either
quotes or spaces.

entry: clear selection before selecting new one

ecore-audio: remove src files that are never compiled

ecore audio: tests - remove #if0'd out code

checking for libmount newer than 2.19.0 means newer, not older

allow --without-tests in addition to --with-tests=no

configure: Add ecore-imf to ETHUMB_INTERNAL_LIBS

It is to fix possible build break of ethumb

I got below message.
lib/edje/.libs/libedje.so: undefined reference to `ecore_imf_context_retrieve_selection_callback_set'
collect2: error: ld returned 1 exit status
make[4]: *** [bin/ethumb/ethumb] Error 1

a small ecore_getopt bugfix

eina: fix test suite for rwlock... hopefully.

autotools: let's not override what we just did.

evas - textgrid - clear up unused var warning

evas: textgrid - fix crash in terminology with rare characters.

So I have a weird crash in terminology.
Reproduction path:
eet -x /path/to/elm/theme/default.edj edje/images/537

Scroll back in the terminal buffer, to show the entire file: CRASH.

Reviewers: cedric, tasn

CC: cedric, raster
Differential Revision: https://phab.enlightenment.org/D468

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

embryo: force the insertion of a '/' in the path to be sure that file path make sense.

eina_tls: fix calls to TlsFree

Registers a fallback handler for a given subsection of the object hierarchy.

Summary:
Goal of this patch is cover in Eldbus library function dbus_connection_register_fallback.
http://dbus.freedesktop.org/doc/api/html/group__DBusConnection.html#gac4473b37bfa74ccf7459959d27e7bc59

Reviewers: stanluk, raster

Reviewed By: raster

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

eina_tls: add eina_tls_cb_new(Eina_TLS *key, Eina_TLS_Delete_Cb delete_cb)

Summary:
   delete_cb is called at thread exit for each Eina_TLS keys used by the thread

Details:
   posix:
      pthread_key_create(key, delete_cb); does it
   win32/wince:
      eina_tls_free/new un/registers key&&cb into a static eina_list.
      eina_tls_set add the key to an eina_list in Eina_Thread_Win3.
      this list is cleared and callbacks are called in _eina_thread_join()

Test Plan: win32/wince has to be tested, I have no setup to do it.

Reviewers: cedric

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

edje: Support to set or retrieve selection of ecore_imf

ecore_imf: fix to call function of immodule when the input method property is changed

ecore_imf: Add selection_set callback type to allow Input Method to select a region of text

Add ecore_imf_context_selection_get API to allow Input Method to get the selected text.

embryo_cc - remove trainling / from path for windows stat so it works

Evas: Fix proxy rendering for animated objects

An invalid optimization was implemented in proxy rendering.
We can't assume a proxy is a smart object.

Refer to 5cefa00d0ad0f54.

Fixes T832.

Proxy rendering is still broken when using cserve2... :(

eina: do not access dead pointer from within valgrind macro.

ximimmodule: fix dereference after null check

CID 1162749:  Dereference after null check  (FORWARD_NULL)
/src/modules/ecore_imf/xim/ecore_imf_xim.c: 1027 in _ecore_imf_xim_preedit_draw_call()

okay, bring back xrandr < 1.4 compat for ecore_x_randr_output_name_get

apparently I read the commit order wrong and this fix went in for 1.4.0, not 1.3.2, which means anyone who has 1.3.2 has been having lots of fun crashes for the past 24 hours

nul terminate string from ecore_x_randr_output_name_get

Synced libunibreak local copy with upstream.

This fixes T805.

Evas: Fix MMX mask function _op_copy_p_mas_dp_mmx()

This function does the following operation:
COPY pixel x mask --> dst

But it wasn't iterating over the source. So it was repeating
the value of the first pixel over and over again.

Is this even used anywhere? RGBA + alpha mask function!?

ecore_x_randr_output_name_get fixme removed

this was a fixme which was likely written sometime before July 2010 when the bug was fixed, just prior to the 1.3.1 release. I think it's probably okay to require that release since it's been 3+ years.

Revert "eet: do not load data when the file is open in read/write mode."

This reverts commit f8b0322704a4d74e4a5545727c18dc8385318657.

this breaks efreets icon cache. i have been noticing this since
yesterday across all my machines once i update just efl. i tracked it
down to this commit.

_op_blend_c_dp_neon miscalculation fix

Summary:
When processing random data result of this function differs from C variant in more than 50% cases.
This difference is due to alpha calculation, in C code :

 a = 256 - (c >> 24)

in NEON:

"vmvn.u8  q7,q6       \n\t"
// ie (8 bit)~(c>>24)    ===   255 - (c>>24)

We cant just add "1" as overflow will occur in case (c>>24) ==  0  (we use only 8 bit per channel in vector registers)
So here is the solution:
copy *d right before multiplication and add it to the result of it later.

This makes the function slower by 20-30% but it is still at least 2 times faster then C code.

Reviewers: raster

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

eet_data: guarantee double-word alignment for temporary stack buffer.

The code was giving enough memory to store doubles and longs, but they
could be unaligned as "unsigned char" allows 1-byte alignment, while
double may require 8 bytes.

By specifying the array as "long long" we force certain alignment in a
platform independent way. As this array is small enough and
short-lived, the number of items were not changed, this results in
more bytes on the stack but it shouldn't matter.

eina/mp/one_big: fix alignment issues.

When over-allocating (past "pool->max" items) a memory slice will be
allocated to the new item as a linked list using Eina_Inlist.

The original code was placing the Eina_Inlist structure (3 pointers)
at the beginning of the allocated memory. However the item must have
proper alignment based on "pool->item_size", otherwise a structure may
end with unaligned members. Take for example MIPS 32 bits, it uses 4
bytes pointers with 8 bytes double. A structure containing a double
could have it unaligned as 12 % 8 = 4 (12 is the size of Eina_Inlist,
that contains 3 pointers), and MIPS doesn't allow unaligned access.

Albin Tonnerre (Lutin) spotted this in his Debian MIPS test machine,
it was breaking at eet_data_get_double() that was storing an unaligned
double. This was being called from within edje test suite.

The current code will place the list node after the requested
"pool->item_size", of course guaranteeing the pointer inside the node
is aligned (otherwise a "char" or "short" would break its alignment).

ecore_getopt: support for per-category help listing

configure: include required NEON build flags.

It is required to use "-mfpu=neon" to build NEON.

Patch-by: Albin Tonnerre <lutin@debian.org>

[ecore_x] Fix to handle paste null data case.

Summary: Do nothing when null data is pasted.

Reviewers: Hermet, raster

Reviewed By: raster

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

ecore - fix CRIT -> CRI macro

edje: Call _edje_entry_imf_context_reset in _edje_entry_user_insert.

Summary:
When user paste text to elm_entry, _edje_entry_user_insert is called.
We need to reset a prediction buffer just like _edje_entry_text_markup_insert.

Reviewers: seoz, Hermet, woohyun, tasn, jihoon, raster

Reviewed By: raster

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

eina-inlist - handle prepend/append relative if list is null (empty)

stable release - cherry-pick me!

eet: do not load data when the file is open in read/write mode.

I do think that it was a left over from previous file format. Removing
this memcpy should make Enlightenment startup faster and should reduce
by 500KB the memory it use.

eina: add eina_{str,bin}buf_manage_read_only_new_length().

It is sometime useful to start from a defined buffer, but to not touch it
until needed. This make life of caller more easier as they don't need to
duplicate the buffer themself as Eina will now take care of that.

eina: add eina_binbuf_append_buffer() and eina_strbuf_append_buffer().

eet: reduce memory usage of Eet internal structure by removing useless structure.

edje: Add changed signal emit in _edje_entry_imf_event_delete_surrounding_cb

Summary:
"_edje_entry_imf_event_delete_surrounding_cb" changes text of entry.
When the callback function is called and the entry is changed,
we need to notice to applications that the entry is changed.

Reviewers: seoz, Hermet, tasn, woohyun, jihoon, raster

Reviewed By: raster

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

edje_cc: factorize two strlen()s

Reviewers: cedric, raster

Reviewed By: raster

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

ecore_evas_wayland_common: fix compilation, missing bits of 892b1c5

Fix doxy typo

Signed-off-by: Chris Michael <devilhorns@comcast.net>

ecore_evas - applied macro to check rotation state.

also, let the potrait compare on the higher priority.

if you can suggest better macro name, then please modify it.

Revert "ecore_evas - use the macro orthogonal check."

This reverts commit 43acf1e82f94dcdca25d6781d8b27b015f4bd656.

some partial changes are incorrectly applied.

ecore_evas - use the macro orthogonal check.

efl - updated AUTHORS

evas: Fixed to compare current render_op with previous render_op for textblock. If the render_op is changed, call evas_object_render_pre_prev_cur_add.

Summary: This change is related to the commit: cdb86e34bbf82bd91aefd686bfc3bdf2a018a956

Reviewers: woohyun, tasn, Hermet, seoz

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

efl/ecore_con - compare same types.

efl/emotion - added missing headers for getuid() geteuid().

pkgbuild -> luajit

edje: use luajit by default, if you want lua use --enable-lua-old

Evas cserve2: Fix invalid memory access

In case of load error, we free the file entry, but forgot to
reset the pointer to it in the image entry.

edje: use EINA_UNUSED instead of __UNUSED__

emotion: Use the GStreamer 1.x module instead of the 0.10 one in the border example application

emotion gstreamer1: Call _emotion_frame_new() for audio-only streams without visualisations too

Without this applications are not notified of any progress.

https://phab.enlightenment.org/T767

edje_entry: When preediting, cursor should not be moved.

ecore_getopt: support for categories

Fix ecore_con example file to call shutdown functions.

Found by Chris Michael.

ecore_x_gesture: Cast maek pointer correctly.

Confirmed by the original author of libxgesture, Sungjin Park.

Textblock: Add test cases for style padding

In the previous commit, style padding has been changed, so
that lines don't get extra space just because there's a special
style (glow, ...)

This adds some test cases that check the actual geometry of the
lines relatively to each other.

NOTE: This test does not fail before the padding commits, as
_relayout_if_needed() adjusts the padding properly.

Textblock: Relayout if needed in line geometry get

evas_object_textblock_line_number_geometry_get() did not trigger
a relayout, so it could end up returning FALSE and invalid values.

Textblock: Fix padding with styles (part 2)

Summary:
If ONE single item in the whole textblock has a padding of k,
then ALL the lines of the textblock will be padded by k pixels.

Here's a solution to add the padding only to the first line.

Test Plan:
Write any multiline text, without styles, in an entry.
Add some glow to one element. All lines should be spaced by
an extra 2 pixels.

Reviewers: tasn

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

Textblock: Fix padding computation (part 1)

Track padding per paragraph, since this is how it is computed.

Problem before this patch:
- If markup text is changed, padding may grow, and the layout is updated (good)
- If the UI itself needs a relayout, the old padding value is NOT reused,
so style paddings will reset the padding to 0.

Test protocol:
- Set some text with style=glow. The whole object should have padding 2,2,2,2
- Relayout the UI, the whole object will have padding 0,0,0,0 (should be 2,2,2,2)

ecore_x: set cursor with xinput2

For some reason XDefineCursor might malfunction when using xinput2. So
when using xinput2, use XIDefineCursor. Also see:

https://bugs.launchpad.net/compiz/+bug/1179155

Evas/cserve2: Add some safety checks when reading socket messages

Fixes CID 1039571 and 1039572.

Evas/cserve2: Remove dead code

Fixes CID 1039925.

ecore xcim module - don't printf every time you start to add stdout noise

ecore ecore_x_e: check 0 and -1 for ecore_x_window_prop_window_get(). According to note for ecore_x_window_prop_window_get() :

Summary:
If the property was successfully fetched the number of items stored in
val is returned, otherwise -1 is returned.
Note: Return value 0 means that the property exists but has no elements.

Reviewers: seoz

Reviewed By: seoz

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

Evas/cserve2: Stop looking at those dirty pics!

What I mean is erm... images marked as dirty don't have a
cache_key, and that's perfectly fine when the image file
has changed.

Evas/cserve2: Fix potential crash in image_free

Don't try to free an object that was already freed.
Basically, allow image_free(NULL).

Eio monitor: Fix crash on invalid data access

Fix race condition when touching/changing a (theme) file often.
An Eio_Monitor was marked as "delete_me" but the rename callback
was still called, leading to memory access to already freed
objects.

Test protocol was:
ELM_THEME=~/default.edj elementary_test &
watch touch ~/default.edj

Evas/cserve2: Keep image_id when reopening a changed file

If a file had changed, a new OPEN message was sent, and the
client image ID was then changed, but the LOAD message was
sent with the previous image ID. So cserve2 would not be
able to honor that request.

Evas/cserve2: Clear up File_Data when opening a new image

After a file changed, the "changed" flag was not reset in the
File_Data structure, although it was supposed to represent a
new entry altogether.

Evas/cserve2: Rename "invalid" field to "changed"

This flag indicates that a file changed, not that the entry
is invalid. There is another "valid" flag already.

Evas/cserve2: Improve error messages a bit

Evas/cserve2: Add another load error check

This one is very unlikely as it would mean the image is
invalid while being in the hash.

Evas/cserve2: Fix crash on image file change

When an image file is changed, it is discarded from cserve2,
so the references become invalid. In case we were loading a
scaled version of that image, no proper error checking was
done, leading to obvious crashes.

Evas/cserve2: Reduce error logs on invalid glyph load

Change message level from ERR to WRN, when a glyph is not
loadable because FT fails to load it or it contains 0 pixel.

cserve2 used to complain about invalid glyph 3, on a few fonts

Evas/cserve2: Fix crash in server on file change

An inotify callback is triggered when an image file changes,
and it is supposed to cleanup all references to this image.

Unfortunately, it was doing it in a very unsafe way as pointers
could become invalid, because of hash free callbacks in the
referenced image list. Add some list safety and always assume
the pointer might be dead after free operations.

TBH, this _file_changed_cb function looks very confused to me
(as it tries to bypass eina_hash_del).

Evas/cserve2: Fix memleak during shutdown

Don't reset the dead slave callback, as it is used to free the
slave worker's resources.

Evas/cserve2: Use calloc instead of malloc

This will silence some warnings from valgrind (and add some
safety as well...)

ecore-x present fixups

ecore-x now supports XPRESENT

see http://cgit.freedesktop.org/xorg/proto/presentproto/plain/presentproto.txt

ecore_evas_win32: Fix build regression in ecore evas win32 engine

Regression introduced with fb59ac34a1ec65680ca4ff3fc62d1fc42aa72f5a
Simple typo during Eina_Bool convertion.

emotion gstreamer1: Set visualizations if the vis name is not NULL

and not the other way around.

emotion gstreamer1: Also handle DATE_TIME tag

emotion gstreamer1: Fix wrong FIXME comment

emotion gstreamer1: Don't forget to allocate metadata struct

emotion gstreamer1: Implement seekability check via seeking query