photocam: merge image_region_show() into image_region

and mark this API as legacy

Signed-off-by: Amitesh Singh <amitesh.sh@samsung.com>

photocam: remove internal_image_get as eo api

and mark it as legacy API instead.

Signed-off-by: Amitesh Singh <amitesh.sh@samsung.com>

elm_code: Fix off by 1 in selection refactor

c++: Fix examples as Win.Standard was removed

elm: Remove some of Gen_Item types from EO

Those belong to legacy only, EO interfaces will use models
and objects, not function pointers and item classes.

Ref T5329

efl model: Fix invalid type in EO

Efl.Canvas refers to a canvas, like Evas or Window. Efl.Gfx is
the base visual object type in libefl.so

evas map: Fix uninitialized fields

This fixes a "jump on uninitialized value" as reported by valgrind.
See evas_map.c:85:
  if (obj->map->cur.map->normal_geometry.x != x1) ch = 1;

prefs: Remove some elm_prefs types from EO

Ref T5329

widget: Remove widget_callback_add/del from EO

Those APIs are not bindable with EO since they pass in a function
pointer. Luckily they are barely used.

Ref T5329

elm_code: Fix invalid read of 1 byte

After end of string...

elm_code: Fix use after free

edje_cc - use mem_alloc to fail and exit if alloc fails and fix err out

PVS sudio thought this was odd and it was a bit. not a real issue as a
mem alloc failure herw would still be fatal and simply a crash... as
opposed to a clean "out of mem" and fatal exit then...

error out eally was not finding originally allocated color and the
malloc should have been a mem_alloc() that guarantees a valid memory
return (or termintion of the process).

@fix

elm glview -> fix pvs studio caught typo in stencil bit mappings

this fixes CWE-570 on https://www.viva64.com/en/b/0508/

@fix

edje_cc docs - fix docs for vector images as it is wrong

it's images { vector: "x.svg"; } not vector { vector: "x.svg"; } ...

@fix

eo - eoid - finally rtemove option to not have eoid. it doesn't work

we use too many bits for metadata now so eoid is broken... remove it
as an option so people dont break out the foot guns

ecore_input: add ECORE_AXIS_LABEL_TOUCH_PALM enum

Summary:
Some of latest touchscreen devices can report if the tool in contact
is the palm of your hand and there is no axis label for this palm flag.
This adds ECORE_AXIS_LABEL_TOUCH_PALM enum as Ecore_Axis_Label type.

Test Plan: N/A

Reviewers: raster, cedric, jpeg, jypark, devilhorns, ManMower

Reviewed By: jpeg

Subscribers: JHyun

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

win: Remove special class Efl.Ui.Win.Standard

All windows should be standard, really. Except when using legacy
elm_win_add() or if type_set() was called with a specific type.

I dislike type_set...

Ref T5322

Efl.Ui.Autorepeat: add 'autorepeat' prefix to methods.

Thanks to Dave for pointing out this.

Signed-off-by: Amitesh Singh <amitesh.sh@samsung.com>

elm_code: Add a save_as API call for writing buffers to a new file

elm_code: Fix insertion of multiline content in widget

A small refactoring to reduce code duplication as well :)

examples: evas: fix else block by adding needed parenthesis

This was broken since over a year. Happened during the automatic eo4
migration in f21ade6123b9053fca8d1632644c48525a99d6e0.

Thanks goes to the gcc warning misleading-indentation:

evas-3d-shadows.c:163:4: warning: this ‘else’ clause does not guard... [-Wmisleading-indentation]
    else
    ^~~~
evas-3d-shadows.c:165:6: note: ...this statement, but the latter is misleadingly indented as if it is guarded by the ‘else’
      evas_canvas3d_node_look_at_set(scene->mediator, EVAS_CANVAS3D_SPACE_PARENT, 0.0, 3.0, 0.0, EVAS_CANVAS3D_SPACE_PARENT, 0.0, 5.0, 0.0);

evas: Rename evas_types to evas_canvas3d_types

Now evas_types has nothing but canvas3d definitions.

Fixes T5312

evas: Remove unnecessary imports of evas_types

Ref T5312

evas: Remove Efl.Canvas.Text.Annotation from evas_types.eot

This moves it to efl/interfaces which may not be the best...
@herdsman should know where to place this opaque definition.

