ecore-con: Mark unused function paramater as unused

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

Canvas layout: fix Efl.text (non-legacy) TEXT parts

Current implementation should also support TEXT parts (like button)
with usage of efl_text_set.

Follow-up to 0061a4bb349e6c8e122913791bf83f7f40b0cede.

Windows: remove XP support and forbid compilation and run on this platform

Test Plan: run on XP

Reviewers: jpeg, cedric, ajwillia.ms

Reviewed By: ajwillia.ms

Subscribers: ajwillia.ms

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

Ui layout: handle Efl.Text legacy code better

This should cover the cases without need to duplicate a lot of code.
Follow-up to deca3c947ff1a2bed3f4baadce576105055af266.

Canvas layout: improve legacy code paths for Efl.Text

Better handling of specific cases where you want to efl_text_set on
a TEXTBLOCK part.
Follow-up to af1595fd0160f76de82cddeb2576143f65492c0b.

elm_code_widget: set appropriate background colour.

Canvas layout: fix legacy text set

Keep the legacy code path when using edje_object_part_text_set.
Fixes e's notification that got broken after
3642b3ae679e3bedba4155fb9e54b44483b3b9e1.

Also, limit new efl_text_markup set to TEXTBLOCK parts.

eina debug - add missing unlock in erite err case

fix CID 1381503

efl_ui_focus_object: also implement focus.get

evas_focus: skip focus set if the focus state is the same

ecore_con: simplify Efl.Net.Server.Ssl by inheriting from Efl.Net.Server.Tcp.

ecore_con: add an intermediate Efl.Net.Server.Ip

eolian gen: simplify generate_normal_free a bit

efl_ui_focus_manager_calc: correctly specify owned where its needed

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

This reverts commit bd6dcbb044ffea543f27cc69c421317bdea4a14a.

This was not fixing the issue, just ducktaping over a bug in eolian, the
bug is now probebly fixed in eo / eolian.

eolian: add tests for the new owning functions

eolian: generate free function for owned parameters

With this commit owned parameters are freed if the object function
cannot be called, this means bla(NULL,param) will not leak anymore

eo: introduce function definitions with fallback

there can be cases where the call to such a function is never going to
be directed to the actual implementation. In those cases the FallackCall
will be called.

edje entry: fill content info when text is deleted by imf event

Summary:
There was missing info when text is deleted by _edje_entry_imf_event_delete_surrounding_cb().
@fix

Test Plan: N/A

Reviewers: kimcinoo, raster, cedric, jpeg, herdsman

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

evas font: check returned value of FcPatternGet function

Summary:
Evas should add font when only FcPatternGet function gives matched result.
@fix

Test Plan: N/A

Reviewers: raster, cedric, herdsman, jpeg

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

efl ui textpath - remove more unused vars

elm test: rename test_slider_interval.c to test_ui_slider_interval.c

efl ui textpath - remove unused vars due to dead code removal

fix warning

evas cserve - fix validation of cserve mesg size field

fix coverity CID 1039567

evas cseve - check msgsize before using - coverity

fix CID 1039568

elm atspi - fix checking return of eldbus_message_iter_arguments_get

fix coverity CID 1353590

elm config - fix eina value setting to check get return

fix CID 1357139

evas ector gl - fix leak of image data on data put

@fix

fix coverity found leak CID 1374645

evas ector gl - make coverity happy about leaks on data put

fix CID 1374646

eina debug - check write return values and complain on failure

coverity - fix CID 1377521

eina debug - check write return value and complain

makes coverity happy - CID 1377524

efl debugd - check return of write func and report errors

makes coverity happy and reports write errors... fix CID 1377525

efl model - fix checking return of eina_accessor_data_get

coverity - fixes CID 1380995

eio: do not copy memory for nothing.

eio: use TIMESTAMP when meaningful.

eio: migrate efl.io.manager.ls to use Eina_Future.

ecore: allow other thread to write local data on a thread to increase utility.

elm widget - event legacy propagation - remove coverity warning

fix CID 1379920 - event_flags is actually never NULL or undefined in
the function logic. it's always set to point to the specific event
flags field depending on struct type or the function will return
before using the pointer.

efl text markup - fix hex check to be or not and - found by coverity

fix CID 1381491

efl ui focusable - remove pointless check of ptr already checked

coverity fix CID 1381492

