evas_font_query: fixed the wrong arrangement when white-char is on wrapping point

Current textblock does not have proper logic for processing line wrapping when -
1. white-char is on wrapping point
2. white-char is drawn with different font from the characters which are before and after it

So, this change would follow the old logic which assumes the font for white-character is
always cached by the current processing font.

After making proper changes in textblock, then this patch can be reverted.

@tizen_fix

Change-Id: I18f782402dafe64358c9094566ce41f96b8cbb84

canvas vg: Correct order of data free.

vfd data is firstly initialized and
must be freed in the last.

Change-Id: Ife3294933b601bfb7a293b409082a1282215a6d7

[atspi] Propagation of app_gesture_support attribute from content to base layout of naviframe_item

Change-Id: I65ba479fc0674dd918012a51096293f7f988a17e

canvas clip: replace hash to cached array.

Each child_map_across_mark() takes 0.026ms with eina_hash
we can reduce the performing time by replacing the hash to array to 0.0041ms in a device.

Before Average: 0.026ms
Mostly it takes 23 times in the scrolling. thus it takes 3%(0.48ms) in one frame rendering (16ms)

After Average: 0.0041ms
Mostly it takes 23 times in the scrolling. thus it takes 0.6%(0.096ms) in one frame rendering (16ms)
result: map calc 5x faster than original (0.384ms improved)

Thanks Subhransu for report this.

@optimization

Change-Id: Iba6d9addd0d092214dda4dd3f60e90b0aec1b8a7

elm/cnp: remove const from static function return

this makes no sense

../src/lib/elementary/elm_cnp.c:54:15: warning: type qualifiers ignored on function return type [-Wignored-qualifiers]
   54 | static inline const Elm_Sel_Format

Reviewed-by: Christopher Michael <devilhorns@comcast.net>
Differential Revision: https://phab.enlightenment.org/D11759

doxygen docs: Fix most errors on Evas_Legacy.h

Oh boy... References to Unified API, horribly misshapen groups,
typos, invalid commands, invalid references, and on top of that,
Evas VG.

doxygen docs: Put Evas_Canvas legacy methods in the right group

"Evas_Group" does not exist, and "Evas" is a top-level group without content
(only subgroups). So I guess all these methods belong to the "Evas_Canvas" group.
This brings even more docs into existence, fixing multiple missing references.

doxygen docs: Fix invalid refs to Layout

All widgets copy&pasted the same description, including an invalid reference
to "Layout", which should be "Elm_Layout".
SOME of them had been fixed over the years but this commit fixes all of them.
This significantly reduces the number of Doxygen warnings and adds meaningful
links to the docs.

doxygen docs: Add missing Elementary group

Elementary has a main page (elementary_main) like other libs (Ecore, Evas...)
but it was missing a doxygen group like the other libs. This group is not
terribly useful but it is manually referenced from dozens of places, resulting
in lots of warnings.
I thought this solution is simpler than removing all the manual references.

doxygen docs: Match doc with correct enum for Render_Op

Docs did not match the intended enum value due to incorrect usage of the
doxygen tags.
Also, Evas_Render_Op was the only member of the top-level Evas group, so
I moved it inside one subgroup, where methods using it reside.

doxygen docs: Put APIs in correct group

"Evas_Object" does not exist as a group, so the docs for these 29 methods
did not show in the output.

doxygen docs: Document compositor sync API as deprecated

This is currently a NOP and we should remove this API later on.
Meanwhile, turn the note about this API being experimental into
a deprecation warning.

elm_spinner: list spinner signal clean up to following Common GUI.

Change-Id: If69ed24820c4a5d39cc8d5667f3050ebd16e5176

efl_ui_image: Fix unnecessary unload of memfile_set.

Summary:
In D11392, fixed to call file_unload when doing memfile_set.
Therefore, "load,error" callback was called while unloading object
that was not loaded when first load with memfile_set.
So fix it to avoid unnecessary unloading.

