eina: fix doxygen typo for eina_thread.

Summary: fix typo.

Test Plan: API Doxygen Revision

Reviewers: cedric, jpeg, myoungwoon

Reviewed By: cedric

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

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

elementary: generic_value is to be replaced by any_value.

elementary: add an exit_code to efl_exit as originally planned.

T6228

elm_interface_scrollable: fix bringing in of a focused item

you also need to calculate in the object position

elm_widget: print a error if there is no provider

elm_widget: also do not register if the top widget is not a win

this is currently quite a performance break, since that call is quite
heavy, lets see if we can optimize it later

elm_widget: only register if you have to

first step into the direction of making that more secure

elm_widget: move evaluation for registeration of widget

this is now earlier done, so we can reacted based on that

elementary: remove documentation error

eo: Use any_value in eina_types.eot

Reviewers: q66, jpeg

Subscribers: jenkins, cedric

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

example: print format correction to avoid warning.

Reviewers: raster, cedric, jpeg

Subscribers: rajeshps, jpeg

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

frame: Implement content set/get/unset

Not a huge fan of my macro as it uses efl_part() where a direct call
could be possible, depending on the widget (win can, frame can't).

See D5241

elm: Rename Elm.Activate to Efl.Ui.Activate

Note: This is an EO-only beta API.

Ref T5329

slider/progressbar: implement format_string of Efl.Ui.Format

remove the unit_format from Efl.Ui.Range.

fix typo space in include on eina_lock

remove space typo between #sharp and include about inline header
in eina_lock.h

efl_ui_format: change Eina.Strbug to @in type

efl_ui_format: fix the namings and docs

it is based on review comments by Gustavo and JP.

ee_wayland: Don't use wrong window size

Borderless windows, such as internal windows, need to be the exact size
they're passed by a configure event.

eina: rename parameter for doxygen

Summary:
Some parameter's name are different in annotations and statements,
so it occurs doxygen warning.
To fix it, rename that parameters.

Test Plan: API Doxygen Revision

Reviewers: raster, cedric, jpeg, myoungwoon, Jaehyun_Cho

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

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

efreet: putting local variable "data" under preprocessor flag "SLOPPY_SPEC" to avoid below warning, if "SLOPPY_SPEC" is disabled.

Reviewers: raster, cedric

Subscribers: jpeg, rajeshps

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

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

eina: fix warning if EINA_SAFETY_CHECKS is disabled

Summary:
Putting local variable "d" under preprocessor flag "EINA_SAFETY_CHECKS" to avoid below warning, if "EINA_SAFETY_CHECKS" is disabled.
        1. local variable "d" is assigned but not used.
        2. If warning 1 is resolved then variable "d" will be unused.

Reviewers: raster, cedric

Reviewed By: cedric

Subscribers: jpeg, rajeshps

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

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

ecore: removing assignment operation to avoid warning.

Summary: assignment to local variable "ret" has no meaning as it is not used after assignment. So, removing assignment operation to avoid warning.

Reviewers: raster, cedric

Subscribers: jpeg, rajeshps

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

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

ecore_wayland: fix assigned value is never used.

Summary:
Fix assigned value is never used.

@fix

Reviewers: cedric

Subscribers: shilpasingh, jpeg

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

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

edje_edit: duplicate assignment to variable.

Summary:
Avoid duplicate assignment to same variable.
@fix

Reviewers: cedric, jpeg

Reviewed By: cedric, jpeg

Subscribers: shilpasingh, jpeg

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

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

elm_gengrid: implement item-content-focus

 #finally

For now we focus the widgets of a item, the item content can be cycled
by tab / Ctrl + tab. up/down/right/left are for now handled by gengrid
and move the focused item (everything else feels super weird with
multiple contents in a item)

ref T6181

efl_ui_focus_composition: support beeing a pure logical composition

efl_ui_composition: flushing elements from outside!

elm_slider: do not eat all key down events

efl intf: Add format interface

Ref T6204

efl_ui_focus_manager_calc: maintain focus when focused element is
unregistered

this should fix T6216.

elm_toolbar: make the manager customisable

so the toolbar items can be registered correctly

elm_menu: implement provider_find to walk the parent relation

it turns out elm_menu is special, it doesnt use parent_obj of
elm_widget, it has his own parent field, which should be used.

efl_ui_bg: this does not handle focus

elm_toolbar: port to composition

elm_table: port to composition

elm_grid: port to composition

elm_box: port to composition

efl_ui_focus_composition: introduce dirty and prepare

this can be used in a container that has his own item management api,
Each item management call results in a dirty call, once we are called to
prepare for logical movement we can simply flush the order. So we reduce
the spam of order calls, which also safes runtime.

elm_widget: support logical parents that are not elm_widgets

elm_widget: do not cache the provider

if turns out that caching the provider here is a problem, since a parent
changing does not change the provided provider

elm_widget: also unregister if the logical parent changes

jp2k module - dont print ERR on llading if format wrong

this is normal - brute force trying loaders until one succeeds is
normal is etn doesnt help identify it or it fails the first
guess-by-extension. printing errors is not good as this is an ok and
EXPECTED error. slience!

@fix

evas gl generic/common - add more linking for gles mode to fix deb build

this should fix T6158

@fix

update author(hermet) email.

eolian: fix setter generation for @auto functions

Because of a typo in generator source (and overlooked error in
tests) we were previously generating incorrect code for setters
with the @auto qualifier. This was brought up in D5306 and is
now fixed.

ecore_file: rename parameter for doxygen

Summary:
Some parameter's name are different in annotations and statements,
so it occurs doxygen warning.
To fix it, rename that parameters.

Test Plan: API Doxygen Revision

Reviewers: raster, cedric, jpeg, myoungwoon, Jaehyun_Cho

Reviewed By: cedric

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

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

eina: this was actually a used variable.

evas: as we do not use the size, we should set it either.

elementary: we actually do not use that variable.

elementary: we actually do not use that object.

elementary: remove unused variable.

evas: mark parameter unused now that it is not necessary anymore.

ecore_ipc: rename parameters and annotations for doxygen

Summary:
Some parameters' name are different in annotations and statements,
so it occurs doxygen warning.
To fix it, change it appropriately.

Test Plan: API Doxygen Revision

Reviewers: raster, cedric, jpeg, myoungwoon, Jaehyun_Cho

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

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

evas: to avoid warning unused variable and unnecessary assignment is removed.

Reviewers: raster, cedric

Subscribers: rajeshps, jpeg

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

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

efl: unused variable removed, removing local variable assignment which has no use at all.

Reviewers: raster, cedric

Reviewed By: cedric

Subscribers: rajeshps, jpeg

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

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

ecore_audio: unused variables are removed to fix warning.

Reviewers: raster, cedric

Reviewed By: cedric

Subscribers: rajeshps, jpeg

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

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

evas: parameter length is not used, so removing it. to fix static analyzer tool warning.

Reviewers: raster, cedric

Subscribers: jpeg, rajeshps

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

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

efl: unsigned int/long never be less than zero. Fixed.

Reviewers: raster, cedric

Subscribers: rajeshps, jpeg

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

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

ecore_evas: preprocessor flag "BUILD_ECORE_EVAS_SOFTWARE_GDI" is not closed properly, if not defined, leads to build break.

Reviewers: raster, cedric

Reviewed By: cedric

Subscribers: rajeshps, jpeg

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

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

edje: no case is defined for Preprocessor Flag "EDJE_EXTRA_MODULE_NAME". So removing it to avoid printf warning.

Reviewers: raster, cedric

Subscribers: rajeshps, jpeg

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

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

eina: uninitialization of array leads to unwanted outcome of "eina_strlcat"

Reviewers: raster, cedric, rajeshps

Subscribers: cedric, jpeg

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

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

elmentary: fix a potential null pointer dereferencing in elm_box

Summary:
if 'evas_object_smart_data_get' return null somehow,
logic that dereference the smart data pointer will cause problems.
This patch prevent a potential bug in advance.

Reviewers: jpeg, woohyun, cedric

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

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

evas: removing unwanted operation in interface create

Summary: Signed-off-by: Godly T.Alias <godlytalias@yahoo.co.in>

Reviewers: cedric, raster, rajeshps, prince.dubey

Reviewed By: cedric

Subscribers: jpeg

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

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

evas: buffer is not used in the case where destination and source dimension is different

Summary: Signed-off-by: Godly T.Alias <godlytalias@yahoo.co.in>

Reviewers: cedric, raster, rajeshps, prince.dubey

Subscribers: jpeg

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

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

edje_cc: fix a memory leak issue when edje_cc writes images

Summary:
If there is no given pathes for image files as parameter of edje_cc,
"img_dirs" will be NULL. Then, a local variable "load_err" is always
EVAS_LOAD_ERROR_NONE. Because of this, the "if" condition just after
EINA_LIST_FOREACH() will fail. It causes memory leak from "iw".
@fix

Test Plan: N/A

Reviewers: raster, cedric, jpeg, woohyun

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

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

eina: spellfix comment in matrix code

Reviewers: cedric

Reviewed By: cedric

Subscribers: cedric, jpeg

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

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

eina: document matrix3 API routines.

Summary: This adds missing doxygen documentation for eight matrix3 operations.

Reviewers: cedric

Subscribers: cedric, jpeg

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

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

eina: comparing unsigned integer with less than zero.

Summary: Unsigned integer should not be compared less than zero.

Test Plan: NA

Reviewers: cedric

Subscribers: shilpasingh, jpeg

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

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

evil: remove strrstr and ffs - not used in the EFL on Windows

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

efl_animation: Fix to set Efl_Event_Description for event animation

Any event represented by Efl_Event_Description can be registered for
event animation. (e.g. EFL_GFX_EVENT_SHOW, EFL_GFX_EVENT_HIDE, etc.)

eolian gen: generate documentation for first object param if present

This prevents doxygen from emitting warnings.

Fixes T6186.

elm_toolbar: use prepare to get focus into the items again

efl_ui_focus_object: introduce prepare_logical

with this call a registered logical item could prepare itself for a
deeper traversal

efl_ui_focus: add parent_provider

thats just a little helper, where the logic to find and fetch the
provider is bound to the position in the widget tree, this means that
for example gengrid could change the way the logical parent is
evalulated. (For example to map the logical parent to a item)

efl_ui_focus_manager_calc: make sure to not access out of bounds mem

tests: eina strbuf - fix compilation

refer patch 2cf24eb30428fd56

eina: strbuf - Add strftime related functions

eina_strbuf_append_strftime()
eina_strbuf_insert_strftime()
eina_strbuf_prepend_strftime() - macro

We need these functions for implementing generic format function
interface especially for calander.

Ref T6204

efl_animation: Fix shadowing local variable warning

evas: Fix crash when object is not fully created

This fixes make check.

evas: Fix C++ compilation

Efl.Animation and Efl.Canvas.Object need each other, and introduce a
cyclic dependency. Eolian doesn't complain... but C++ fails to compile,
as one header must be included before the other, and vice-versa.

Do we have other cyclic dependencies? I remember we lifted the
limitation in eolian itself, but can't remember exactly how it should be
handled...

Ping @q66 @felipealmeida

edje: Fix make check

Oops. The API was tested.
See 19dff855194f2c4b411fc5d27b33d212ff26bcb0

eo: Simplify debug_name_override

Simply pass in the strbuf and don't expect the callee to own it. This
makes things simpler and safer (it'll crash only if the callee frees
said strbuf, and shouldn't leak). efl_ebug_name is new in the upcoming
release, EFL 1.21.

Realised this after talking with Amitesh. Thanks.

See 999dbd9764426890c0e11841358f9219082b89b2
And c4769ff8989bab2b745017b843f073e5737e91e0

ecore_wl2: Synchronize state with surface commits

This is really several inseparable commits mashed together, as doing this
a piece at a time would introduce broken intermediate revisions.

Double buffer incoming "configure" state from the compositor so it's held
back during asynchronous render and processed at frame completion.

Hold off on certain requests if their API has been invoked during async
render.

This should fix a lot of races, cosmetic issues, issues where weston can
kill our clients for acking configure (or not) at bad times, etc.

ecore_wl2: Don't overwrite saved geometry if switching fs to max

If we switch from fullscreen to maximized we need to keep the pre-fs
geometry around in case we ever unmaximize AND unfullscreen.

ecore_wl2: Don't overwrite saved geometry if switching max to fs

If we switch from maximized to fullscreen we need to keep the pre-max
geometry around in case we ever unmaximize AND unfullscreen.

ecore_wl2: Push out maximize set with a display flush

We want this request to get to the compositor as soon as possible.

ecore_wl2: Push out fullscreen set with a display flush

We want this request to get to the compositor as soon as possible.

wayland_dmabuf: Clean up unused variable

Pending is useless now that we use immediate buffer creation.

wayland_dmabuf: Use create_immed

We've really always needed to do immediate creates.  On a surface resize
there's no place to wait for the round trip for the new buffers to exist.

We've gotten away with this until now by good luck because we dispatch
wayland events during asynchronous render.  However, with async render off
or if schedule happens in an unfortunate order, we can end up with
tearing.

ecore_wl2: Bind dmabuf version 2

Version 2 allows for create_immed requets.  No new events are added until
version 3, so this is a trivial update.

ecore_wl2: Fix setting empty input regions

We should still update window->input_rect when the input region is empty.

ecore_wl2: Fix setting empty opaque regions

We should still update window->opaque when the opaque region is empty.

ecore_evas_wayland: Use ecore_wl2_false_commit to trigger frame callbacks

This will allow us to receive a tick without changing any other state.

ecore_wl2: Add as partial commit function for setting off a tick

This adds the concept of a "false commit" that just sends a surface
commit without changing any other state.

This is intended to be used by ecore_evas to request a frame callback
from the compositor

ecore_wl2: Add new API to latch window state for updates

Add an API to call at the beginning of render to latch state at
that moment, and use it for the wayland engine.

ecore_wl2: Split configurations into client set and compositor requested

This should lead to clearer and more maintainable code than having one
pile of state that's asynchronously fiddled with by both client and
compositor.

ecore_wl2: Move some state into its own struct

These need to be double buffered, and that will be done in a follow up
patch.

ecore_wayland: Use ecore_wl2_window_resizing_get() to get resizing

Use the getter instead of directly accessing the structs.

ecore_wl2: Add a getter for window->resizing

ecore_wl2: Fix some @since tags

Seems my brain missed the efl release and started tagging new API
incorrectly in the doxy.

This is all beta API that should probably only be used by other EFL
internals anyway, but I suppose it's a good idea to try to be somewhat
correct.