cxx: Mark wref::lock as const

This allows passing in wref to a lambda.
If we passed normal refs, we would end up with dangling references and
many ERR messages.

cxx: Add nullptr constructor for weak refs

This allows constructs such as:

  wref_obj = obj;
  ...
  wref_obj = nullptr;

cxx: Completely rewrite box example

This was showing off box transitions, which don't exist in EO API.

cxx: Fix a few examples

 - Calendar: Some examples can't be ported. Not good.
 - Toolbar: Needs the new API to be completed.
 - Clock: Crashes at runtime.

Ping @felipealmeida

cxx: Instantiate objects with 2 refs

This is the original meaning of efl_add_ref().
Somehow the "ref" bool was set to true in the first call, and false in
the second call.

cxx: Place beta API declarations behind ifdef

NOTE: protected APIs are placed behind ifdef as well in the
implementation file. This makes sense since the define is required for
the C code to compile, but this isn't what @protected means.

docs: be better at a responsive layout when embedded

edje_pick: Fix segfault due to wrong memory free

evas-object-main: Fix potential null dereference

Coverity reports that EVAS_OBJECT_DATA_SAFE_GET returns NULL here (184
out of 188 times). As such, we should not be calling
_event_animation_object_get with a null object so add a null check here.

Fixes Coverity CID1381713

@fix

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

evas-object-main: Fix null dereference

Coverity reports that EVAS_OBJECT_DATA_SAFE_GET returns NULL here (184
out of 188 times). The _all_animation_objects_cancel function
directly dereferences the obj parameter passed into it, so lets null check it
before calling that function.

Fixes Coverity CID1381711

@fix

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

evas-object-main: Fix null dereference

Coverity reports that EVAS_OBJECT_DATA_SAFE_GET returns NULL here (184
out of 188 times). The _all_animation_objects_cancel function directly
dereferences the obj parameter passed into it, so lets null check it
before calling that function.

Fixes Coverity CID1381710

@fix

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

emotion: Fix data race condition

Coverity reports that we access vfmapped here without holding a lock.
This patch implements eina_lock_take/release while accessing
priv->vfmapped.

Fixes Coverity CID1381624

@fix

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

evas-wayland-shm: Do not dereference a potentially freed pointer

Coverity reports that _evas_dmabuf_buffer_init function here can
potentially free the surface that was passed into it. If that happens,
we should not be calling the _fallback function with surface as the
parameter as that will directly dereference the freed pointer.

Fixes Coverity CID1381707

@fix

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

evas-gl-cocoa: Remove need for useless region_free function

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

evas-gl-drm: Remove need for useless region_free function

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

evas-gl-sdl: Remove need for useless free_region function

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

evas-gl-x11: Remove need for useless free_region function

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

evas-wayland-egl: Remove need for useless free_region function

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

evas-wayland-shm: Remove need for useless free_region function

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

evas-software-x11: Remove need for useless free_region function

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

evas-software-gdi: Remove need for useless free_region function

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

evas-software-ddraw: Remove need for useless free_region function

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

evas-eglfs: Remove need for useless free_region function

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

evas-drm: Remove need for useless free_region function

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

evas-software-generic: Check for render engine function before calling

In most engines which inherit from software_generic, they do not
implement the outbuf_free_region_for_update function. Most engines
have it as an unused function. If we simply add a check here, then we
can reduce the need for having useless function in multiple engines.

@fix

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

efl_ui_calendar: apply efl_ui_focus_composition

evas textblock: handle ellipsis when text's height exceed its area by "br"

Summary:
Textblock's ellipsis feature only worked when text's width exceeds its area.
So, it didn't work when text's height exceeds its area by "br" tags.
This patch will do ellipsis when only ellipsis=1.0 is set.
@fix

Test Plan: make check

Reviewers: herdsman, raster, cedric, jpeg, sohyun

Reviewed By: raster

Subscribers: woohyun

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

gitignores - update for new pc files

edje: don't return negative width and height from _parts_extends

Summary:
It should return width and height with positive values or zero.
@fix

Test Plan: make check

Reviewers: raster, jpeg, cedric

Reviewed By: raster

Subscribers: jiin.moon

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

Adding danish translation

eo: fix typo.

Thanks Gustavo.

edje_cc_script: remove build warning

Summary: initiate pointer var with NULL to remove build warning

Reviewers: conr2d

Subscribers: cedric, jpeg

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

eo: add helper to manipulate Eo object as Eina_Value.

eo: refactor use of efl_replace.

eina: add a function return a reference to a copy of Eina_Value.

elm_code: Trying some markdown syntax, this is complex

elm_code: Trying some markdown syntax, this is complex

