emotion xine module - fix minor complaints but not bugs

silence CID 1357156 , 1357172

evas module load - make coverity happy but not a bug

silence CID 1357364

edje cc mo encoding leak - ensure data is freed when setting up encoding

this should fix CID 1306604

elm_prefs_cc - clean up return with added brackets to be clear

it seems coverity gets confused with the sizeof(c) / sizeof(type). add
() hoping it will silence it. this is related to:

CID 1353600 and 1353599

elm_prefs_cc - fix correct struct size calculation

use correct struct type even though previous struct was the same size
and it wasnt an actual bug - but a potential one.

eio model - fix leak found by coverity

fix CID 1355012

gl generic ector leak - fix not using return value of eng_image_data_put

this fixes CID 1339963

@fix

ecotr gl - fix leak found by coverity

fix CID 1347412

elm map - fix coverity found leak in xml parsing

fix CID 1352814

elm map - fix mem leak in xml parsing found by coverity

fix CID 1352815

atspi bridge - fix leak on error found by coverity

fix CID 1353602

atspi bridge - fix leak on error found by coverity

fix CID 1353604

edje cc - fix leake found by coverity

fixes CID 1355586

edje cc - fix leak of name string on edje compilation time

fixes CID 1355587

edje edit - fix leaks spotted by coverity

fixes CID 1356630

elm cnp - wl - fix memory leak

fix memory leak specified in CID 1357160 , 1357159 , 1357158 , 1357157

elm: Fix cnp crash on WL

See D4144.

Original author: @slotus.lee

SEG_FAULT happens when the object which has selection is deleted,
and new selection is done at another object.
Reason: loss_cb is not removed when the object which has selection is deleted.
When new selection is set for new object, the loss_cb is called for deleted
object. As result, SEG_FAULT happens.
This issue was also happened in X11 (https://phab.enlightenment.org/D2763)

Test plan:
(on wayland environment) Run elementary test, open Entry, do selection,
close Entry window, open Entry one more time, do selection.

evas: Fix warning about unused function

I kept it here for the record. It was useful, could be useful
again.

eo: Fix warnins (return NULL instead of bool)

ecore: Fix warning about err_no

Double warning, yay:
 explicitly assigning value of variable of type 'int' to itself
 variable 'err_no' is uninitialized when used here

See 1abbfdd1f7b09ba9931657d57ffc4172c662c2eb

elm quicklaunch binary - fix possible 0 termination issue with strings

the input strings for agrs/env could be not 0 terminated tho the
sender guarantees it. ensure they are by 0'ing the last byte

fix CID 1353607

evas regions in render2 - work around coverity complaints

CID 1352392 is a false positive as the ptr is not dereferenced, but
not making coverity confused it good.

elm view list - just be less obviously a possible bug to coverity

CID 1355014 is a false positive, but it's god to not confuse it too
often. clean up.

emotion libvlc module - fix possible argv init issue with tokens

fixes CID 1357170

elm conform - fix leak on jproperty change

this fixes CID 1353606

@fix

elm layout - fix possible leak

fixes CID 1357365 (new!)

ecore - be paranoid about storing errno from select immediately

in theory another libc call could overwrite errno between select
exiting and errno being used for errors. be paranoid. i know of no
real bug that this causes though.

evas event handling4- fix more corner cases where bounding is bad

so smart object bounding box wasnt updated properly in several other
cases. fix those other cases too by dirtying bounding box region.

this continues on from:

f6b3c31561276a6c7afc8fb56ae2e5363772782c
25d77bc1d24d9fd539c681fa58db976c1ca65051
9f0fd66ab818d212fa88faef316ac17625f1a2f5

this fixes T4017

@fix

elm_panes : modified to keep panes content left size after orientation changed

Reviewers: woohyun

Subscribers: cedric, jpeg

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

edje_edit: on save_all firstly save all cached group, then all other

this will fix case when aliased group was changed and alias-group was loaded,
so then on saving alias-group WON'T rewrite all changed to aliased group

@fix

eolian: simplify enum generation logic

docs: much improved enum/struct/typedef C signatures

docs: support refs and new types in generator

elua: support is_restart for events in eolian api

data: fix last set of data files with executable bit

examples: edje: remove executable bit from svg files

These are plain data files, no need for an executable bit here.

examples: edje: make sure new svg files are packaged

In commit 0fc151cddc4cdb50661d11a124d42a9d42fa9875 these new svg files have
been added but forgotten to be made part of DATA_FILES.

elm: efl_ui_frame: add back last efl_ui_frame_eo.h header

During the rename this header got lost and distcheck broke for me due to
problems when generating the files from eo. I fixed a similar problem after
the rename ro efl_ui_flip in commit c3c344da4184ec3b790986c4b7d9008ade3dd9d6

Subhransu, please keep this in mind when renaming another one so I do not have
to do another of these fixes. :)

