[Evas GL Common] remove commented code.

Remove commmented code in evas_gl_context.c
Consider that for EFL version upgrade.

Change-Id: Id748a77d8a3e19fcd938ae329690caba9f3ca5aa

ecore_wl_input: fix key repeat logic.

In Tizen, Unfocused App can receive key event.
So repeat timer should not be deleted,
when _ecore_wl_input_cb_keyboard_leave is called.

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

[tbm_backend] implement tbm backend (gl_tbm and software_tbm)

1. dumy engine for Create Tbm backend
2. [Ecore_Evas_TBM] Add ecore_evas_tbm and evas_tbm_backend
3. [tbm_backend] remove bufmgr
  - remove bufmgr in tbm backend
  - redesign ext API
4. [tbm_backend] implement software tbm backend
5. add acquire/release pixel api & delete image fuction
6. [tbm_backend] implement tbm backend (gl_tbm and software_tbm)

Change-Id: If141e2d85e33cd5257a51108338c7afa8f0e3994

edje entry: return correct selection

Summary:
In entry, when selection_get function is called, selection is not
always returned the current selection.

Scenario:
- In select mode, entry has selection (e.g, by double click).
- When selection handler is moved, set the cursor the current coordinate
  with edje_object_part_text_cursor_pos_set() API.
- Edje emits "selection,changed" signal.
- Elementary gets current selection and stores it.
- Elementary does not get selection as visual selection (e.g, text is
   highlighted with "entry test", but the selection returned
   from edje is "entry").
- If we copy and then paste to the entry, the pasted text is not same as
   selected text.

Reason:
- In _edje_entry_cursor_coord_set function, if entry has selection, we only
  emit "selection,changed" signal without freeing en->selection.
- When _edje_entry_selection_get is called, we check the en->selection,
  since it is existed, we just return it which is not updated one.

This patch clears en->selection, so that it is updated at _selection_get,
and the updated selection is returned to caller.

@fix

Test Plan:
In mobile profile, open entry
- Right click, choose select, double click -> selection handlers are shown.
- Drag selection handlers to change selection.
- Right click, do copy.
- Right click, do paste.
- See the pasted text is not same as selection.

Reviewers: raster, tasn, herdsman

Subscribers: seoz, JackDanielZ, cedric

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

Change-Id: I62e7bd9c8af9ae9048a3ff2d4f4c6912e1b33692
Signed-off-by: Wonki Kim <wonki_.kim@samsung.com>

ecore_wl_input: Add option for key conversion

Signed-off-by: NakGyeong Kim <nakkyong.kim@samsung.com>
Change-Id: Idad899845c4d90f4f2efbbc3682f10085cdc76f8

ecore_wl_input: add last_device_name_kbd for repeating key events

Signed-off-by: Duna Oh <duna.oh@samsung.com>
Change-Id: I0c15587e930ac4c5c66354a6343c5d72368c2656

[evas] bugfix - color format of native tbm surface
       - sdk camera test
       - If a other flag setup before the SHD_EXTERNAL
         then color format problem happens

Change-Id: Id31de742e2a7527486c3816d2b6153c8fede12af

efreet : Remove launching efreed on Tizen

efreetd is used for freedesktop environment
But, Tizen is consisted of another environment
 and desktop managing(based on freedesktop standard) does not needed to efreetd

Therefore, efreetd can be removed on Tizen.
But, libefreet is still left beause a few functions use libefreet in elementary.

- src/lib/efreet/efreet_cache.c: skip daemon launching sequence in efreet_cache_init()
- packaging/efl.spec: exclude installing efreed

@tizen_fix

Change-Id: I50f892e7121ae33532e8138d4dfc52741ca38986
Signed-off-by: Jiwon Kim <jiwon177.kim@samsung.com>

Change a xkb state if chached keymap is changed

Change-Id: I94e04c79dd99db6fcdaa3a943aaa7e0bf1e76c0d

ecore_wayland: added the utility window type

Change-Id: I7472ba323f4492ccbf49bf470c470e9a4dc70817

ecore_wl_input: fix key repeat condition.