Test Plan:
{F3871804}
cp ./test_imge.c ./efl/
cd efl
gcc -o test_image test_image.c -g `pkg-config --cflags --libs elementary`

./test_image
Loading image...
Async file load failed.
Image is ready to show.

Reviewers: Hermet, kimcinoo, herb

Reviewed By: Hermet

Subscribers: cedric, #reviewers, #committers, zmike

Tags: #efl

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

Change-Id: I5d30e5b35c08481cb2eaa7eb4a45d07e49433082

canvas gl: fix nutty code of partial rendering toggle.

clean up partial rendering code for maintaining
and makei it work properly while toggling.

Change-Id: I8827c2526397fb73c961a0f4449f2bacd6525418

elm - combobox - pretty borken but make it a little less

add some tests so i can see smaller comboboxes with padding and fix
the scaling which was broken.

eolian: require opaque structs to be used by ref always

For now only in stable API.

efreet - work around conconforming apps putting startupwmclass in quotes

riot does

StartupWMClass="Riot"

https://specifications.freedesktop.org/desktop-entry-spec/desktop-entry-spec-latest.html

does not allow for quoting here. you can quote fields in an exec line
- for shell and replacement purposes, but not other fields like this
above. this is a workaround a broken app so matching window to desktop
file works.

@fix

evas gl - fix coverity complaint in shader cache update

evas gl - shader cache was needless losing and overwriting

we were losing cached chaders and overwriting the cache all the time
when apps finished compiling new shaders and at other points... when
they should have already had a populated shader cache that was all
fine and happy. this fixes that so once the cache is fully populated
it wont write anymore and it wont lose existing shaders in the cache

@fix

evas_gl_context: reduce conditions for image orientation check

Reviewers: woohyun, eunue, kimcinoo, Hermet

Reviewed By: Hermet

Subscribers: kimcinoo, cedric, #reviewers, #committers

Tags: #efl

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

canvas layer: +prevent null layer objects.

Change-Id: I7a822d314ee16e984db25aa0180b61affa0cdd8c

canvas layer: find the top item for only valids.

In some case, top layer doesn't have valid objects,
That case, we must find and return top object in the next layer.

Change-Id: I3c6fc8a683a1b3673a06f8e544a04809ed3a353e

elm_spinner: Implement Tizen_6.0 Common profile GUI.

Add internal API to enable scroll item align feature only in list spinner.
Enable scroll item align and valign in list spinner.
Spinner will sending signal to genlist for item font change.

Change-Id: I8b3d6fe38ce14e73502846df29b50dd09b707d2a

evas_gl_common: Add env "EVAS_GL_TEXTURE_ATLAS_DISABLE" to control texture atlas for image upload

To disable texture atlas, calling setenv before elm_win_add
*This patch was added for a fix issue related to atlas image of product code.(tizen_4.0)

Change-Id: I422433c05244b785c96e7a9d52469cde7189cb83

exactness: check action for NULL

If the given events list is NULL the data pointer would be as well. Make
sure we check for NULL here before accessing.

CID: 1419843

Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D11754

exactness: use convinience ecore_file function to get dir

Instead of doing our own parsing here we should use ecore_file_dir_get()
which uses dirname() already to solve this.

CID: 1422196

Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D11727

exactness: do not access potential NULL pointer

These to code lines should be in one block and not one exectued without
the if.

CID: 1422198

Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D11726

Revert "meson: do not install .eo files of not installed header files"

This reverts commit d4946a0d072ff301026e6c56e095a87305def1a0.

install these - e builds against efl will fail if they try use
efl-canvas-wl without headers... :)

EFl_Canvas_Text: add typedef for backward compatibility in legacy headers

Reviewed-by: Carsten Haitzler (Rasterman) <raster@rasterman.com>
Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org>
Differential Revision: https://phab.enlightenment.org/D11744

