efl_config: Fix type of returned value (EO)

An ugly const char * is returned from strdup, freed later with a custom
function that just calls free(). This was probably intended for
stringshare but now free is used. We have to fix the free function for
the EO API so let's keep free(). I'd rather avoid declaring
elm_config_profile_dir_free in the EO files.

elm: Drop unused class inheritance (EO)

elm_test: Scale down icon by 8x

This makes the icon test actually work. Otherwise the icon data is too
big and basically seems ignored by the compositor or X.

Note: In E (X11) it seems that the window icon remains unchanged? xprop
shows the proper data, though. Ping @zmike.

eina_array: Fix magic checks to return

The EINA_MAGIC check inside eina_array were just printing the error
message, without failing. This could lead to more crashes than
necessary.

@fix

win: Fixup doc & code of icon_object

Ref T5322

win: Use array for available_profiles (EO)

The legacy API wraps the EO API.
All tested with "Configuration" test case in elm_test.

Ref T5322

win: Drop calls to lower when using the EO API

Note: The distinction is made on how the window was created, not on
which API is used (evas_object_lower or efl_gfx_stack_lower or
elm_win_lower).

Ref T5322

win: Implement raise from stack (EO)

I think the intercept is not required anymore. All raise functions call
the ecore evas raise function.

Ref T5322

focus: Fix shadow warning

widget: Forward focus_highlight_style to win

See the previous commits. All focus_highlight APIs are defined in the
Widget class but only implemented at the Window level. For consistency I
believe this call should also be forwarded to the window. The previous
logic had absolutely no effect at all, except storing a stringshare.

The day focus_highlight becomes widget-specific (i.e. each widget has
its own highlight style) then this can be changed.

Note: This will apply to legacy API as well.

Ref T5363

win: Move focus_highlight_animate to widget (EO)

Similar to the previous commits. Moved for consistency.

Ref T5363
Ref T5322

win: Move focus_highlight_enabled to widget (EO)

This was actually declared in the internal legacy API in widget.
Forwards the calls to the window.

Ref T

win: Merge focus_highlight_style with widget (EO)

Note: elm_test "Focus Style" can be used to test this API. The test case
is a bit broken (overly complex EDC?) but if you're patient you can see
the difference between "glow" and "glow_effect".

Ref T5363
Ref T5322

widget: Rename on_focus to on_focus_update (EO)

on_focus seems to imply that focus was just given to the widget, but
that function is called on any focus change (in and out).

Ref T5363

win: Merge focus with elm_widget's (EO)

I believe this should work fine!

Ref T5363
Ref T5322

gfx_size_hint: Fix int vs. double (EO)

Thanks Sanghyeon for letting me know!

ecore_wl2: Fix session recovery

I broke this in commit 1bb45f6e6194fa046d3d1409e1aacecc14dc4b4f

We intentionally *don't* flush in window_semi_free, as this could be
when there's no compositor left to flush to (in the case of session
recovery).

docs: elm/evas: document some missing return values

With this patch we finally reached 100% doc coverage in EO files again.
It would be nice to keep it like this.

docs: efl_canvas_object: document last missing part

docs: efl_ui_panes: document last missing part

docs: elm_widget: document function pointer for on show region

docs: efl_ui_focus_manager: fill in missing docs

docs: efl_text_annotate: fill in missing documentation

docs: efl_inout_device: fill in missing docs

docs: even give the external types some documentation

docs: efl_text_cursor: add missing documentation for many parts

docs: efl_text_font: fill in missing parts of documentation

docs: efl_text_format: fill missing parts documentation

docs: efl_text_style: fill in missing documentation parts

docs: efl_text_style: document color components in styles

RGBA should be pretty clear for most cases but it does not cost us much
to document these values as well and make the documentation maybe easier
to understand for some people.

docs: efl_ui_textpath: add documentation for all other missing parts

When bringing in new EO files I would appreciate if people would
document all parts. Even if they seem small and self explaining.

docs: efl_ui_textpath: document newly added textpath direction enum

docs: efl_ui_manager: document newly added struct

This was added recently and thus has a since 1.21 tag

docs: evas_canvas_text: document missing events

elm focus manager - dont leak child lists when updating focus order

REALLY fix T5800 by duplicating lists rather than taking ownership

@fix

focus manager - fix spelling of variable

saved_order not safed_order

elm - focus manager - revert free list