Key repeat will be operated when window is unfocused.

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

ecore_wl_input: Fix memory leaks

Change-Id: Ideff36a5923afb7d7df68573c46f85b8b159a880

ecore_wl : add wl_log_set_handler_client

Change-Id: I7e4d26db0181153d5cffcadaf90ca2a4d6d5d154
Signed-off-by: Junghwan Choi <jhhh.choi@samsung.com>

ecore_drm: do not use eeze udev for output and make output list only main output

Change-Id: Icff557766a616c69739c7ac8d26925404a7c019e
Signed-off-by: Junkyeong Kim <jk0430.kim@samsung.com>

efl_gfx_shape: Improve coordinate cacluation of vector graphic

Improvements have been already applied in Tizen 2.4 to resolve
the problem that the renderer don't draw a circle properly,
when a radius of the circle is odd.
Then this commit will resolve the same problem in Tizen 3.0.

Change-Id: Ibf856375c97ea811d4946fad0a1fad6d1f818451
Signed-off-by: Wonki Kim <wonki_.kim@samsung.com>

[evas] use EVAS_GL_WAYLAND_Y_INVERTED_WL attribute of
eglQueryWaylandBufferWL

 - 'yinvert=0' in YaGL of emulator, so we need check
EGL_WAYLAND_Y_INVERTED_WL attribute to query orientation of
   wl_buffer

Change-Id: I98abbe72c882d83a4bf90081f5feb2462027fb7c

 [Evas] Avoid setting AFILL flag for gl_drm based surface

Change-Id: Ifa79de5893e7b63fcdae5e4eebd24b511ccd5d21
Signed-off-by: Prudhvi Raj Vasireddi <prudhvi.raj@samsung.com>

make input_region into NULL after destroyed

Signed-off-by: Seungjin Park <sj8808.park@samsung.com>
Change-Id: I23e31a690ee7b773e38bbd55fe94a8b8d97b43a7

Call a release request if a device is removed

Change-Id: I00ab477fdebb799a3554daee3ed18544613973dc

evas_object_key_grab exclusive option logic is changed.
 - added is_active property in Evas_Key_Grab
 - Evas key down and key up events transferred to active grabs only.
 - If evas grabs has a exclusive grab, other grabs will be deactivated.
 - If a exclusive grab is ungrabbed, remained grabs will be activated.

Signed-off-by: JEONGHYUN YUN <jh0506.yun@samsung.com>
Change-Id: I36ff752cc86262a0cfeb92d4f849bb3ae5b881d4

set pending_flip before calling tdm_output_commit

    The commit handler can be called in tdm_output_commit.

Change-Id: I1de00b2877d08bad0456463ea28acf02b047cf57

evas_device: fix build warning - use parentheses around assignment used as truth value

Signed-off-by: Duna Oh <duna.oh@samsung.com>
Change-Id: I674d8c437c053782cc4d9170851bb8e58408a7c1

ecore_wl_input: fix build warning of incompatible pointer type

Signed-off-by: Duna Oh <duna.oh@samsung.com>
Change-Id: Ie079a31786a675e530da900da543154fbc975890

ecore_wl_input: use release as a destructor since wl_seat ver. is upgraded to 4

Signed-off-by: Duna Oh <duna.oh@samsung.com>
Change-Id: I8f23ac19cba976bfb83b578ccac569b098cd8d3f

wayland egl : Remove build warning for unused variable

   Remove unused variable. and EINA_UNUSED macro was inserted after unused parameter.

Change-Id: Ic0ee6f5e0c62a87b0b8eeb1d539b7676b6ce51ff

evas_engines_gl_drm : Remove build warning

   - Remove unused variable.
   - Some variables were type casted into void* for arithmetic warning.
   - Changed type of 'last' variable type in _evas_outbuf_cb_pageflip() function.
     Because, That is only used as 'struct gbm_bo' type.

Change-Id: Ic660edb8d5d1ea096bd3830736f57db85ed2b105

evas_engines_drm : Remove build warning for unused parameter

   EINA_UNUSED macro was inserted after unused parameter.