eo base - add weak object refs as per jpeg needs

so after some discussion with jpeg, weak referenced keys are also a
good idea. these need del track handling to be weak, so i made strong
reffed keys also do del tracking again as it's just nice to do this
and be more robust. also added and improved the test suites for this
key value stuff.

@feature

edje/svg: added _edje_svg_node_free() function

Reviewers: cedric, jpeg

Reviewed By: jpeg

Subscribers: cedric, jpeg

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

svg/loader: use stringshare instead of malloc for storing string.

Reviewers: cedric, jpeg

Subscribers: cedric, jpeg

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

ecore_drm2: Fix mouse pointer when using absolute coords

Running E wl2 in KVM led to the following issue: integrated mouse
pointer would always be stuck at (0,0).

The reason was that calibration would never happen, and it's
required* for absolute pointing devices, such as the qemu mouse
integration.

Fix: Listen to device add and calibrate based on the first
output. No idea if we could calibrate on any other output,
or how this should be done in case of multiple screens.

[*] I believe calibration might actually not be required, as
    the absolute position is already the correct one when
    received from libinput.

edje_pick: realloc bug fix

@fix

evas event handling3 - fix yet more corner cases for clipped objects

this is a continuation fix from
25d77bc1d24d9fd539c681fa58db976c1ca65051 and
9f0fd66ab818d212fa88faef316ac17625f1a2f5

this fixes yet more corner cases after the above 2 fixes. our clip
cache tracking code seems to be broken somewhere and not updating - at
least when events are processed so i did ti the slightly slower way
and recursed through clippers to figure it out in this path. it all
works now it seems but it's got a small speed hit. better be right
than a little faster.

@fix

svg/example: updated svg example with gradient svg files

Reviewers: cedric, Hermet, jpeg

Reviewed By: jpeg

Subscribers: cedric, jpeg

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

svg/loader: parse the style attribute in doc node

Reviewers: cedric, Hermet, jpeg

Reviewed By: jpeg

Subscribers: cedric, jpeg

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

Revert "elm_entry: remove unnecessary text clearing in text_set"

This reverts commit 38db4b4c0397ce599d5c50859d8431fbbdff284c.

This changes entry behavior that introduce enventor break.
Can't accept it until that break is reasonable.

svg/loader: refactored the copy of structure.

Reviewers: cedric, jpeg, Hermet

Subscribers: cedric, jpeg

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

elementary: use initialization information to initialize test application once.

ecore: on first arguments event notify the needs to initialize the process state.

evas: fix initialisation and shutdown of eet and eina by mesh loader and saver.

edje: fix parsing and and visualizing gradients.

Reviewers: jpeg, cedric

Subscribers: cedric, jpeg

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

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

elc_popup: move the code making buttons buffer for readability

Summary:
The code of making buttons buffer is away from related codes.
@fix

Reviewers: Hermet, Jaehyun_Cho, jpeg, raster, cedric

Subscribers: Blackmole

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

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

elm_calendar: fix first week is not changed

Summary: @fix

Test Plan:
1. elm_calendar_first_day_of_week_set(cal, ELM_DAY_MONDAY);
2. First week string should be changed to "Mon"

Reviewers: tasn, cedric

Reviewed By: cedric

Subscribers: cedric, jpeg

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

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

elm_entry: remove unnecessary text clearing in text_set

Summary:
When text_set is called for entry, it sets to empty text
before calling _entry_text_append().
but, _entry_text_append() has 'set' parameter.
Therefore text is set in that function, and clearing is not needed.

It can fix "changed" smart callback is invoked multipe times
when a short text is set to entry.

In addition, if text length is larger than ELM_ENTRY_CHUNK_SIZE,
set text directly as amount of chunk size and idler will append
about remain text.

@fix

Test Plan:
1. elementary_test -to "entry5"
2. click "set 10000" button
2-1. changed message should be printed 1 time
3. click "set 10001" button
3-1. changed message should be printed 2 times

Reviewers: tasn, Hermet, id213sin, cedric

Reviewed By: cedric

Subscribers: woohyun, cedric, jpeg

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

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