revert 65d2dfc8925a183bf7f2709677d87edcd759e9c3

no no. this fixes the leak ... but it adds new segv bugs. revrt it.

naviframe: Fix to load default style if the given style does not exist

Naviframe item theme name is combined with item style name and widget
style name.
Therefore, if the item theme name is not found, then "item/basic" item
style name should be loaded as a default item style name.

elm focus manager - dont leak child lists when updating focus order

this should fix T5800

@fix

efl_ui_clock: Add 'O' character to ignore POSIX alternative format in format.

Summary:
_parse_format() function filtering "_/-/0/^/*" to filter POSIX alternatvie/extension formats,
But the Datetime do not appear in some locale(fa_IR, lzh_TW, my_MM, or_IN) with 'O'.

+ improve code. (not compare with all of the characters. instead of that using strchr)

https://lh.2xlibre.net/values/d_t_fmt/

@fix

Test Plan: Set locale as fa_IR and create DateTime. and see fields, not appears.

Reviewers: jpeg, cedric

Subscribers: cedric, jpeg

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

mbe: Fix infinite recursion on focus change

Ref D4882

multibuttonentry: Separate selected item and focused item logic.

Summary:
When user set a selected item on MBE creation time.
The internal logic will be broke.
So we need to separate logic them.

@fix

Test Plan:
Calling elm_multibuttonentry_item_selected_set() before MBE got a focus. in elementary_test.

Reviewers: cedric, jpeg, woohyun

Subscribers: bu5hm4n, marcelhollerbach, cedric, jpeg

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

widget: Fix legacy for focus_mouse_up_handle (EO)

This removes the special code in the legacy API for
elm_widget_focus_mouse_up_handle. Add an internal helper to find the
first widget parent. And mark as protected.
Apparently this functions is still required for the new focus manager.

Ref T5363

widget: Mark old focus API as beta.

It's not beta. It's about to die.
Also, move #define ELM_WIDGET_BETA to the common header file, as it is
consequently required by ALL widgets. :(

Ping @bu5hm4n :)

Ref T5363

Luncher Theme: Set no border for alpha. This needs to be backported.

efl_ui_focus_manager_calc: only focus none NULL candidates

efl_ui_focus_manager: make logical_end work better

it turns out that we should also repsect logical elements that are
having a redirect_manager, since they are more at the "end" then a
potential regular node.

The user now needs to handle the logical_end call on this manager, or
handle at all what he wants to do with this information.

efl_ui_win now handles it in the way that it just focuses that logical
node, (which results in the redirect manager beeing set, then calling
again logical_end on that manager. Repeating this until we have finally
found a regular node that does fit out needs.

ecore_wl2: Kill the wl idle enterer

We do flushes when we need them now, so we don't need this somewhat
non deterministic flush mechanism anymore.

Anything that breaks as a result of this should be fixable by adding an
ecore_wl2_display_flush() somewhere appropriate.

ecore_wl2: Use ecore_wl2_window_commit for all wl_surface_commits

This adds an explicit flush to every surface commit.

ecore-wl2: Formatting

NB: No functional changes

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

wayland: Force a display flush when committing surfaces

We had a hack in place to flush the display from an idle enterer instead
of after a surface commit.  This led to a problem where the idle
enterer dispatch order was:

renderer for main canvas
wayland dispatch idle enterer
renderer for mouse cursor canvas

The surface commit for the mouse cursor was never dispatched, so the mouse
cursor animation would only update at the rate other events occurred.

By flushing at the appropriate times instead we ensure a proper update.

ref T5850

ecore_wl2: Add ecore_wl2_display_flush() api

Allowing engines to explicitly flush at the right times will remove some
awkwardness and bugs from our current display flush paradigm

wayland: Store Ecore_Wl2_Display instead of wl_display in engines

We'll need to call ecore_wl2 functions on it soon, and anything that
really needs the wl_display can query it.

walyand_shm: Remove unused variable

Nothing in here actually cares about the wl_display anymore.

ecore_wl2: Fix small bug in flush handler

If wl_display_flush() returns a non negative number it was successful,
so we should disable the write handler then.  We were theoretically always
doing an extra flush on an empty buffer.

This is mostly cosmetic as that's not a costly operation.

ecore_drm2: Fix warning with a dirty hack

So because we include wayland-server.h we're told that wl_buffer is
deprecated - however clients are still expected to use it, it's only
deprecated for usage in a compositor.

Making these into void pointers shuts up the warning.

We need to include wayland-server.h since some of the code provided by
ecore_wl2 is for compositors, but some of it is also for clients...

evas font: do floating point division for calculating more accurately

Summary:
Assigning a result of integral division to a double type variable is
not useful for next division calculation. For more accurate calculation,
it needs to be casted to double before doing division.
It does not fix some bugs. It was reported by a code quality advisor.

Test Plan: N/A

Reviewers: raster, cedric, jpeg, herdsman, eunue

Reviewed By: cedric

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

Signed-off-by: Cedric Bail <cedric@osg.samsung.com>

elua: load regular file if bytecode cache fails

If bytecode is requested and fails to load, load original file
instead. Force re-write of bytecode if that succeeds.

This is useful if there is bytecode newer than the source file that
the current Lua version can't load, for example when migrating from
LuaJIT 2.0 to 2.1, which changed bytecode format.

Fixes confusing cases such as in T2728.

elua: update eolian bindings and make docgen work again

eolian: remove c_only

Unused and of questionable value.

evas: we always do use the common backend, so always init.

efl_ui_focus_manager: reset focus stack once we are changing redirect

ref T5923

eolian: @extern for function pointers

eolian: clean up and fix function pointer generation

Out/inout params are now correctly handled.

efl interfaces: fix incorrect enum val

eolian gen: clean up param gen

eolian: remove unused variable

ecore-wl2: Formatting

NB: No functional changes

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

widget: Simplify & document "translate"

This is a protected function. It doesn't need to return anything, as all
implementation just returned true, always. Also, the legacy API was just
a wrapper doing nothing special (except verify that we have a widget,
which the recursive code already does).

Tested with fr_FR :)