Change-Id: Ice5b9222dc7c566c78f8e27e5befabbc52c8d678

evas_object_vg : Fix build warning for uninitialized variable.

initialized two variable

Change-Id: I9d1d48fbae1f1df3c6ed9de38b3b8ef36d5a59d1

ecore_drm : Fix build warning for unused parameter and function.

Instead of some unused parameters (void)variable_name line
and remove unused function.

Change-Id: Iddf588aa35c7284539de1a88d27b717094b010bb

Ecore Wayland : Remove build warning

   Remove build warning from ecore_wl_input.c, _window.c
   I did remove unused variable and add EINA_UNUSED for unused parameter.
   Also, I added type casting for 'uintptr' type

Change-Id: I8a53efbd1712debc3331627d8e2da4bcb4176c0c

ecore_input_evas : Remove build warning for unused parameter

   EINA_UNUSED macro was inserted after unused parameter.

Change-Id: Id8ba8ec955f6dc4e9f7de89aa736d9d6dbdb5c60

[evas] bugfix - color format of native tbm surface

 there is color format bug in camera test, so change shader code
 and DDK modification is necessary

Change-Id: I547199174aa063453baefaa1c4b48ff8926f6b79

ecore-wayland: fixed wrong variable for NULL check

Change-Id: I8d01cfbc0e331951c3e84077d0708eb9a5e3d71c

[Event] Fix worng data type calc in keyboard_repeat_setup.

  - repeat's member is double type. but input data is int type.

Change-Id: I6eda2f8530aa9df8149342f8fa204369a04cbb8b

Evas Device: fix segv in evas_device_cleanup

Signed-off-by: Duna Oh <duna.oh@samsung.com>
Change-Id: I4194399a60ee9c99b9c884d8559dcb29608c4127

Merge "ecore_wl: Considering screen height size, when calc dpi." into tizen

Merge "[wayland_egl] delete evas_engine_info_set in ecore_evas_wl_hide" into tizen

ecore_wl: Considering screen height size, when calc dpi.

Change-Id: I7ba6b5961d23369c443f62c2c4d73645e32a9c5a

ecore_wayland: fixed resource leak

Change-Id: I6966ec74582b710ea3b4f7f0c3f60cc03af998ec

ecore_evas: remove build warning

Change-Id: I2d703a88cc13ac53c9b4ffaf6742bd8ed4939add

Evas Device: use EINA_LIST_FOREACH_SAFE instead of EINA_LIST_FREE

evas_device_del does eina_list_remove, and EINA_LIST_FREE does the same thing.
It can causes memory leak and possible crash. (reported by valgrind)

Signed-off-by: Duna Oh <duna.oh@samsung.com>
Change-Id: I2166f1eb4672ffe65dd961d608557018ade1cae0

Merge "ecore_imf: Add multiline hint in input hint" into tizen

ecore_wl_input: add logic for waiting until "wl input" is ready

Sometimes wl input is not ready, because it is async process to create
wl input.
Signed-off-by: Hosang Kim <hosang12.kim@samsung.com>
Change-Id: I8869fb51f178425830dc1ff3825c60e2b7920b68

ecore_imf: Add multiline hint in input hint

@feature

Change-Id: I6d20ba7315fed8d17be212a1c03f63ce674afce2

Merge "Evas text: fix RTL text ellipsis issues" into tizen

Merge "Evas text: Remove a whitespcae from the right end of ellipsized RTL text" into tizen

Merge "Evas textblock: Calculate exact width for current item" into tizen

Merge "Evas textblock: fix ellipsis item at wrong place in RTL text" into tizen

Merge "Evas textblock: Apply scale factor to <linesize>, <linegap> formats" into tizen

ecore_drm: destroy hal_output

Change-Id: I0ab342965d86929b2bbfc00fb379d6918c3b39af

ecore_wayland: use configured width/height to send rotation event

It's necessary to store configured width/height values to rotation event
structure instead of allocation width/height to prevent overriding latest
configured size by allocation size.

Change-Id: I0b5a7f577a105cb107e3300af19ee0d91b56476f
Signed-off-by: MinJeong Kim <minjjj.kim@samsung.com>