ecore_file - address issue about false positive of finding a file in ./

it may report an exe is installed if it happens to be in cwd but cwd
isnt in path. the "full/relative" path handling case mishandled this.
fixes T8678

@fix

[atspi] _accessible_reading_material_get: Added Efl.Access.Selection interface check before calling Selection method

Change-Id: I32852473f98445e7c2af177d74632ce077b45e0a

genlist : fix direction calc for adjust align get

Change-Id: I8aeb9067089c83327b583574dcc6f25073e38a61
Signed-off-by: SangHyeon Jade Lee <sh10233.lee@samsung.com>
(cherry picked from commit dc3fa9f66766fbb008e327f837268b31136317b6)

elm_atspi_bridge: fix unregister object usage

All objects paths in accessible_reading_material_get
method should be registered before return them to user.

Change-Id: Idd5f824cc25ef083a0ec490e4fdfe911c3e11373

elm_entry: do not set SENSITIVE_DATA in INPUT_PANEL_LAYOUT_PASSWORD

If input_hint is set to SENSITIVE_DATA in elm_entry_input_panel_layout_set,
Even when the password mode of elm_entry is false,
some functions cause the problem to behave like password mode.
This patch makes the entry's password mode operation depend only on the elm_entry_password_set API.

Change-Id: Ie1a9efc13fc34cf09212209d0c6459e3221ca541
Signed-off-by: Bowon Ryu <bowon.ryu@samsung.com>

ecore_wl2: EFL Wayland Build on FreeBSD

This enables a user to build EFL with wayland support enabled
on FreeBSD. It is NOT functioning, but everything starts at
some point.

This requires also linking against -lepoll-shim.
Meson arguments:
-Deeze=false -Dv4l2=false -Dfb=false -Ddrm=false -Dwl=true \
-Dsystemd=false

@fix T8659

eolian: clarify @move tag usage in documentation

elementary: fix string definition

eolian is note made to have ptr(char) be the same as string.

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

eina: add new definitions to a old enum

this is sadly needed due to the fact that we have a .eo defnition in the
.eot file which contains the flags keyword.

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

elementary: these definitions are in real header files

there is no reason to have them in the .eot files. They are also not
namespaced in the way we want to have types.

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

evas: remove dead struct definition

this is not used anywhere

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

eo: install eot files correctly

eina_types.eot is needed in order to fetch all needed informations.

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

elementary: do not install legacy .eo files

they do not have any purpose to be installed on the filesystem.

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

meson: do not install .eo files of not installed header files

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

release: Update NEWS and bump version for 1.24.0-beta3 release

evas_object_textblock: add null check for return of eina_inlist_remove

ecore-drm2: Change ERR to WRN

No real need for these to be ERR because we are not going to crash if
these happen anyway.

ecore-evas-wayland: Fix formatting

NB: No functional changes

marquee: add stack above between p_obj and textblock.

Stacks p_obj for marquee animation on the existing textblock.
Adjust the z-index with this.
This prevents other ui elements from being placed between the textblock and p_obj.

Change-Id: I2f71ef66233938b3c758eac5c1b3548c5d107d42
Signed-off-by: Bowon Ryu <bowon.ryu@samsung.com>

eolian: fix C type serialization of error()

eolian: only warn instead of erroring on unused imports

exactness-inspect: do not use sprintf

there is not yet a CID for this, but we should probebly not do that.

Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org>
Reviewed-by: Xavi Artigas <xavierartigas@yahoo.es>
Differential Revision: https://phab.enlightenment.org/D11731

exactness-inspect: remove abscure xor check

this is a obscure check, if this was ever reached, then it would simply
crash, because one pointer will be NULL, but strcmp will be called on
it.

CID 1419854

Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org>
Differential Revision: https://phab.enlightenment.org/D11722

doxygen docs: Fix all invalid @param names

There were quite a few of these...