Ref T5312

evas: Add seat arg to modifier/lock EO APIs

This now matches the new seat-related set of APIs present as EAPI.

evas: Remove Evas.Modifier from EO

Similar to the previous patch with Evas.Lock

Ref T5312

evas: Use enum instead of string in Efl.Input.State

Ref T5312

ecore_x: Fix build warning (invalid enum used)

evas: Remove Evas.Lock from EO

This is a step towards the removal of the modifier & lock API
as it is and use enums exclusively.

Ref T5312

evas: Move Efl_Gfx_Fill_Spread to legacy

This was never implemented and was translated to EO without
any implementation.

evas: Move evas event info types to Evas_Legacy.h

This moves the following definitions from Common to Legacy:
 - Evas_Event_Mouse_Down
 - Evas_Event_Mouse_Up
 - Evas_Event_Mouse_In
 - Evas_Event_Mouse_Out
 - Evas_Event_Mouse_Move
 - Evas_Event_Mouse_Wheel
 - Evas_Event_Multi_Down
 - Evas_Event_Multi_Up
 - Evas_Event_Multi_Move
 - Evas_Event_Key_Down
 - Evas_Event_Key_Up
 - Evas_Event_Hold
 - Evas_Event_Axis_Update

evas: Move Evas_Engine_Info to Evas_Legacy.h

gesture: Use new EO API to grab Ctrl key

See previous commit

evas: Replace Evas.Modifier_Mask with Efl.Input enums

This removes Evas.Modifier_Mask from EO.

This introduces two new types instead:
 - Efl.Input.Modifier
 - Efl.Input.Lock

Those are enums, not strings, containing all the known and
currently supported lock and modifier keys. The enums are
bit masks.

This effectively removes the ability for an application to
create and handle specific modifier or lock keys - with EO
API (legacy compatibility is unchanged, of course). I wonder
who ever required this?

evas: Remove evas_key_modifier_mask_get from EO

Such an ugly API. This is an API to match a string to a number,
build a bitmask from it, and then use that. The supported
strings are well known (should be enum) and would need a
recompilation (ABI update) to support anything new anyway.

elm: Add missing eo.c file for elm_button

This fixes a recently introduced crash in elm.

Efl.Ui.Autorepeat: Add efl autorepeat interface

Signed-off-by: Amitesh Singh <amitesh.sh@samsung.com>

ecore_drm2: Clear next fb if we use it

Fixes a race that's either really hard to hit if you're a developer
or really easy to hit if you're a user.

Thanks to ApB for the debug assistance.

Fix T5484

evas: Improve linespacing consistency

Reviewers: cedric

Subscribers: jpeg

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

ecore_evas: Omit redundant return info in doxygen

Summary:
These routines all already have a @return line that states the return
values appropriately; it's unnecessary to restate the return values in
the documentation text.

Signed-off-by: Bryce Harrington <bryce@osg.samsung.com>
Subscribers: cedric, jpeg

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

photocam: Fix invalid type in legacy header

Sorry, this temporarily broke the build of external apps.

evas: Remove Evas.Coord from EO

This may be a bit more controversial...
But Evas_Coord really is just an int and all the internals
of EFL assume that the base coordinate type is a 32-bit int.
So this type is a bit redondant and can't easily be changed
to, say, a float or int64.

Ref T5312

evas: Remove font_hinting from EO

We only need it in elm_config.
This removes the type Evas_Font_Hinting_Flags from EO,
as well as the functions font_hinting_set/get and
font_hinting_can_hint.

Ref T5312

evas: Remove evas touch_point from EO

Note: it seems the EAPI evas_touch_point_list_xy_get() was
lst at some point, as it doesn't appear in the headers anymore.
It looks like we fail to catch an ABI break! abi_checker didn't
catch this!?

Ref T5312

evas: Remove Evas_Border_Fill_Mode from EO

Ref T5312

evas: Remove Evas_Image_Orient from EO

Ref T5312

photocam: Move image_orient to legacy

This makes photocam implement Efl.Flipable and Efl.Orientation
in order to use the proper API.

image_orient is now split into two function calls.

Ref T5312

evas: Remove some types from EO

Those are legacy only.

Ref T5312

evas: Remove Evas_Callback_Type from EO