ecore_wl_input: increase wl_seat version from 1 to 4.

To register "repeat info" listner. Because it is supported since version
4.0.

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

Evas text: fix RTL text ellipsis issues

Summary:
Visual position of ellipsis item should be set according to
its bidi direction. But, by setting visual position in same way
as logical position, the end ellipsis could be put opposite side.
Also, start ellipsis must placed on left side of RTL text.
@fix T3187

Test Plan: Test an sample on T3187

Reviewers: tasn, herdsman, woohyun

Subscribers: cedric, jpeg, minudf

Maniphest Tasks: T3187

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

@tizen_fix

Change-Id: If78b6becb5059052ddae23f7dee39d91ddce38c7

Evas text: Remove a whitespcae from the right end of ellipsized RTL text

@tizen_fix

Change-Id: I89d2e658e986f9fa569608562e494715701ae264

Evas textblock: Calculate exact width for current item

@tizen_fix

Change-Id: I19af0791b709637369f327811f8b1471eb486468

Evas textblock: fix ellipsis item at wrong place in RTL text

Summary:
An ellipsis item could be placed in wrong place in RTL text
with separated and short text items. Normally, an ellipsis
item has to be behind(in logical position) of previous item.
But, when the length of previos text item is 1, ellipsis
item could have same position with the previous item.
It was working fine for LTR text. But, wrong ellipsis could
be shown in RTL text.
@fix

Test Plan:
The test case included.
Run "make check"

Reviewers: tasn, minudf, woohyun, herdsman

Subscribers: Blackmole, z-wony, anand.km, cedric, jpeg

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

@tizen_fix

Change-Id: I17cee4742ca0e947b2aeb413cf4c1a6b89a15de7

Evas textblock: Apply scale factor to <linesize>, <linegap> formats

Summary:
Font size is scaled according to scale factor.
The linesize, linegap formats also have to be scaled properly.
@fix

Test Plan:
Test cases are included.
Run "make check"

Reviewers: woohyun, Jieun, tasn, herdsman

Reviewed By: tasn

Subscribers: cedric, jpeg

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

Change-Id: I0845c013a734f0feffa203689d404de4edecd898

edje: fix typo affecting our 32bits signal matching code.

Big thanks to Maxim who did a serious digging in this issue making it almost
a patch review.

T3125

@fix

upstream commit: 61f7236a62db2c932c82bc68fb1ac8082514831f
Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>
Change-Id: Id4fc301cf5e700914073bd1c3447b1281844ae49

ecore_evas, ecore_wayland : enable pointer warp

Change-Id: I23b15cfc87ec50bbc01422e3a9a8e3c615f94a7c
Signed-off-by: Sung-Jin Park <sj76.park@samsung.com>

ecore_wayland : add tizen_input_device_manager interface listeners not to make a crash

Change-Id: Iae23d6e9d4ae6357c88f3105ff7780ed9dbed59a
Signed-off-by: Sung-Jin Park <sj76.park@samsung.com>

evas/gl-drm: fixed memory leak

Change-Id: Ib462eb8dc43a59a05ffe7e0f98e00f02e0f3056c

Merge "evas: skip 1x1 size frame" into tizen

evas: skip 1x1 size frame

Don't need to render about 1x1 size.

Change-Id: I999a5b77450ae75ad19d54e923330ec5794734ba

edje_callbacks: fix click signal emitting even though mouse event was cancelled

    even though each part want to get the signal regardless of ON_HOLD,
    click signal should not emmitted if event flag is set to ON_HOLD.
    ON_HOLD means this event is useless, so up event with ON_HOLD flag cannot source of clicked signal.

    @fix

    open_commit 27495fe6705a5e28971cffdd8e9cfcb92868a439
    Author: Ji-Youn Park <jy0703.park@samsung.com>
    Date:   Wed May 11 22:14:27 2016 +0830

Change-Id: I953268970ef1bcf74725a5e938ee37eed19d953d

[wayland_egl] delete evas_engine_info_set in ecore_evas_wl_hide