exactness: make sure we handle a negative file descriptor

In an error case the fd could be negative here and we should check
before feeding it into fdopen(). This is the same patch we used in
recorder.c

CID: 1422194

Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D11728

exactness: make sure we handle a negative file descriptor

In an error case the fd could be negative here and we should check
before feeding it into fdopen().

CID: 1422197

Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D11725

exactness: handle case where eet_data_image_write() fails

We never checked how many bytes had been written. Check on return and
propagate error upwards to caller.

CID: 1419856

Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D11724

exactness: remove unused code

We no longer support the old .rec format and we can always expect the
file to be exu. Coverity found this block to be always true so the else
part could not be reached.

CID: 1421996

Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D11723

elm theme - add support for smooth restart for e

just extra data item and extra program only used if it's a new e

canvas svg: fix memory leak.

free svg node commands & points data after usage.

@fix

Change-Id: I598b93a083699e032c12f53b01dda1b6573d2b62

elm_spinner: Prevent to create inc/dec button in list spinner.

There is no inc/dec button in list spinner UX

@tizen_feature

Change-Id: Ic4eff7193f2b782c29d27037164c002f06def200

elm - theme - add outline style for frames

new standard style. outline.

eolian: globally enable eolian_state_check, except for beta classes

Beta classes won't have their namespaces validated at this point.
It is possible to set EOLIAN_CHECK_NAMESPACES_BETA=1 to enable
checking those as well, if you want to fix them all.

remove unused imports in eo/eot files

eolian: add composites into hard deps when checking

This prevents composites from being considered unused imports.

evas_textblock/evas_text/evas_textgrid: pick textrun fonts

Summary:
Picking font on textrun, will now give priority into font picked by the user, regardless of script type.
picking font due script can cause many inconvenient results

Example of wrong results:  (User font is **NotoColorEmoji**)
{F3847118} -> add 'a' at the end (notice how text render is wrong) {F3847119} -> add tab before 'a' (text rendering now is right) {F3847120}

After Change results: (User font is **NotoColorEmoji**)
{F3847118}  -> add 'a' at the end -> {F3847122}-> add tab before 'a' -> {F3847123}

Also now the following lines will be shown exactly the same, regardless of characters order
```
"가123A321"
"A321가123"
"123가A321"
"A가123321"
```

Test Plan:
```
#include <Elementary.h>
/*
gcc -o example test.c `pkg-config --cflags --libs elementary`
*/

EAPI_MAIN int
elm_main(int argc EINA_UNUSED, char **argv EINA_UNUSED)
{
   Evas_Object *win, *en;

   elm_policy_set(ELM_POLICY_QUIT, ELM_POLICY_QUIT_LAST_WINDOW_CLOSED);

   win = elm_win_util_standard_add("", "");
   elm_win_autodel_set(win, EINA_TRUE);

   en = elm_entry_add(win);
   elm_entry_scrollable_set(en, EINA_TRUE);
   evas_object_size_hint_weight_set(en, EVAS_HINT_EXPAND, EVAS_HINT_EXPAND);
   evas_object_size_hint_align_set(en, EVAS_HINT_FILL, EVAS_HINT_FILL);

   elm_entry_text_style_user_push(en,"DEFAULT='font=NotoColorEmoji font_size=30 color=red'");
   elm_object_text_set(en, "&#x262a;123456a");

   evas_object_show(en);

   elm_object_content_set(win, en);
   evas_object_resize(win, 400, 200);
   evas_object_show(win);

   elm_run();

   return 0;
}
ELM_MAIN()
```

Reviewers: woohyun, bowonryu, tasn, raster, cedric

Reviewed By: tasn

Subscribers: cedric, #reviewers, #committers

Tags: #efl

Maniphest Tasks: T8556

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

evas_textgrid: update text props to use current script font

Summary:
This update make the textgrid same as evas_text and evas_textblock to use current picked font for text_props instead of script_font