Ref T5363

widget: Remove part_text from EO

It should be implemented as a efl_part() API.

For now I've only done a quick hack, as the only overrides were:

 - notify: already a Part implementation. Also it turns out the default
   theme does not even have any text part in the notify group.

 - combobox: not a Part implementation, but also very badly defined wrt.
   parts in general. efl_part() is handled by the parent class (button)
   which makes it tricky to override just for one function.

With this patch I'm trying to keep the same behaviour as earlier (where
efl_part() is used for layouts and there is a special path for
combobox).

Ref T5363

widget: Mark some functions as protected

I believe all of those APIs are in fact meant for widgets to use
themselves:
 - on_focus
 - on_show_region_hook
 - focus_region
 - focus_register
 - focus_manager_factory

Ref T5363

widget: Move item_loop_enabled to scrollable (EO)

I was told that the scrollable interface is being redesigned for EO.
This API definitely does not belong to the base Widget class, as it's
quite specific to item-based scrollable widgets, such as lists and
grids. Since Elm.Interface_Scrollable is itself being revamped, it is a
good place to move that EO API for now.

Ref T5363

widget: Use rectangle in show_region

Also make it a property. It's asymmetric because of the force show
argument, but the get is much cleaner.

Ref T5363

eina: Add convenience eina_rectangle_equal

@feature

widget: Remove scroll_lock from EO

1. Uniformize the API, which is now for internal use:
   This uses the same enum as scroller "movement_block" instead
   of 2 separate properties. Less APIs, more consistence.

2. Remove scroll_lock x/y from EO widget. I was told it is not going to
   exist in the upcoming scrollable interface.

3. Remove scroll hold/freeze getters.
   scroll hold/freeze push/pop are still there but it remains to be seen
   how the EO scrollable interface will exploit them. Right now they are
   full of bugs.

Ref T5363

scroll: Rename elm enum to efl.ui

This scroll block enum should also be used in widget itself.

Ref T5363

widget: Rename drag_lock to scroll_lock (EO)

This also includes the drag_child_lock APIs. This had nothing to do with
dragging beyond maybe the case where scrolling is done by thumbscroll
(ie. finger drag).

Note that the EAPI were called already scroll_lock, not drag_lock.

Ref T5363

widget: Remove drag_child_locked APIs (EO)

These are just internal helper functions for the scrollable mixin. I
don't think they need to appear in the external API.

Ref T5363

widget: Use rectangle on the stack for region_show

Follow @k-s recommendation and simply use rectangles on the stack rather
than by pointer.

Ref T5363

docs: remove outdated NOTES files with random notes about evas_vg