wayland spec is not define whether wl_egl_window_create() can use null surface or not.
so current tizen device does not allow to create null surface wayland window.

Change-Id: Iec3ab7833e619fe93ea90bd6cfae3d40a308a1fd

Merge "edje: fix edje RTL description in case of custom state of a part This patch is from opensource (59681c331139e4d5424e77ff1614dc47fa8ddd8e, f5c4cd2f7f51f739f5f69eb4c857ccbfd8d51b0a)" into tizen

Merge "edje: Fix issue with width calculation and padding This patch is from opensource (acab227544b7903ed6c267f022ebc389246e9ccc)" into tizen

Merge "[ecore_evas] rotation engine info set bug fixed" into tizen

[ecore_evas] rotation engine info set bug fixed

if rotation is occured twice during the output flush, it cause crush.
the reason of problem is rotaion-> engine setup cause buffer free.
so only engine setup not async render

Change-Id: I76b8c97fc32571962ddc08d0fe70a3bf0f026647
Signed-off-by: JEONGHYUN YUN <jh0506.yun@samsung.com>

Merge "edje: add error log message for missing description" into tizen

Merge "eina: make eina rectangle an allocated thread safe structure to use." into tizen

edje: add error log message for missing description

Summary:
Developer cannot notice that any description didn't applied due to missing description or typo.
This message will be helpful to make correct the application.

Reviewers: cedric, Hermet, raster

Subscribers: soohye.shin, minkyu, cedric, jpeg

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

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

edje: return default description depends on RTL when description doesn't exist

Summary:
If not found edje part description, edje just set default description in spite of RTL status.
This adds to call function for getting the correct description as RTL status.

Reviewers: raster, Hermet, cedric

Subscribers: minkyu, sju27, cedric, jpeg

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

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

Do not convert keycodes if this system does not support old tizen keys

Change-Id: I16e6b0894f9839d193255004f693ccc4f13b68c3

evas: add radius, radius_x, radius_y, pressure, and angle to mouse event structs

Evas Mouse events are used for mouse and also the first touch point.
In order not to lose touch information of first touch,
we add extended members to Evas_Event_Mouse_XXXwq structs.
The values of these members come from Ecore_Event_Mouse_XXX.

Signed-off-by: Duna Oh <duna.oh@samsung.com>
Change-Id: I66f4abdb733c1aec1b34c86b24e183f2706751ae

Merge "[2.4][edje_entry] Fix to pass proper text and cursor position to imf in case of selection" into tizen

ecore_wayland: added ecore_wl_window_stack_mode_set API

Change-Id: I99f9d8a0ea344cf3a3d7e18a62dccfa9909d31b8

edje: fix edje RTL description in case of custom state of a part
This patch is from opensource (59681c331139e4d5424e77ff1614dc47fa8ddd8e, f5c4cd2f7f51f739f5f69eb4c857ccbfd8d51b0a)

    Summary:
    In case of RTL, the "custom" state properties does not apply. It happened because we don't copy the latest src to
    dst in set_state(PART:.., "custom", 0.0); in case of dst is already populated.
    We should copy the updated src to dst whenever we set the new custom description.

    @fix

    Reviewers: cedric, raster, jpeg, zmike, jaehwan

    Subscribers: kimcinoo, seoz, jpeg

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

Signed-off-by: Minwoo, Lee <minwoo47.lee@samsung.com>
Change-Id: Icf0b73f0cf2056e8f368c4215fc8139f2b4828e0