This patch will fix terminology emoji rendering when used applying D11302.

Reviewers: raster, woohyun

Reviewed By: woohyun

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

evas render - flush, dont clear unref/clean arrays on  render done

we build these arrays to hold tjhings to clean up when done ... but if
one frame happens ot have a lot - the allocation stays huge as we only
clean not flush, so flush to keep memory down.

@optimize

[Genlist] Unfreeze event fix
Unfreeze event is not occuring properly. If item is on deletion genlist remains in freezed state Unfreeze genlist regardless of deletion state

Change-Id: I50d517edb85811ea6c367f2d22d80f5759bce326
Signed-off-by: Suresh Kumar <suresh.ku1@samsung.com>

ecore_anim: fix animator ticking logic to gain accuarte

ecore_loop_time has set by a improper timestamp just before animator callback is invoked.

ecore_animator is built over ecore_thread. ticking is handled in 'thread_func'
and a tick is fired by calling ecore_thread_feedback with a timestamp argument.

In notify func, ecore_time_loop_set has take place with the argument from thread func.
the argument could be unacceptable because notify function is invoked asynchronously.
if there were many thing to do in the loop, time gap must be incresed.

animator tick leads elm logic to calculate next position in animating.
so it is neccessary to use actual time for the calculating.

this patch fixes the animator to set loop time by a actual time.

Change-Id: I1f00a5d074a78da5a87fba55bc1ab13a4f1311df

ecore_wl: To use vsync as custom source of animator

This patch includes below patches
ecore: fixed wrong codes regarding ecore_anim_vsync
ecore: Remove condition about timeout in animator with vsync
ecore_anim : Add safety code to ecore_anim
ecore_anim: Fix don't call animator_cb
ecore_wayland: Remove codes irrelevant to poll()

@tizen_only

Change-Id: I2223cc9b4574d17b99f91c5b7e6c8587c9e7852f
Signed-off-by: Hosang Kim <hosang12.kim@samsung.com>

Add debug information to ecore_wl2

- add debug information which for analyzing of many issues related to touch event.
Sometimes we need more information regarding touch events of ecore_wl2.

Change-Id: I0c495c2b25fef7a6062febeddd43f7753a67e406
Signed-off-by: ch.jeong47 <ch.jeong47@samsung.com>

spinner: flick to open feature

Change-Id: Ice51bbca22d07552193e3c10bb1a22c33481e365
Signed-off-by: HyukSoon Choi <hs619.choi@samsung.com>

edje : add new feature of color class can be aliased by item data in the group

search object color_class can be aliased different name by looking into
item.data of group style.

data {
   item: "color/aliased_color_class_name" "original_color_class_name"
}

@tizen_only

Change-Id: Ic94f51d3066d8942ad86f6f3ef91de29917be9b7

[Genlist] Avoid unnecessary refcount in item
Item is not getting deleted. Refcount was not becoming < 1.Unref the item for allowing item to be deleted

Change-Id: Iaf8b71bb06c7c6f5a3e70fc89796e815e5336424
Signed-off-by: Suresh Kumar <suresh.ku1@samsung.com>

[Genlist] Contribution of item free optimization on clear

Change-Id: Id229f881b45dfc27ecf4f7bda8b22648c0e4d2c7
Signed-off-by: Suresh Kumar <suresh.ku1@samsung.com>

[Genlist] Avoid non alignable items being aligned during recalculations

Change-Id: Iec2208485f4b5e62e954270eb2c5f78c08637161
Signed-off-by: Yukti Pandey <yukti.pandey@samsung.com>

[Genlist] Safety checks to avoid crashes

Change-Id: Ic951e1382b7e82488b82a2cec042bb0ab50b026c
Signed-off-by: Yukti Pandey <yukti.pandey@samsung.com>

Revert "evas_textblock: pick textrun fonts"