elementary: clear callbacks before destroying object to avoid crash.

@fix

T3188

efl_ui_image: fix documentation

efl_ui_image: fix documentation

elm_icon: Mark lookup_order functions as deprecated

I can't mark the enum itself as deprecated as it's used in too
many legacy APIs.

Fixes T3910 (@jypark)

efl: Rename "pointer" device class to "wand"

Because I like magic. And to avoid confusion with mouse pointer.
Suggested by @zmike in D3858 (Ecore_Device).

eolian-cxx: Fix generation for ref generic_values and Eolian classes

Edje_Edit: allow set NULL as a style name for textblock part.

NULL as 'style' value remove style name from textblock part state.

elm widget - stop segv during shutdown when objects become deleted

@fix

evas event handling2 - fix incorrect object reporting

this fixes a new bug brought up by
9f0fd66ab818d212fa88faef316ac17625f1a2f5 which fixes event reporting
etc. etc. .. this fixes T4017

@fix

elc_combobox: add/remove item doesn't resize container with genlist

recalculation of item count is getting only on a first load of combobox.
whenever user add more items, size of container with items doesn't getting more
place (provoking scroller to appear).
but its event more annoying when user delete items, then there are lots of
free spaces left that looks ridiculus

@fix

notify: change return value of "theme_apply" function with inner _theme_apply

Summary:
notify's theme_apply function return value with eo_do_super(... elm_object_widget_theme_apply)
but notify's super class is widget, widget_theme_apply return always ELM_THEME_APPLY_SUCCESS.
so, notify's theme_apply function always return ELM_THEME_APPLY_SUCCESS, if it couldn't apply style correctly.

notify apply style via _notify_theme_apply function, so that function must decide theme_apply function's return value.

Reviewers: herb, singh.amitesh, Hermet, cedric, raster, jpeg

Reviewed By: jpeg

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

evas: Fix crash (?) in evas events

Reported by Shuhrat Dehkanov on the ML.
See f0fd66ab818d212fa88faef316ac17625f1a2f5.

Note: I didn't have a crash myself.

efl: Remove del_intercept before calling eo_del

In class destructor. Still not sure if we should do this
or just set the pointer to NULL.

Ping @TAsn

eo: Fix crash during eo_shutdown

I was getting a crash in eo_shutdown, inside
_efl_event_pointer_class_destructor as I was calling eo_del
from there. But the parent class was already destroyed.

Assuming class IDs can only go up, and child classes are only
instanciated after all their parents, it is safer to call the
class destructors in reverse order.

Obviously, still pretty sure eo_del() in a class_destructor
is not a good idea...

tests: ecore_audio: disable stalling audio_obj_pulse test

This test is stalling. Locally as well as on Jenkins. I tried to bisect it
without any luck. Even running it from the 1.17 release it does no longer work
so i guess it is some change coming from a pulse update on my system. I have
version 7.1 here. As we have no-one working actively on ecore_audio I disable
the test here and we can track the problem on T4018.

tests: ecore_audio: remove dead code

These test cases do not exist and the code is commented out for a long time.

tests: ecore_audio: move _finished_cb behind PULSE guard

This callback is only used from the pulse object test.

efreet - fix mime file path messing with short paths like /

this fixes T4015

@fix

[efl_ui_image] fix documentation

evas event handling - fix incorrect object reporting

this fixes event reporting for mouse in/out/move/down/up due to evas
just totally getting tese objects wrong.

this fixes T3718

@fix

Evas GL: Fix warning about invalid type (GLES 3.1)

elementary/fileselector: Fix legacy path set/get of Fileselector.Entry

 elementary config: Use SI units.

 efl ui frame: Update the file name for translation purposes.

 This unbreaks build.

 See 53035f23e6d4d0cfc93356c1776d8a2a58d18f91.

elc_combobox: fix theme and style set for combobox

@fix

efl - threads and signals - make efl work on "insane" os's with signals

so. on linux signals are delivered to the main process thread/loop.
thats' where signal handlers are set up and always run. this is sane.
it's predicatble. but of course this is not the same in bsd land.
there "just send the signal to any old thread and call the signal
handler there" seems to tbe the order of the day. this explains why
wer are losing sigchld signals in edje_cc - it's heavily threaded and
bsd is just randombly picking a thread to call it on.

this fixes that. in theory. i hope. i can't test, but putting it in to
share

@fix

eo: use ref types in eo_base

efl - fix lots of little init/shutdown pairs that are wrong