This moves the enum to legacy only, but in Evas_Common.h and not
Evas_Legacy.h because some other legacy things are in Evas_Common.h

Ref T5312

elm_widget: Remove widget_event_propagate from EO

It is an internal only API.

Ref T5312 (one less use of a legacy type)

evas: Remove Evas_Text_Style_Type from EO

It belongs to legacy only... Evas.Text is not part of EO.

Ref T5312

evas: Remove Evas_Font_Size from EO

Ref T5312

ecore_drm2: Fix use after free

In trying to clean up some code and fix a hypothetical buffer leak, I added
a use after free error that can break rendering on the drm and gl_drm
evas engines.

Coverity did the heavy lifting for me on this one.

Fix Coverity CID 1375047
Fix T5484

ee_drm: Remove some commented out code

I think this was intended as a reminder to do something that's already
been done.  Or something.

ecore_drm2: Add some safety checks for accessing dead fbs

Accessing an fb after discarding it is bad karma, so we should log something
if it ever happens.

ecore_drm2: refcount fbs

Removes the previous "busy" flag, as now we might have an fb attached to
multiple outputs at once, and need to be careful to destroy them only
after they've been removed from all outputs.

Removed the old "busy_set" API which nothing used, and renames fb_destroy
to fb_discard to make it more clear that it's not immediately destroyed.

It's all beta api, so I can do this.

evas: simplify fb engine.

evas: simplify buffer engine.

evas: stat introducing a concept of engine and output in the backend.

Shelf/Gadget themes... Update with proper horiz/vert theme strings for future theme developers.

ecore-wl2: add functions for proxying a selection receive externally

in some cases (e.g,, x11 bridged selections) it is necessary to use
alternate means when transferring a selection, and so performing the
entire piped receive is not necessary. instead, extend the lifetime
of the data offer until the proxied receive has completed

@feature

ecore-wl2: add function for setting a drag source's actions without a drag

in some cases it may be desired to have a drag data source without ever
initiating a drag, so ensure that it's possible to set the actions of the
source which will be passed for drag operations in this case

@feature

ecore-wl2: add function for accepting a single mime type

@feature

ecore-wl2: do not offer a null mime first in ecore_wl2_offer_mimes_set()

this seems to have been done to account for the case where no mimes were
passed, but sending null in this way is bad behavior for a client. the
spec indicates that null should be sent only when no mimes are accepted

@fix

ecore-wl2: add events for changes in keymap, selection, and kbd repeat rate

@feature

ecore-wl2: add 'cancelled' to Ecore_Wl2_Event_Data_Source_End

it's impossible to know the result of a drag operation without this
member

ecore-wl2: add display object to all events

this is required in order to determine which connection an event originated
from

ecore-wl2: add const to internal function param

ecore-wl2: add seat id to most events

seat id is required in order to handle multiseat environments

elm_cnp: only attempt to manage wl selections which are created by elm_cnp

use serials of the selections and events to avoid mangling external selections

ecore-wl2: emit serials in selection-related events

these allow apps to match the serial of their selection to the related
event in order to avoid mangling selections from other components

ecore-wl2: change selection setting apis to return the serial of the request

in the case where multiple places in an app may be handling selections, this
is necessary in order to manage the selections accurately

@feature

ecore-wl2: remove ERR when trying to find the focused window id

this can trigger in the case where a selection transfer occurs and
no input resource has received an enter (e.g., the window is not focused)

@fix

ecore-wl2: correctly refcount display objects during events

ensure lifetime of display object is longer than events in which
the object is passed

@fix

ecore-wl2: add safety check in ecore_wl2_display_inputs_get()

ensure this is not called on server displays

ecore-wl2: handle case where session recovery listener triggers on dead surface

@fix

ecore-wl2: add function for checking if a client connection sync is done

@feature

ecore-wl2: add some accessors for useful Ecore_Wl2_Input struct members

@feature

ecore-wl2: split data offers into selection and drag

these are distinct objects and can coexist simultaneously; a drag
source should not overwrite an existing selection source

@fix

ecore-wl2: store names for inputs and add api for retrieving name

@feature

ecore-wl2: add checks for a valid display object in ecore_wl2_input fns

no display = invalid object, so this needs checking

ecore-evas: account for framespace when processing mouse movement