This reverts commit 9b987c67e1e38602ba2e4c9c990798e99d938bbe.

this breaks color emoji in terminology (evas textgrid).

exactness-inspect: unit1 can never be NULL

checkinf for it beeing NULL means that we would have to equip every
usage of unit1 with a check, but that is useless.

CID 1419859

Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org>
Differential Revision: https://phab.enlightenment.org/D11721

exactness-inspect: fix NULL access

ii might be NULL so we should ensure it is not NULL to call item_select

CID 1419865

Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org>
Differential Revision: https://phab.enlightenment.org/D11720

exactness-inspect: sanitize if clause

these if clause where a bit bottom up, and the xor operation here seemed
totally wrong, with this code we are simply displaying both entiteis of
the two structs when they are there. *or* we are replacing it with the
fallback.

CID1419875

Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org>
Differential Revision: https://phab.enlightenment.org/D11719

elm - genlist test - handl g_data being null

fix CID 1396944 1396946

ecore - examples - fix gnutls fd handler example warnings

just error returns not being checked. not an issue really, but this
silences warnings.

fix CID 1164185 715896 98094 1400818

ecore - proc env - fix values return NULL case - coverity

this fixes CID 1419857

evas cache - up padding just in case i was off by one

coverity was complaining but it's a false positive - it seems to not
realze the loadopts_append does nul terminate at the end.

tests/elm: add image tests to verify internal object size is clamped to object size

ensure that this doesn't break again

ref 11587

Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org>
Differential Revision: https://phab.enlightenment.org/D11599

canvas svg: fix memory leak.

free svg node commands & points data after usage.

@fix

evas_textblock: prevent segfault if there are no style

```
#include <Elementary.h>

EAPI_MAIN int
elm_main(int argc EINA_UNUSED, char **argv EINA_UNUSED)
{
   Evas_Object *win, *textblock;

   elm_policy_set(ELM_POLICY_QUIT, ELM_POLICY_QUIT_LAST_WINDOW_CLOSED);

   win = elm_win_util_standard_add("", "");
   elm_win_autodel_set(win, EINA_TRUE);

   textblock = evas_object_textblock_add(win);
   evas_object_resize(textblock, 360, 360);
   evas_object_show(textblock);
   evas_object_textblock_text_markup_set(textblock, "<b>b</b>");
   evas_object_resize(win, 360, 360);
   evas_object_show(win);
   elm_run();

   return 0;
}
ELM_MAIN()
```

this application will crash.

Any font format specified in text, if there are now style will cause the crash.

Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com>
Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org>
Differential Revision: https://phab.enlightenment.org/D11686

eina/value: use correct time types when comparing time types

these functions all expect struct timeval, as the name implies

../src/lib/eina/eina_value.c: In function ‘_eina_value_type_tm_compare’:
../src/lib/eina/eina_value.c:3358:19: warning: array subscript 1 is outside array bounds of ‘time_t[1]’ {aka ‘long int[1]’} [-Warray-bounds]
 3358 |    struct timeval ret = *input;
      |                   ^~~
../src/lib/eina/eina_value.c:3569:11: note: while referencing ‘ta’
 3569 |    time_t ta, tb;
      |           ^~
../src/lib/eina/eina_value.c:3358:19: warning: array subscript 1 is outside array bounds of ‘time_t[1]’ {aka ‘long int[1]’} [-Warray-bounds]
 3358 |    struct timeval ret = *input;
      |                   ^~~
../src/lib/eina/eina_value.c:3569:15: note: while referencing ‘tb’
 3569 |    time_t ta, tb;

Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org>
Differential Revision: https://phab.enlightenment.org/D11715

eina_content: free eina content in the test:

Fix the unbelievable High prioritized CID 1420314

elm/win: declare prototype for ecore_evas_dnd_mark_motion_used