eolian: fix tests after extra checks

eolian: validate inner types of complex types

This makes sure the inner types are all correctly defined and
that freefuncs are actually correctly filled.

eolian-cxx: Add optional parameter to lambda for unnamed proxy instantiation

ci: disable examples build for osx for now

On the 10.12 build we regularly run into the maximum time limit for a
osx build job (45min?). Lets focus on the normal build for now.

Ui layout: support markup for efl_part()

Also implement markup_set/get for:
  - Efl.Ui.Frame
  - Efl.Ui.Slider

Users may choose between text_set/get and markup_set/get, depending on
whether they want to escape their text or not.

Canvas layout: support markup for efl_part

Users can now do either:
   efl_text_set(efl_part(obj, "part"), "text");
   efl_text_markup_set(efl_part(obj, "part"), "text");

Also have efl_text_get/markup_get.

Using markup_set will allow to choose whether to set a markup or a text
to the text part.

Canvas text: use EINA_UNUSED for unused variables

Nothing special here. No need to for the "(void) eo_obj".

Ui text: add test for Efl.Text.Markup

Efl text: add Efl.Text.Markup interface

Adds basic markup operations for text objects, and implements it in
Efl.Canvas.Text.

Also, this adds the Efl.Text.Markup_Util class.
This utility class allows basic text-markup conversions.

@feature

ci: README: simple docs for the current CI setup

Describe what we currently do in terms of building on Travis and how
developer can do a local test with the docker build.

ci: initial travis CI support for Linux distros and MacOSX builds

This has been used by myself in a branch for a while now and it is time
to bring it into master as a base for all future CI related work.
I plan to use the same scripts and other bits for Jenkins as well as
other CI systems later on.

What we currently cover with this setup are linux builds for three
different distros and MacOSX builds for two different versions.

Travis will only be called when new commits get mirrored onto our GitHub
mirror (which only happens once an hour). Expect delays on these builds.

https://travis-ci.org/Enlightenment/efl

elementary: atspi - fix dbus abort

This patch set fix following abort occured on dbus side.

(gdb) bt
0   __GI_raise (sig=sig@entry=6)
1   __GI_abort ()
2   _dbus_abort ()
3   _dbus_warn_check_failed
4   dbus_message_iter_append_basic
5   append_basic
6   eldbus_message_iter_basic_append
7   _bridge_signal_send
8   _text_text_removed_send
9   _bridge_accessible_event_dispatch
10  _elm_interface_atspi_accessible_event_emit
11  elm_interface_atspi_accessible_event_emit
12  _entry_changed_user_signal_cb

(gdb) f 12
(gdb) p *$1
$2 = {change = {insert = {content = 0x0, pos = 0, plain_length = 1},
                del = {content = 0x0, start = 0, end = 1}},
      insert = 0 '\000', merge = 0 '\000'}

The following patch set would fix the root cause of this problem.
https://phab.enlightenment.org/D5240

efl: Remove @owned tag from pack/content APIs

Refer to the previous commits for more context.

When an evas object is given to a container, be it with a variant of
efl_pack() or efl_content_set(), its "ownership" is given to the new
parent container. But ownership here means that the new container may
delete the child when it sees fit (i.e. when the container itself dies,
for instance). This does not mean that a reference was passed from the
calling context to the container. The actual EO owner of the child is
always the canvas or another canvas object, even if it the object is
unpacked.

Note: This means that invalid calls to efl_pack or efl_content_set will
not automatically delete the child object. This is the same as legacy,
and results in floating objects. Just check the return value! :)

Hopefully this is correct for bindings.

tests: Use efl_unref() on duplicated event objects

efl_del() is valid but bindings such as C++ should be using efl_unref()
and not efl_del() in this situation: a reference was given, so it should
be released.

evas: Give a parent to duplicated event objects

This comes back to an earlier situation where the fake event objects
created by efl_input_dup() had a parent (it should be the evas or maybe
the window). But that led to issues as efl_unref() would print error
messages. Using efl_allow_parent_unref_set() we hide the error message,
because in this situation we want the caller to use efl_unref() and not
efl_del() (eg. think of bindings...).

See d3436af616319f1fdbbd4a0aa71a7835a14699ac
See 6bed255fc4bbe1b30c960339e8dda2fc6e7925cd
See 96d94e0076773d6b29107035dc61f5df01a3347f <-- main commit