edje_recc: don't use negative exit code in a /bin/sh script

Apparently this isn't well supported by dash, which will print an
error and return a 2, where zsh and bash will return 255.

Explicitly returning 255 seems least surprising.

see https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=772322

 #IHaveNoIdeaWhatThisScriptDoes

A more readable layout for doxygen landing page

Also moved out the C++ reference as that should be separate.
(C++ link now on the main /docs page)

eolian: disallow ptr() on things that are already pointer-like

This disallows deeply nested pointers, you can only explicitly
ptr() on types that are strictly value types.

For a few cases where it was necessary to override this behavior,
you can use legacy(ptr(x)) as a temporary measure.

ecore_evas: always find a source for ticking legacy animator.

As we do not know when a window won't be able to tick, and we do
not know which window a legacy animator is attached to, we require
all windows to tick as often as they can and only generate one tick
per loop run.

This might need more adjustement especially with multi output.

Correct struct labelling

Avoids linking every occurrence of 'The'

cxx: fix application build error

gesture files are not found because the include path is not supplied by
pkg-config.

eo files: convert various to use @cref

efl-ui-focus-manager: Remove set but not used variable

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

elm radio: fix build of things that don't enable eo APIs

docgen: better space out includes

docgen: fix generation and generate notes correctly

eolian: fix tests following the container restrictions

eio: properly mark array contents as ptr

eolian: only allow reference (ownable) types in containers

Update my name in more places

Update auto API doc styling to match main website

Not perfect, but it's a good start

elm test: panes - revert the code

test_panes.c code was added into previous patch by accident.

Efl.Ui.Radio: remove value{} api

Efl.Ui.Nstate already implements value{}.

@fix

ecore_evas: if an async rendering is still going on, there won't be a new one starting.

ecore_evas: do not propagate drawing change blindly from sub surface to parent.

It is not because a round of rendering happen for a child, that it result in
actually drawing anything in the parent. The parent will always be aware of
the rendering request of the sub surface and we should only track what the
parent think.

T6049

Tested-by: Derek Foreman <derekf@osg.samsung.com>

efl_ui_radio: remove some legacy code

Summary:
This uses constructor/destructor instead of group_add/group_del.

Note: finalize can't be used for theme loading as any action done
inside
efl_add(...) would be lost (eg. part text set).

Test Plan: 1) run elementary_test -to radio

Reviewers: jpeg, woohyun, cedric

Subscribers: akanad

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

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

evas-wayland-shm: Fix formatting

Make this line readable in 80 columns

NB: No functional changes

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

evas: Minor formatting fixes

NB: No functional changes

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

evas-drm: Fix formatting

NB: No functional changes

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

efl_ui_focus_manager: remove strange naming of a function

this is just normal next or prev, the prepare is done on the caller side
now

efl_ui_focus_manager: improve debug informations

ecore-drm2: Fix issue of not being able to set output mode

If we pass in screen geometry here when trying to set an output mode,
we can encounter "out of memory" errors from libdrm with outputs
that have a high resolution. As it turns out, we should be passing 0, 0 for
the x/y values when trying to set an output mode.

@fix

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

efl_ui_win: revert accidental override remove

elm: mark access interfaces as beta

Subscribers: cedric, jpeg

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

elm: Major cleanup of EO files

This prevents legacy EO classes from being exposed through .eo.h headers
or .eo in share/eolian/includes. Also removes a slew of useless xxx_eo.h
intermediate headers.

Notes:
 - elm_systray has no proper API: it's not clear if the EO API should be
   released (in which case it needs to be renamed to efl_something) and
   there is no legacy API to create a systray object.
 - Some files have been placed in a "FIXME" section, as I believe they
   are necessary within EO land, but at the same time still don't
   conform to the interfaces (eg. name starts with elm_).
 - elm_interface_scrollable is required by photocam. This means photocam
   needs to be adapted to fit the EO scroller API (still to be
   completed, I believe).

Bugs:
 - This breaks most C++ examples. I KNOW. And I'm working on it.

Ref T5301

eolian: add part validation (including dup check with funcs)

elementary: use @cref where possible

eolian: add @cref as alternative to @in

This is a "pass by reference to const" equivalent. There is no explicit pointer
and currently it's the same as ptr(const(x)) on the type. However, it is also
usable on properties.

Genlist Item Pin Feature

Summary:
**@feature** T6241

This feature enables genlist to pin an item to viewport which will
be available always for user to view/select.

**Use Case**:
In a big list of music, most times when user finds a song which they
like, before playing that they may want to go through the entire list
to check whether there is some other good songs, but
after seeing the entire list user have to again scroll back to the
position of item which they liked to play it then.
In this case item pinning can be used, so that the item
which they want to keep for future selection can be pinned
and then it will remain in viewport, finally when user want to do
operation on item, it will be readily available in viewport.