i've fixed almost all the eina init/shutdown pairs to do the right
thing now... except one (ecore_shutdown) with comment inline where
eo_shutdown is not called. if this is called we are in crash land.
this needs further inspection.

eina btlog - add -n option to not use color escapes

EvasGL: Add support for Evas GL 3.1

Summary:
2 wrapper functions are added.
(glFramebufferParameteri, glGetFramebufferParameteriv)

update gl2ext.h(generated 20160209) in evas_gl.h

Test Plan: OpenGL ES Conformance Test

Reviewers: jpeg

Subscribers: spacegrapher, wonsik, cedric, jpeg

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

edje: Remove shadow variable

elm_frame: updated the eo prefix and documentation

Summary: No idea what else can be done with this widget.

Reviewers: jpeg, cedric, woohyun

Subscribers: raster, cedric, jpeg

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

spinner: spinner entry reactivate when focus come again.

Summary:
When user edit spinner value on entry.
The user want to back on entry to edit spinner value even swiching view(focus out and focus in).

I considered a couple of exception case ( case that entry should not reactivate.)
1. User click spinner button to give focus.
2. User give focus to spinner's object using key action.

Test Plan:
Add sample in elementary_test.
Edit spinner value.
Gives focus to other window.
Back to spinner view.
See the action.

Reviewers: Hermet, woohyun, cedric, jpeg

Subscribers: cedric, jpeg

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

test: Fix strange black rectangle in a test case

elementary_test -to "Entry on Page Scroll"

Click on "Click this and close popup"

A weird rectangle would be temporarily visible while the
popup is fading in. The rectangle was the block_events
rectangle. By default, elm_popup blocks events, and its
parent must be the whole blocked input area. This fixes the
parent in the example.

I think the block_events region should be fading in and out
as well, because it looks terrible right now.

Note: this changes the position of the popup, but makes
the whole "block_events" thing actually work (avoiding
other render artifacts).

theme: Add fade-in/out for block regions (popup & notify)

Those are a bit faster than the content transitions. I believe the
popup transition time should be reduced. 500ms is way too long for
such animations.

elm: test_glview - allow window to close

evas: Fix rare issue with recursive proxy src invisible

Scenario:
  smart {
    text
    proxy -> text, src_invisible
  }
  proxy -> smart

What we should see:
  smart {
    (blank)
    proxy -> text
  }
  proxy -> {
    (blank)
    proxy -> text
  }

What we saw:
  smart {
    (blank)
    proxy -> text
  }
  proxy -> {
    text
    proxy -> text
  }

Solution:
Check in evas render, when we're inside a proxy render, and the
proxy src_invisible flag is on (evas_object_source_visible_set(0),
that we're rendering the object itself to its proxy surface. If not,
it means we're rendering another proxy surface, ie. a parent smart
object's proxy surface.

Still loving evas render.

Fixes T4006.

@fix

ecore: Fix obvious crash with legacy timers

ecore_timer_del() checks a flag "inside_call" that can be
set before calling the timer cb... but it was never reset
to 0. So, all legacy timers would keep on ticking forever
and ever, until they return CANCEL.

Anyway, I find the distinction between eo_del and
ecore_timer_del very troubling. eo_del() should work
on a legacy timer. Ping @cedric. Maybe override eo_del()?

Fixes T3898

edje - reduce memory footprint by rearranging structs and types

this should cut some memory used by edje by using smaller types like
shorts instead of ints where we just dont need a full int range and
short will do, and re-ordering in memory data soit packs better when
accoutning for alignment

examples: Fix typo in filename

Elementary: cnp - check on NULL data in elm_drag_start

Summary:
check on NULL add in wayland elm_drag_start API
@fix

Test Plan: Call elm_grag_start with data = NULL under wayland

Reviewers: zmike, NikaWhite, jpeg

Reviewed By: NikaWhite, jpeg

Subscribers: bu5hm4n, myoungwoon, cedric, jpeg

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

elm/gengrid: Fix null pointer dereferences.

CID: 1357210 and 1357209.

evas/textblock: Mark unused variable.

ecore-wl2: Fix error handling for bad fd

If we end up failing to get the fd from ecore_main_fd_handler_fd_get,
then we should just bail out of this function and try again later.

Fixes Coverity CID1357213

@fix

Signed-off-by: Chris Michael <cpmichael@osg.samsung.com>

elementary: Grab all key presses in elm_code for now

Working around issues where new key bindings were breaking the editor