eo: Add API to allow deletion by unref

This allows deleting an object by simply calling efl_unref() on it, even
if there is a parent. Normally the parent is in charge, and you can
request deletion by calling efl_del() or efl_parent_set(NULL).

But in some rare cases, you want to give ownership of an object (@owned)
and still give a parent to that object. efl_unref() should be used (and
would be used by bindings when going out of scope or on garbage
collection), which would then print an error message. This API allows
the specific behaviour.

@feature

eo_debug: Allow disabling of backtraces

export EO_LIFECYCLE_BACKTRACE=0

This means that the lifecycle can be tracked but we don't care about the
backtraces. Related to some recent eo_debug patches.

eo: Allow efl_reuse to be called with a parent

If an object still has a parent inside the del intercept, we shouldn't
reset the "parent_sunk" flag. This would indeed break logic as
parent_sunk == false implies that a parent was never set, which means
the parent must be null. Right now this case isn't used but it can be
imagined with caches of evas objects (they should remain parented to
their evas).

@fix

ecore wl2 - fix null checing of drag offer - coverity

fix CID 1378629

ecore evas - wayland - avoid divide by 0 if height is 0

fixes CID 1379247

elm widget - fix possible null ptr access found by coverity

_propagate_event_legacy checked event_flags ptr for null then just
derefs it anyway... so check everywhere. fixes CID 1379920

efl ui focus - fix coverity complaints about derfer before null check

indeed there was a child cnull check after dereferencing the child...
so check completely. fix CID 1379925

efl ui win - quieten coverity complaint about dead code

the count ?: 1 check is pointless as count is already checked above.
make it clearer that it's > 0 and remove the ? check. silence for
coverity CID 1380542

efl ui text - remove logically dead code

it is indeed pointless code. fix coverity CID 1380544

edje - fix check return of _edje_fetch

fixes CID 1381326

efl ui text - fix resource leak of strbuf and pointless buffer code

edje_object_part_text_style_user_push() doesnt need you to
stringshare_add a string... some other code did it because it "gets"
the string from an obj that might free/delete the string on push etc.
or before so it keeps a copy but then it stringshare_del's it
anyway... the _textpath_ellipsis_set() code creates a trsbuf that
wasn't freed if enabled was false and it never stringshare_del'd the
string is stringshare_added... all of this was pointless code anyway.
you could just pass the constant string directly.

fixes CID 1380543

evas font loading - fix possible divide by 0 when font size is 0

fix coverity CID 1381446

eo: Fix unnecessary ERR logs with eo_debug

efl_data_scope_safe_get() is often used to assert that an object is of a
certain type, but assuming it may be NULL or not of the required type.
This means that encountering an invalid type is an error handled by the
caller (we return NULL), and shouldn't print out extra ERR() logs.

This fixes issues with E run under eo_debug. E's code was safe as it's
using evas_object_smart_data_get() and verifies whether the returned
value is NULL (which is expected for a rectangle, for instance).

@fix

tests: Fix leak of input event & devices

After the previous patch, the caller of efl_input_dup() clearly owns the
reference on the returned object, which means she must absolutely delete
or unref if manually.

Note that the previous patch changed the bug from use of invalid eo
pointer to leaking of objects. But the only way to support bindings with
something like dup() is to ensure we give a ref to the caller, and thus
the parent should be null.

Note: eo_debug was used extensively to reach this point.

evas: Fix dangling references with input devices

This solves issues with efl_input_dup() which didn't properly give a
reference to the caller, resulting in dangling eo ids.

Note: This may trigger leaks (instead of invalid refs), but this now
actually reflects the meaning of @owned. This should work with bindings
and C API users should know to call efl_unref().

This patch is the reason for the previous improvements on eo_debug.

@fix

eo_debug: Also print out the current backtrace

We can't use EINA_LOG_BACKTRACE=1 for that as eo_lifecycle itself prints
out ERR messages for its backtrace dump.

eo_debug: Enable logging of all ref/unref

This is done via the syntax (not documented yet):

  EO_LIFECYCLE_DEBUG=N:<class names>

Where N is:
 1. new/free
 2. new/free/ref/unref
 3. new/free/ref/unref/reuse

@feature

evas : Improve the performance by reusing map surface