edje: Fix issue with width calculation and padding
This patch is from opensource (acab227544b7903ed6c267f022ebc389246e9ccc)

    Summary:
    The size of the style pad isn't considered when the min value of the
    textblock is calculated.  In case of putting the lable that there is an
    outline in the box, the letter is cut.  So, I revised so that
    evas_object_textblock_style_insets_get could be called after a
    evas_object_textblock_size_formatted_get in
    _edje_object_size_min_restricted_calc function.  And then the style pad was
    considered in the result value of the edje_object_size_min_calc.

    @fix

    Test Plan:

    EAPI_MAIN int
    elm_main(int argc, char **argv)
    {
       Evas_Object *box, *label;
       Evas_Object *win;

       elm_policy_set(ELM_POLICY_QUIT, ELM_POLICY_QUIT_LAST_WINDOW_CLOSED);

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

       box = elm_box_add(win);
       elm_box_padding_set(box, 10, 0);
       elm_box_align_set(box, 1, 0.5);
       evas_object_size_hint_weight_set(box, EVAS_HINT_EXPAND, EVAS_HINT_EXPAND);
       evas_object_size_hint_align_set(box, EVAS_HINT_FILL, EVAS_HINT_FILL);
       elm_win_resize_object_add(win, box);
       evas_object_show(box);

       label = elm_label_add(box);
       elm_object_text_set(label, "<font=default align=rignt color=#ffffff font_size=200 style=soft_outline outline_color=#ff0000ff>label");
       elm_box_pack_end(box, label);
       evas_object_show(label);

       evas_object_resize(win, 500, 300);
       evas_object_show(win);
       elm_run();

       return 0;
    }
    ELM_MAIN();

Signed-off-by: Minwoo, Lee <minwoo47.lee@samsung.com>
Change-Id: I5ca56206064233698ca9802c761bf94320381dfb

eina: make eina rectangle an allocated thread safe structure to use.

This is a better fix than D3913.

Change-Id: I18a1fb7b49e1b09794298f136ed00f8b061fddd3

[2.4][edje_entry] Fix to pass proper text and cursor position to imf in case of selection

@tizen_fix

Change-Id: I62beb5ce8027d00fe3e64de5320bb02ddc2d0359
Signed-off-by: Subodh Kumar <s7158.kumar@samsung.com>

[evas-gl] destroy surface when evas-gl is direct renderable

In tizen 3.0 & upstream , evasgl create more buffer when evas gl is direct renderable status.
but according to Log,this code want destory buffer.
Note that tizen 2.4, it is already fixed.

Change-Id: I4daa76a5c1b4b1f1cc288170beaf8b84b53bc027

Merge "[evas] change rotation angle(90 and 270) because tbm is yinvert" into tizen

Merge "evas: enable dynamic flag for zero copy texture upload in ector / Bind to TEXTURE_EXTERNAL for images with dynamic hint and tbm" into tizen

edje_entry: fix crash issue in case of no selection text

Change-Id: I90362a605cdaa10a64548003852e4448ba81c984

[evas] change rotation angle(90 and 270) because tbm is yinvert

Change-Id: I95481dc1419ad5de4c6609de34e4805df791dc88

Merge "Evas textblock: Replace the replacement character of Tizen to 0x00A0 from 0xFFFC" into tizen

Merge "Evas textblock: Add glyphs shaping exception with checking language script" into tizen

Merge "Evas textblock: Use width of item format to position cursor." into tizen

Merge "Evas textblock: improve and fix line range rectangles" into tizen

Merge "Evas textblock: fix evas_textblock_cursor_line_set" into tizen

Merge "textblock: support escaped &apos; tag" into tizen

Merge "Evas textblock: fix null-derefernce in _style_match_tag()" into tizen

Merge "Evas textblock: add a stringshare fastpath to markup_set" into tizen

Merge "Evas textblock: fix cached markup's stringshare refs" into tizen

Merge "Evas textblock: Fix markup cache, was completely broken." into tizen

edje_text: Deleted duplicate line.

Change-Id: I83af52a56b329a230aebf610fa90c21c92a3ad00

Merge "edje_text.c: Fix text size bug when text size calc for fit option support." into tizen

eina: return NULL iterator in all case, not only when safety check is turned on.

origin from upstream:
https://git.enlightenment.org/core/efl.git/commit/src/lib/eina/eina_hash.c?id=5aa6a73b3461af4d67ba2dfe369ee2da3e11c9b5

Signed-off-by: Duna Oh <duna.oh@samsung.com>
Change-Id: I329c710efe7d71b4dc0e28004add06f0754cba8e

ecore_evas_drm: change the log level

Change-Id: I30eafe040893f8ec63cdd44f86cf4780cae90e20