ecore: remove doxygen warning messages

Summary:
remove doxygen warning messages
@fix

Reviewers: jpeg, cedric, woohyun

Reviewed By: jpeg

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

elm: enhance documentation for following files

 - elm_config.h
 - elm_focus.h
 - elm_genlist.h
 - elm_icon.h
 - elm_image.h
 - elm_image_legacy.h
 - elm_index.h

licensing - remove dates as they aren't strictly needed under common law

common law copyright establishes copyright at point of creation anyway
and there isn't a need to keep dates there. saves having to update them.

widget: Add FIXME note and cleanup whitespaces

edje: Use EO's beta auto_unref API for parts

This simplifies code and avoids unwanted leaks.

eo: Add beta API for auto_unref after a call

Before screaming in horror (C++...) here's why we may need this:
Efl.Part.part API returns an object that is by definition valid for a
single function call only. Enforcing this in practice is actually quite
hard as all implementation functions must manually take care of the
life-cycle. This is a lot of code in many places and a lot of
opportunities to forget to properly handle that life-cycle. Also, this
means any invalid function call on a part will leak an object.

This API absolutely must remain either "internal" or "beta" and
definitely not become abused by applications. On top of that such an API
can cause great trouble for bindings like C++. As a consequence, only
specially crafted APIs like efl_part() should return an object marked as
auto_unref.

Alternatively this API could be defined in Eo.h or some other
Eo_Internal.h. I placed it in efl_object.eo because it's much more
convenient :) (read: I'm lazy)

****

Performance notes:

Tested with clang & gcc (with -O2), I had a look at the output of perf
top, in particular the asm view. I used eo_bench in a loop. My
conclusions are:

- EINA_LIKELY/UNLIKELY actually works. The jump statement varies
  according to the expectation. I highly doubt all those ugly goto in
  eo.c / Eo.h are even useful.

- The impact of auto_unref on a call_resolve is so small it doesn't even
  appear in the trace. It is significant inside call_end, though
  (obviously, that function is just a few lines long). That function
  accounts for ~1% to ~4% of all CPU time. The impact of auto_unref in
  call_end is ~4% of the function time. This means ~0.16% of all CPU
  time (worst measured case). _efl_object_op_api_id_get simply doesn't
  show up because of caching, so the extra check there is negligible.

PS: I also tested EINA_LIKELY/UNLIKELY by compiling with -O2 and looking
at the output with objdump. The flag is well respected, and the jump
instructions are what you would expect (no jump for LIKELY and jump for
UNLIKELY). Conclusion: The goto's in eo.c only make the code harder to
read...

benchmarks: Fix titles in gnuplot

Just replace _ with \_ as _ means subscript.

edje: correct the usage of EINA_SIZE2D

The usage of EINA_SIZE2D macro as left hand side variable
does not save the size value. The Eina_Size2D should be used.

textpath: get the right size of text object

Recent change to use EINA_SIZE2D made the size be gotten incorrectly.
This corrects it.

eina: use a stringshare to store the filename internally.

T6164

eina: fix eina_hash_stringshared_new to actually compute the hash on the pointer.

Before this patch, the key would always be zero and the hash would solely
rely on the rbtree to be efficient. This improve the situation by using the pointer
as the key during hash computation.

eina: refactor eina hash computation of key length and hash value.

elementary: fix double assignment

Summary: local variable "bp" assigned twice unnecessarily. The duplicate assignment is removed.

Reviewers: raster, cedric, jpeg

Subscribers: jpeg, rajeshps

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

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

eina: add missing documentation for quad.

Summary: This adds missing doxgen documentation for eina_quad.

Test Plan: API Doxygen Revision

Reviewers: cedric, jpeg, myoungwoon

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

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

eina: add missing documentation for rectangle.

Summary: This adds missing doxgen documentation for eina_rectangle.

Test Plan: API Doxygen Revision

Reviewers: cedric, jpeg, myoungwoon

Reviewed By: cedric

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

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

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.