evas-gl-common: Fix dereference after null check

Coverity reports that there may be a null pointer dereference here so
check that 'error' exists before trying to set it.

Fixes CID1374272

@fix

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

eolian-bin: Free previously allocated eina_strbuf

Coverity reports that the eina_strbuf 'param_call' leaks when it goes
out of scope here, so fix the leak by freeing the strbuf

Fixes CID1381502

@fix

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

evas: remove image_load from the engine.

evas: remove use of engine image_load from Evas.Canvas3D.Texture.

evas: remove duplicated code and rely on Efl.File { get; set; }

We can almost remove image_load from the engine backend after this patch.
One little bit left in Evas_3D.

edje: remove duplicated code and rely on Efl.File { set; get; } mixin.

efl: make Efl.File a mixin that implement file_set for everyone.

eina: Eina_Prefix should never contain a NULL, no code expect it.

eina: allow eina_file_close(NULL) to simplify code using Eina_File.

elementary: update code highlighting to follow eolian syntax change.

ecore-evas-drm: Implement support for hardware output rotations

This patch adds support for doing output rotations via hardware. This
is implemented inside the Ecore_Evas engine so that it is transparent
to the caller of ecore_evas_rotation_set.

ref T5999

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

ector: minor grammar fixups

Reviewers: cedric

Subscribers: jpeg

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

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

edje: moved from edje_multisense.c to Ecore_Audio.h for select the type of render.

Reviewers: NikaWhite, raster, cedric, vtorri, rimmed

Reviewed By: NikaWhite, vtorri

Subscribers: artem.popov, cedric, jpeg

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

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

ecore_audio: a minimal template for playing sound on WINDOWS is added

Summary:
Realization of audio data playback through WASAPI (Windows Audio Session API).(minimal)
WASAPI model:
1. Find  a playback device (in the default system)
2. Register the client on the playback device.
3. Create a playback format for audio data.
4. Initialize the client with the created format, and access mode,  ....
5. Take the object-rendering from the client for play the data stream.
6. Play data stream

ecore_audio model:
1. create a object for play the data stream(out object)
2. create a object to receive the data stream(in object)
3. register in the out-object the in-object
4. play data stream

Necessary:
Realize the ecore_audio object to play the data stream using the WASAPI model.

How implemented:
1. The object ecore_audio_out_wasapi is implemented
2. object ecore_audio_out_wasapi - the object constructor is find a playback device (in the default system)(WASAPI)
3. _ecore_audio_out_wasapi_ecore_audio_out_input_attach -  register in the out-object the in-object
   3.1 Register the client on the playback device.(WASAPI)
   3.2 Create a playback format for audio data.(WASAPI)
   3.3 Initialize the client with the created format, and access mode,  ....(WASAPI)
4. _write_cb - play data
   4.1  Take the object-rendering from the client for play the data stream.(WASAPI)
   4.2  Play data stream(WASAPI)

Reviewers: cedric, vtorri, raster, an.kroitor, NikaWhite, FurryMyad, rimmed, t.naumenko, Jaehyun, bowonryu

Reviewed By: vtorri, NikaWhite

Subscribers: artem.popov, cedric, jpeg

Tags: #windows

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

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

elput: Fix issue of pointer rotation not being set on startup

elput_input_pointer_rotation_set may be getting called before input
devices are actually added, so we need to be able to set pointer
rotation when a device actually gets added.

To fix this, we will store the requested input rotation value, and
apply it at the time of input device creation.

NB: The current case is that the wl_drm module will apply existing
output rotation before Input has completely initialized, thus any
calls to set the pointer rotation there will not succeed as input
devices have not been created yet.

@fix

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

ecore-file: do not emit events when an inotify watch is removed

???????????????????????????????????????????????????????????????????????????????

@fix

elementary: Mark unused function parameter as unused

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

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.