in some cases (e.g., the mouse-out emission from the ee wl engine),
this was calculating wrong coordinates, which would lead to a broken
chain of events like:
  object mouse out -> object mouse in -> object mouse out
which would severely break some apps

@fix

ecore-evas: add function for unsetting a specific seat's cursor

@feature

ecore: allow creation of fd handlers with no flags

it's possible to set flags to 0 with another function, so allowing
creation with 0 makes the api more consistent without breaking the
documented behavior

elm_entry: request both text and markup when pasting

setting this to only markup means that it's impossible to paste into
an elm_entry from a plaintext app under wayland

@fix

elm_cnp: don't explicitly set wl window type when starting drag

this is no longer necessary

evas: ensure even no-op renders emit RENDER_PRE

ref 67fae7aa0fdc9d778e8db88fc49bc149576994d2

tests: win_dialog: remove unused variable

efl ui win -> add test for centering + fix center of window before show

we haven't gotten replies yet on what our position or size should be,
so we should store them so centering works before show but after
resizing is evaluated (that also fixed by forcing an eval).

@fix

efreet: Fix failure to save list data to output

Summary:
This fixes a typo in the fix 55676b33, which introduced an invalid early
return from the save_list function, preventing it from outputing the
list data to the file.

@fix CID1375005, CID1375004

Reviewers: jpeg

Reviewed By: jpeg

Subscribers: stefan_schmidt, cedric, jpeg

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

evas gl: Fix typo and revert hack (GLES 3.1)

Omg... Thanks Daekwang Ryu for pointing me to my error. I remember
struggling a lot with this OpenGL API and libGLdispatch (glvnd) when
in fact this was all just a typo in the code.

GLES 3.1 and the upcoming 3.2 support need a proper test case...

See c68a40987404375b36460b6a7f5ccceb45444b57

@fix

elm_entry: Cursor disappears on calling select function in an empty entry.

Summary:
Issue: Selection cannot happen on an empty entry, if selection functions
are called on empty entry, cursor is hidden, even if entry is focused.
Soln: Check for whether entry is empty or not before proceeding with selection.

Test Plan:
1. Call select_all, select_region_set on a focused entry
2. You can observe cursor cannot be seen.

Reviewers: jpeg

Reviewed By: jpeg

Subscribers: cedric, jpeg

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

popup: move "visible_set(FALSE)" before calling dismissed callback

Summary:
 After hiding animation is finished, "dismissed" callback will be called.
 It means popup's visible state is FALSE.
 But if evas_object_show(popup) is called in it's custom dimssed callback function,
 popup's visible state is TRUE, so evas_object_show is canceled.
 So visible_set(FALSE) is must done before call dimissed callback.

@fix

Reviewers: Hermet, herb, jpeg, cedric, singh.amitesh, raster

Reviewed By: jpeg

Subscribers: woohyun

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

popup: enable to set last_button_number to 0

Summary:
If last added button is removed, any statements to set last_button_number in for loop doesn't be executed.
So last button number has previous value, not 0.
Because of that, action area is not removed, but that area has zero height, so it doesn't be shown.

@fix

Reviewers: herb, jpeg, cedric, singh.amitesh, raster

Reviewed By: jpeg

Subscribers: woohyun

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

evas: Revise documentation for evas_async_events*()

Summary:
Straightens up grammar and simplifies phrasing to make the functions
purposes clearer.

Subscribers: cedric, jpeg

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

examples/evas: Fix weird use of word synchrony in comment

Summary: 'keep in sync' is more familiar programmer jargon.

Subscribers: cedric, jpeg

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

efl_ui_focus_manager: Fix misspellings in comments

Subscribers: cedric, jpeg

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

evas: Fix two whitespace errors

Summary:
Removes extraneous newline, and fixes the indentation of the while
statement's sub-clause (which looked like a code error at first
glance).

Signed-off-by: Bryce Harrington <bryce@osg.samsung.com>
Subscribers: cedric, jpeg

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

evas textblock: fix dereferenced memory after free issue for ellipsis

Summary:
_layout_ellipsis_item_new() function deallocates "ellip_ti" and
allocates memory for new one. The local "ellip_ti" pointer should be
updated.
@fix

Test Plan: N/A

Reviewers: raster, cedric, herdsman, jpeg, woohyun

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

evas map: Rename raw_coord to coord_absolute

This is more in line with the other "absolute" APIs.