../src/lib/elementary/efl_ui_win.c: In function ‘_motion_cb’:
../src/lib/elementary/efl_ui_win.c:9271:14: warning: implicit declaration of function ‘ecore_evas_dnd_mark_motion_used’ [-Wimplicit-function-declaration]
 9271 |              ecore_evas_dnd_mark_motion_used(ee, seat);
      |              ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

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

exactness: limit length for font_path to stay in buffer

We get fonts_dir from a getenv() without and length check. Make sure
that we stay in the given buffer size when stitching the file path
together.

CID: 1422195

Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D11718

exactness: fix typo in function name

Simple spelling mistake. overlay instead of overloay.

Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com>
Differential Revision: https://phab.enlightenment.org/D11714

exactness: remove no longer used code since move to preload

This has been around for prg handling before we switched to preload. No
need for it anymore. Found when looking for a Coverity issue, which also
got fixed now by luck. :-)

CID: 1421994

Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com>
Differential Revision: https://phab.enlightenment.org/D11713

evas_textblock: remove special treat for variation sequence text run

Summary:
Now there are no need for special care for variation sequence text run break.
General method will care about all types of characters\

this is also related to D11096

Reviewers: woohyun, bowonryu

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

evas_textblock: pick textrun fonts

Summary:
Picking font on textrun, will now give priority into font picked by the user, regardless of script type.
picking font due script can cause many inconvenient results

Example of wrong results:  (User font is **NotoColorEmoji**)
{F3847118} -> add 'a' at the end (notice how text render is wrong) {F3847119} -> add tab before 'a' (text rendering now is right) {F3847120}

After Change results: (User font is **NotoColorEmoji**)
{F3847118}  -> add 'a' at the end -> {F3847122}-> add tab before 'a' -> {F3847123}

Also now the following lines will be shown exactly the same, regardless of characters order
```
"가123A321"
"A321가123"
"123가A321"
"A가123321"
```

Test Plan:
```
/*
gcc -o example test.c `pkg-config --cflags --libs elementary`
*/

EAPI_MAIN int
elm_main(int argc EINA_UNUSED, char **argv EINA_UNUSED)
{
   Evas_Object *win, *en;

   elm_policy_set(ELM_POLICY_QUIT, ELM_POLICY_QUIT_LAST_WINDOW_CLOSED);

   win = elm_win_util_standard_add("", "");
   elm_win_autodel_set(win, EINA_TRUE);

   en = elm_entry_add(win);
   elm_entry_scrollable_set(en, EINA_TRUE);
   evas_object_size_hint_weight_set(en, EVAS_HINT_EXPAND, EVAS_HINT_EXPAND);
   evas_object_size_hint_align_set(en, EVAS_HINT_FILL, EVAS_HINT_FILL);

   elm_entry_text_style_user_push(en,"DEFAULT='font=NotoColorEmoji font_size=30 color=red'");
   elm_object_text_set(en, "&#x262a;123456a");

   evas_object_show(en);

   elm_object_content_set(win, en);
   evas_object_resize(win, 400, 200);
   evas_object_show(win);

   elm_run();

   return 0;
}
ELM_MAIN()
```

Reviewers: woohyun, bowonryu, tasn, raster, cedric

Reviewed By: tasn

Subscribers: cedric, #reviewers, #committers

Tags: #efl

Maniphest Tasks: T8556

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

evas_test: change START_TEST to EFL_START_TEST

Summary: evas_test: change START_TEST to EFL_START_TEST

Reviewers: zmike

Reviewed By: zmike

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

ecore_evas: fix fallback code

this was using a stringshare reference that was deleted. While that is
true, the stringshare reference will always be alive, because 2 people
took a reference. Anyways, this code is now searching the other way
arround, which makes the code also easier.

fixes: CID1420331

Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org>
Differential Revision: https://phab.enlightenment.org/D11712

tests/elm: explicitly destroy hoversel while active

verify that there's no errors when this happens

Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org>
Differential Revision: https://phab.enlightenment.org/D11704