evas_vg_load_svg: get rid of static Evas_SVG_Parsing

Summary:
Refactoring.
It is good to store values from that struct in a parsing/loading context

static variable is a big NO NO:
1. Ugly code design,
2. Might not work when trying to load more than one SVG file.

@fix

Reviewers: jpeg, smohanty

Subscribers: jenkins, cedric

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

evas_vg_load_svg: default stop opacity color should be 255, not 0

@fix

vg_common: property apply "objectBoundingBox" morphings to radial gradient

When objectBoundingBox flag is set up, all gradients should morphed into
multiple gradients for every entity that has gradient to be applied on

svg_parse: parse correct values for radialGradient

Especially Radius calculation which was a huge pain to find formula
within SVG documentations!

vg_common: apply related coordinates for linear gradient

multiply it with object size or with view box sizes depending on
gradientUnits value

svg_parse: parse linear gradient variables accroding to gradientUnits

There are difficult cases according to
https://www.w3.org/TR/2000/CR-SVG-20000802/pservers.html

svg_parse: implement <length> parse

Accodring to https://www.w3.org/TR/SVG/types.html#Length
length ::= number ("em" | "ex" | "px" | "in" | "cm" | "mm" | "pt" | "pc" | "%")
This is still work in progress since some of lengths should be treated
differently, for example gradient lengths

svg_parse: save global size variable to use when gradient size recalc

For example while userSpaceOnUse it uses not figure's x,y,w,h
but entire canvas, so we need to save it publically and recalc while
parsing (or after)

vg_common: parse and store userSpaceOnUse (for gradient) when it is set

So it can be calculated during the draw later on

efl_gfx_path: fix typo in _path_bounds_get

Wrong position was used while going through the array of
points

@fix

vg_loaders/svg: set up default focal values for radialGradient

Since when not specified, focal values same as center points
fx = cx, fy = cy by default

@fix

ector: use cairo_pattern_add_color_stop_rgba in gradient_radial

It actually invoked SIGSEGV when trying to load SVG file with Radial
Gradient

@fix

saver/svg: parse loaded tree data into svg XML file and save it

Just base source code which will parse all structures of svg tree into
xml text to save current vector image into file

@feature

vg_savers/svg: empty (for now) module that will save svg in original file

Just as a starter to make a working background that, later on, will go
through Svg_Node's and build a certain source code to be saved in SVG
picture as a file

evas_vg_cache: load svg from any file, not from defined only

so it doesn't only load from eet or edj only
but also from those any extensions who is ACTUALLY eet, but named as,
for example, like, "file_with_svg.dev"

@fix

evas: Fix invalid header protector

clang is smart enough to warn about this.
I've mentioned it multiple times...

Remove evas internal dependency from the evas_font module

Summary:
dev branch : devs/subhransu/font

The Final goal is to move the evas_font module to ector so that both ector and evas can reuse the code.
make the api simple so that sam eapi can be used by evas_textblock and ector text.

This is the 1st stage to achive that gola, first remove the evas internal dependancy as much as possible before moving to ector library.

Reviewers: jpeg, raster, herdsman, cedric, id213sin

Subscribers: cedric, jpeg

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

elm: Try to fix compilation (link) for OSX

evas: Add some crash prevention checks

This could happen if objects still have references while evas is being
shut down (a bad situation).

tests: Fix make check for atspi beta APIs

See 5c997b3aaac6277b8c4965ee8e90d05e1959ac05
Ref D5365

cxx: (Temporary) fix for make check

This fixes the C++ compilation tests.

list<int> is not allowed by eolian anymore, as the semantics were
unclear whether a pointer to int was passed, or if the int was stuffed
inside the list by casting.

ptr(string) is also not allowed as it's a pointer to a pointer type.

Both of the above types are strange when it comes to bindings. While C++
could probably be made to work, it's not clear we could have such types
in other bindings. Those types are in fact likely not so useful, as most
APIs can be designed around those restrictions.

cxx: Fix log color in eina_log.hh

Changed from invalid "black" to arbitrary choice of green.

elm: Fix examples/tests using elementary_config

elementary_config.h should not even exist. It's been hijacked as a
private header for elementary, but all "real" configuration is stored in
efl's main config.h now.

cxx: Fix some examples, disable or remove others

make examples now builds all c++ examples but some of them are in fact
empty skeletons. Those either need some c++ love or the final eo api to
be ready (eg. menu, popup, ...).

I removed some examples that don't have an exact equivalent in EO since
the widget is legacy only.

cxx: Add beta define to Evas.hh

This is probably temporary as well, but it's necessary to make examples
compile (with rectangle).

cxx: Fix binding temporarily (efl/interfaces)

This uses the new cref keyword instead of inout, as the c++ code has no
access to the internals of struct Efl.Text.Cursor.Cursor

Something tells me cref may not be the perfect solution here (we lose
information on whether this is in, out or inout).

cxx: Fix ifdef protections in function impl

Mote: for protected we should use "protected:" if that is somehow
possible.

cxx: Add support for operator == with objects

cxx: Modify button example with wref

This is more of an experiment than anything else.
@felipealmeida I would like to know what you think.

Notes:
 - events still need a better API (event_add isn't part of the object
   definition...).
 - references are an issue, when you want to actually delete an object.

cxx: Add experimental "easy" way to use wref

I hid it behind ifdef for now as I'm very much unsure of what I'm doing.
This whole modern C++ thing is still weird to me :)

Prerequisite:
 #define EFL_CXX_WREF_EASY

This allows constructs such as:

  auto wobj = obj._get_wref();
  std::cout << wobj->text_get() << std::endl;

cxx: Add _get_wref() convenience to all classes

This allows constructs like:

  auto w_obj = obj._get_wref();
  auto cb = std::bind([w_obj]() {
    auto o = w_obj.lock();
    if (!o) return;
    o->call();
  });
  event_add(..., obj, cb);

Note: I don't like how those wref work. Close to c++ wref but far from
EFL wref.

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