I have to say I am buffled that this file made it into the tree. The
naming is very generic (similar to NEWS; README, etc) but it only
contains some random notes about evas_vg. It has not been updated since
its initial commit in 2015 either.

Cedric, if you want to have these notes somewhere put them into
src/lib/evas, or better make it a real doc and put it into the wiki.

eina mmap safety - only map zeropages if it's an eina file that sigbuses

restrict mapping /dev/zero to only eina files having a sigbus
reported. the mmap was before all our file access used eina_file i
think thus the raw mmap of it. now walk all eina files and find the
candidate and only then if it exists flag is as having a faulty i/o
backing and map the zerto pages then return, otherwise call abort.
more restricted mapping and perhaps a fix for not trapping non-efl
issues.

@fix

eina file - use recursive locks for cache and file to avoid deadlock

since we have a sigbusd handler that flags an eina file with io errors
it has to walk the file cache and every file... taking locks. if those
locks were taking already in the current thread the sighandler was
called in... we'd deadlock. since this basicallly never happens (when
do we see i/o errors really? not much)... we never saw this as it'd
also reauire this race condition to happen too. but it is a problem
waiting to happen. this fixes that by moving to recrusive locks.

@fix

emotion: gst: use proper GStreamer time-unit macros

Summary:
Do not harcode numbers that make no immediate sense.

Additionally: add some wont-hurt doc note and fix
two related typos.

Subscribers: cedric, jpeg

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

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

emotion: gst: drop Frenglish from English messages

Subscribers: cedric, jpeg

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

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

ector: cleanup grammar in comments

Reviewers: cedric

Subscribers: jpeg

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

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

evas: make the high level documentation more concise

Summary:
This is a very informative document but is much longer than it needs to
be.  Tighten it up by condensing redundant information and expressing
the ideas more efficiently.  Focus more on Evas and what it is than what
it isn't.  Avoid explaining general graphics concepts like immediate
vs. retained, replacing with synopses.  Switch from 2nd person to 3rd
person (i.e. don't say You/Your) to be less awkward, since we don't
really know why the reader is reading it.  Simplify the compilation
directions; these are pretty standard, and most people won't be manually
linking to Evas anyway.

While this shortens the document considerably, it retains the
important key points, and makes it far more readable.

Reviewers: cedric

Reviewed By: cedric

Subscribers: jpeg

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

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

evas: fix color set on Efl.Canvas.Group.

PKit gadget: more contrast for the design

the blue icon was not really readable on that light gray bg,
changed the icon to be dark gray

evas: fix invalid return of evas_textgrid_cellrow_get()

evas_textgrid_cellrow_get() is supposed to return NULL if the row index
fed to the function is outside of the textgrid's range. It was instead
returning garbage pointer.

@fix

eina: fix typo in documentation

elm: fix uninitialized variables

Canvas text: fix non-dirty paragraph width calculation

Follow-up fix for 1624417d9137b87826962e431d08dc591f83d5e5.
Changed for a max comparison, rather than just assigning the line's
width.
Also, added a test case.

Fixes T5939

@fix

Fix PS3 backend removal (Cedric...)

When running elementary_test with ELM_ENGINE='buffer', we got a crash.
The removal of PS3 backend patch didn't remove ELM_SOFTWARE_PSL1GHT and
didn't shift the _elm_engines indexes.
ELM_SOFTWARE_DDRAW stayed at the index 13 (value NULL) instead of moving
to index 12.
A strcmp with NULL occurred, leading to the crash.

@Cedric, I excuse you to not have run Exactness your code before pushing
:P

Introduce text on path widget

Text on path (textpath) allows application to make text follow a path.
The path can be a efl_gfx_path or a circle.

Thank hermet for initializing this work.

@feature

widget: Add protected tag to focus_highlight_geometry

This is meant to be called by the widget or elementary internals,
potentially reimplemented by widget subclasses.

Ref T5363

widget: Use rectangle for focus_hilight

Ref T5363

widget: Mark focus_region_show as protected (EO)

This function is meant to be used by the widgets themselves, or internal
features such as elm_access.

Also remove const tag: this function call is definitely modifying the
widget (panning around and all that).

Ref T5363

widget: Simplify code with rectangle (EO)

This replaces x,y,w,h with a rectangle in parts of the focus_region
code.

Ref T5363

widget: Merge on_focus_region and focus_region_get

This simplifies code and API.

Ref T5363

widget: Improve access_info doc

Ref T5363