Summary:
If object just has update of map point without other changes included child,
the map surface can be reused.

Reviewers: jpeg, cedric

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

tests: Improve test case for win/evas event refeed

intv slider: use scope_get instead in finalize()

eio: convert Efl.Io.Manager.stat() to return Eina_Future.

eina: add an helper to setup Eina_Value structure description.

This is particularly useful on system like Windows who doesn't allow
setting pointer at compile time in static const structure.

emile: unmap the memory for image file after decode.

Summary:
In case of emile, unmap the memory of the image file when the image file is closing.
But closing of image file happen when the image is flushed from the cache, so the
mapped memory of image file remain for long period of time with no benefit (If it is
in cache, it won't get decoded again). So I think we don't need to keep the mapped
memory of image file.

Test Plan: self

Reviewers: cedric, jpeg, jypark

Reviewed By: cedric

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

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

elementary: avoid segfault when getting focus due to an infinite recursion.

This was two line switched and setting the callbacks on the wrong object.

ecore: we can actually inherit from Efl.Object, just not in the first position.

eina: remove EFL_HAVE_WIN32_THREADS define as it is unused since winpthread is used

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

elementary: remove useless fprintf.

elementary: actually implement the terminate event.

ecore: we need a different event for tracking the lifecycle of the main loop.

If the main loop is destroyed a lot of companion information might already
have been so relying on EFL_EVENT_DEL might be way to late in the shutdown
process. terminate will be triggered as soon as the process of shutting down
is started, so all the data are still valid and have not been destroyed yet.

elementary: don't crash if API is called after shutdown.

elementary: restore input propagation and destroy array properly.

efl_wl: Stop queuing buffer release events

This was an optimization that caused more pain than benefit in the
long term.  Weston no longer does it, so we're just following suit.

examples: Fix C++ bg example

This is not nice yet... part API needs to be improved in C++.

elm: win part EO files are public

elm: Fix make distcheck

See ebff5e8639851ab9d80fa6ff36f20f08952c015e

evas textblock: use main font's ascent/descent as minimum value

Summary:
The patch will adjust ascent/descent with considering main font's ascent/descent
as minimum value. Actually, this patch re-apply D5217 with resolving linegap issue
on the _layout_format_ascent_descent_adjust() function.
@fix

Test Plan: Put a text which will load smaller font. And put a text which will load primary font.

Reviewers: raster, cedric, herdsman, jpeg

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

win: Remove invalid callbacks

Faulty commit (guess who?):
  596dba5fe5fa8b4101f50bbed66c29c791a2572f

See also d567faa7787afb3d478f1c857cb28ab4fa0e5fc3

Finally make check works again. That was... a painful lot of patches to
fix.

Revert "Fix infinite recursion for legacy + efl ui win events"

This reverts commit d567faa7787afb3d478f1c857cb28ab4fa0e5fc3.

This is not trying to fix the root cause for the infinite loop.

elm: Fix shutdown of atspi (unregister handlers)

@fix

ecore: Reset do_quit when ecore shuts down

After ecore_shutdown the main loop is dead, so the flag do_quit can be
safely reset to 0. This will fix issues with cycles of
elm_init/shutdown. This fixes:
  CK_FORK=no tests/elementary/elm_suite elm_win

This patch relies on a few of the previous patches which ensure that
ecore is well shut down.

@fix

elm: Delete atspi root object on shutdown

Ugly implementation but this will do.

@fix

eo: Fix crashes with call cache after init cycle

It's not enough to check the init generation count, if the cache data is
not reset properly.

This patch adds safety in two ways:
 1. Actually reset the op call cache when we detect a new generation
 2. Verify that we don't return NULL function pointer

The 1st point is absolutely necessary, 2nd is optional (extra cost of
one if()).

@fix

ecore_ipc: Unregister log domain on shutdown

An old comment mentioned that this was explicitely not unregistered, but
I believe the comment isn't valid since ecore_event_type_flush() was
introduced.

elm: Fix elm_shutdown

ecore could not shut down properly in an elm_init()/elm_shutdown()
cycle, with 7 remaining references, all because of a typo.

This should help @cedric as well

win: Avoid calling same function twice on shutdown

This avoids calling:
  ecore_evas_callback_delete_request_set
  ecore_evas_callback_resize_set
twice when deleting a window. Also adds safety over sd->ee.