Signed-off-by: Godly T.Alias <godlytalias@yahoo.co.in>
Test Plan: Elementary Test -> Genlist -> Double click on items to enable/disable pinning

Reviewers: raster, cedric, prince.dubey, SanghyeonLee

Subscribers: rajeshps, jpeg, shilpasingh

Tags: #efl

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

Ecore_Imf: fix wrong annotation and enum name

Summary:
There are wrong annotation about version and wrong enum name
so fix that.

Reviewers: jpeg, cedric, akanad

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

Eet: fix wrong doxygen definition

Summary:
Eet_Image_Encoding and Eet_Colorclass is from enumeration in Emile,
but it's definition is typedef, not enum.

It makes broken link like below webpage.
https://docs.enlightenment.org/efl/current/group__Eet__File__Image__Group.html

In above page, Eet_Image_Encoding is not linked.

So doxygen documentation should be changed to typedef.

Reviewers: cedric, jpeg, myoungwoon, conr2d

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

Elm_Entry: Add annotation to unnamed enum

Summary:
There are unnamed enumeration in Elm_Entry,
so add annotation and names like D5376 (https://phab.enlightenment.org/D5376)

Reviewers: jpeg, cedric, myoungwoon

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

eina: Fix spelling errors in documentation (log..matrix)

Reviewers: cedric, ajwillia.ms

Subscribers: segfaultxavi, jpeg

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

evas: Move evas_canvas.eo to legacy-only

This isn't meant to be installed. The canvas API in EO is based around
the interfaces Efl.Canvas and the widget Efl.Ui.Win. Anything else is
not EO (eg: ecore_evas, evas, ...)

Note: evas_canvas3d is the last remaining thing that is installed along
EO files, but those are all beta APIs.

evas: Add seat argument to 'pointer_inside' (EO)

This merges pointer_inside and pointer_inside_by_device.
Affects only EO.

edje_cc: Fix shadow warning (global scope)

evas: Remove specific pointer_inside API

This is a name clash. It doesn't matter too much since Evas.Canvas isn't
meant to be exposed to the EO API.

evas: Remove erroneous EINA_UNUSED

'output' variable is actually used in this function, so remove
EINA_UNUSED from function parameter

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

efl_animation: Change protected methods to be internal methods

target_state_save, target_state_reset, target_map_reset methods are used
internally.
Therefore, those methods become internal methods.

eina: Fix spelling errors in documentation (file..lock)

Summary: Signed-off-by: Bryce Harrington <bryce@osg.samsung.com>

Reviewers: cedric, ajwillia.ms

Reviewed By: ajwillia.ms

Subscribers: segfaultxavi, jpeg

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

eolian: properly free parts

eolian: fill parts into db

edje_cc: move offset_scale to correct place and fix doc

eolian: Add some color to those warning messages

Note: This is not using eina_log, not entirely sure why, but this
doesn't prevent us from adding some color.

eolian: Skip @beta APIs for duplicate warnings

No warnings:
export EOLIAN_WARN_FUNC_DUPLICATES=0

No beta warnings:
export EOLIAN_WARN_FUNC_DUPLICATES=1

All warnings including beta:
export EOLIAN_WARN_FUNC_DUPLICATES=2

EOLIAN_WARN_FUNC_DUPLICATES is not an API and may change in the future
as we improve the tool :)

elm test: calender - use eina_strbuf_append_strftime instead

efl_ui_slider: add eina_value_flush after using the value

efl_ui_calendar: apply new format_cb

eo-cxx: Fix compilation error with any_value received by value

ecore_evas_wayland: Fix session recovery

Turns out when apps reconnect to the compositor they don't always
realize they need to redraw themselves.  Force a manual render
at startup if we end up in a state where an update is needed but
has probably been dropped on the floor.

efl-net: fixup copy & paste abuse.

edje_cc: fix parsing and documentation

This fixes parsing of properties in edje_cc and makes it more
consistent. All bool and color values are now parsed in same way.
Default values for many values are now given in edc_ref.
Some documentation blocks are moved to correct places.
Fixed mistakes in description of edc properties.

@fix

edje_cc: fix item.max parsing

It is now possible to set item.max back to default value (-1 -1).

@fix

edje_cc: make box/table item.spread minimum 1 1

Minimum value was 0 0, but it has same effect as 1 1.

edje_cc: fix program.filter description

@fix

edje_cc: fix text.align description

@fix

edje_cc: limit map.zoom to non-negative values

@fix

edje_cc: move box and table properties documentation

Documentation comments for properties should be near parser
implementation.

@fix