ecore_x: NULL derefs.

Reviewers: raster, devilhorns

Subscribers: raster, cedric, jpeg

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

edje - language emit - handle null language and convert to none

cleans up an emit string being edje,language,(null) if no language is
set. use none instead of (null).

edje: update a style when a style is added as class's member

Summary:
If there is no member styles when a text_class is updated,
newly added styles can't be updated.
So, newly added styles as member of text_class should be updated.
@fix

Test Plan:
Test case is included.

1. Run "elementary_test -to "font overlay""
2. Press Next button. Check the font size.
3. Press Prev button.
4. Put font_size as 50
5. Click Apply button.
6. Press Next button. Check the font size is not changed.

Reviewers: cedric, tasn, herdsman, raster

Subscribers: jpeg, z-wony, Blackmole

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

Accessibility: Read pre-edit text.

Summary:
Edje_entry:  In order to read pre-edit characters as well,
send entry_change_info with preedit,changed
Elm_entry: In order to read pre-edit characters as well,
send text to screen reader on preedit,changed.

Test Plan:
1. Change keyboard language to korean/enable prediction
2. Check the reading, text should be read even before committing

Signed-off-by: Shilpa Singh <shilpa.singh@samsung.com>
Reviewers: thiepha, jihoon, cedric, raster

Reviewed By: raster

Subscribers: raster, kimcinoo, cedric, jpeg

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

evas sw generic - handle alloc failures correctly for map struct

map struct allocation was not handled right - we assumed successthen
later checked for failure with an if() after using the ptr. this
should fix CID 1353722

elm view list - fix allocation of werong type size

the code allocated a much bigger data struct than needed. it used the
wrong sizeof() type. this fixes CID 1355013

edje edit - api was broken with incorrect l r t b ordering for border

this fixes ordering to match everywhere else ANd usage of the get
func. this was broken already where top/bottom border would swap -
plese see coverity scan CID 1355590 for an example of problems this
created.

evas generic loader raw - fix small coverity complaint

technically we dont handle a failed fwrite to stdout according to
coverity, but this is harmless as it doesnt matter if it fails - the
owner process cares. slave does not. silence CID 1356615

ecore animator - remove dead code

fix CID 1356618

evas textblock - fix deref before null check shown by coverity

fixes CID 1356927

elput - simplify and fix handling of string overflow with xdg runtimedir

this addresses an issue pointed to by CID 1357168

elput - remove pointless code in handling touch event

i commented the pointless code out. this fixes CID 1357145

elm atspi bridge - fix resource leak

previous fix brought out a new leak not found before by coverity.

fix CID 1353602

evas gl generic - fix ector end to place return in right var

fixes new CID 1357394

edje_cc - fix new coverity issue

fix CID　1357395

edje: fix build without ephysics.

T4050

edje: add a helper for recursive searching in hash

Summary:
If color class of an edje part is defined as "aaa/bbb/ccc/ddd",
edje will search for color class by the following sequence.
   "aaa/bbb/ccc/ddd"
   "aaa/bbb/ddd"
   "aaa/ddd"
   "ddd"
So, without additional lookup table, edje classes (color, text, size)
can have the functionality like inheritance.

Reviewers: jpeg, raster, cedric

Subscribers: cedric

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

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

edje_player: create scene in the edje_player if there are some 3D-components

Summary: check if the size of scene is bigger than 0x0 and build 3D scene in the edje_player in this case and use "opengl_x11"

Reviewers: Hermet, jpeg, cedric

Subscribers: NikaWhite, Oleksander, artem.popov, cedric, jpeg

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

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

spinner: fix entry align of spinner.

Summary:
Spinner's entry align changed to center.
(Makes same align with spinner's text.)

Test Plan: elementary_test

Reviewers: jpeg

Subscribers: cedric, jpeg

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

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

elput: Fix horrid typo

Not sure how this happened, but use a ; not a : ;)

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

elput: Fix insecure temporary file

Fix using mkstemp directly without securely setting umask first by
making use of eina_file_mkstemp which does set unmask.

Fixes Coverity CID1357165

@fix

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

evas-wayland-shm: Fix Insecure temporary file

Fix using mkstemp directly without securely setting umask first by
making use of eina_file_mkstemp function which does set umask.

Fixes Coverity CID1357164

@fix

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

elm: Fix potential null point dereferences.

Summary:
If every assigning logics to enginelist is not performed some how,
system will crash because the first parameter of strcmp is null.

Test Plan: N/A

Reviewers: Hermet

Reviewed By: Hermet

Subscribers: cedric, Hermet, raster, jpeg

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

edje: fix the map state issue.

While the part state switching, map won't gonna be disabled again,
due to the incorrect state checking.

If there param1 indicates the current state with map off.
then it should disable the map obviously.
But It assumed the param1 is the previous state, not the current!,
so the map didn't go disabled state.

It was intended to avoid the duplicated map state setting.
So to keeping the intention, now it disable the map, only if
the map is actually enabled.

So both are fine.

@fix

eina thread - fix window build with sigprocmask

fixes T4048

ecore-drm2: Perform input device calibration based on output name

When a new input device gets added, device calibration may need to be
performed. In order for that to be done properly, we need to know
which output this input device is associated with. This patch makes a
function call to Elput in order to retrieve that output name and
perform the proper calibration.

@fix

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

elput: Add API function to return output name associated with input
device

This patch adds a new API function which we can call from Ecore_Drm2
which will return the name of an output which is associated with a
given input device. This output name can then be used to find a
matching output, and perform any device calibration that is needed.

@feature

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

emotion xine module - silence coverity

silence CID 1357140 as its harmless to not check fcntl here, but dont
confuse coverity.

emotion vlc module - fix potential divide by 0

fix CID 1357146

emotion xine module - silence coverity complaint

getting the fd from a valid fd handler wont return < 0 but in theory
it can, so just be silent coverity

silence CID 1357155

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.