ecore-wl2: Fix issue of misleading 'if' statement

Gcc reports that this 'if' statement is misleading due to indentation.
We should only be sending the 'finish' call if the dnd version
supports it, however we should always be sending 'destroy'

@fix

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

edje: Fix issue of null pointer dereference

Coverity reports that 'text' here is a null pointer dereference so
check for valid 'text' variable before trying to use it.

Fixes Coverity CID1267490

@fix

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

docs: generate proper descriptions for structs and enums

Canvas text: fix corner case in range geometry calc

The trivial case of [pos,pos] (i.e. range of length 0) didn't work if there is a
format item in 'pos'.
The condition was fixed to not include such items. The reason it was not
apparent for text items is that these have further handling in the rest of the
code and would've been disposed of.

@fix

If there is situation when server is closed prior to client, the client pointer pointing to server must be reset and check must be there during client deletion

Summary: During mobile product testing, we got a crash with callstack which suggest server is getting deleted prior to client. On valgrind analysis we found invalid write operation with same callstack. callstack is pasted in comment section.

Test Plan: create a situation where server got deleted prior to client.

Reviewers: raster, cedric

Subscribers: govi, rajeshps, jpeg

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

edje_cc handle theoretical string overflow (not real) shhh coverity

these strings are internal fixed strings so never bigger than buffer,.
but silence coverity to avoid noise. fixes CID 1355588 , 1355589

evas render2 region - handle if src region is empty correctly on add

fix coverity CID 1313542

evas engine cache - drop useless iff that assert already checked

CID 1267465 pointed this out - assert(eim) already checked this so no
need to do it again - looks confusing

evas cache - avoid possible deref of null pointer

this fixes CID 1267472

ecore pipe - fix close of pipe that doesnt set fd to invalid

this should fix CID 1267459

eet - fix possible integer overflow in ptr diff on parse

coverity spotted this - with silly long strings (like 1gb in size or+)
it might happen. fix CID 1256196

evas scale cache - address possible coverity deadlock

this should fix CID 1106338  where we don't lock and unlock caches in
the same order.

evas software x11 - dri swapping - fix possible coverity bug

fix CID 1039620 - coverity is right, in theory.

ecore x xrandr - fix possible divide by zero

fixes CID 1039437 , 1039438

evas object freeing - fix missing null check for layer

fix CID 1039409

Edje entry: Add link press effect

Summary:
Add link press effect

Custom tag (linked_pressed) has to be supplied in entry tb style
with color value for press effect to be applied on link text.

Reviewers: tasn

Subscribers: shilpasingh, cedric, jpeg

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

embryo_cc: Buffer overflow check.

Summary:
A string is copied to the array '&symbolname[0]' of size 32,
without checking it's length at first at embryo_cc_sc1.c:2101.
So, used assert  to avoid possible overflow.

Signed-off-by: Umesh Tanwar <umesh.tanwar@samsung.com>
Reviewers: Hermet, cedric, raster

Reviewed By: raster

Subscribers: singh.amitesh, jpeg

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

elput: check for a special return value

sd_pid_get_session returns ENODATA or ENXIO if it was unable to fetch
the session data for the given pid, reason for that is mostly that the
pid is not running is a session. Adding this as the error value can help
the user debugging the problem without the need of gdb´ing into the
function and checking the return value.

ecore_wl2: Make offer destroy more secure

Check if it is NULL and NULL after each destroy call

ecore_wl2: split of cnp_selection_data_ready and
dnd_selection_data_ready

before there was the case that both handlers are called even if just dnd
data arrived.

ecore_wl2: only destroy source when not needed anymore.

Otherwise we are destroying the source before getting dnd_finished or
cancled events, which is a problem.

ecore_wl2: buffer reading of the data

Otherwise callbacks can go out even if the selection data is not ready
to read.

ecore_wl2: delay the destruction of the offer or do it in the read

Evas textblock: Apply scale factor to <linesize>, <linegap> formats

Summary:
Font size is scaled according to scale factor.
The linesize, linegap formats also have to be scaled properly.
@fix

Test Plan:
Test cases are included.
Run "make check"

Reviewers: woohyun, Jieun, tasn, herdsman

Reviewed By: tasn

Subscribers: raster, cedric, jpeg

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

elm main - fix indentation whitespace since iw as the last to touch it

Edje entry: Skip codes for updating cursor when cursor position is not changed

Summary:
When ever a Edje's cursor function is called, "cursor,changed" signal is
emitted. Even if the position is not changed. And, in Elementary, the signal
will trigger evas_smart_objects_calculate() from elm_widget_show_region_set().
It causes bad performace.
@fix

Test Plan: N/A

Reviewers: tasn, herdsman, cedric, woohyun

Subscribers: jpeg, z-wony, Blackmole

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

Evas Text: Fix width of BiDi text

Summary:
BiDi text is truncated because the way we find the last visual item in text object is wrong.
This patch is similar with _line_native_last_visual_get function in textblock.

Reviewers: cedric, tasn, herdsman

Subscribers: id213sin, cedric, jpeg

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

elm_calendar: provides way for configuration of weekend text color

Summary:
Suggest new mark_type for elm_calendar_mark_add().
The mark names are "weekend/day1", "weekend/day2".
It can be used for each country.
Some country can apply to "weekend/day2" to friday if day rest only a day for a week.
Until now, there is no way to separate holiday, saturday and sunday.

Test Plan:
1. saturday
```
struct tm t = { 0, 0, 12, 6, 0, 0, 6, 6, -1 };
elm_calendar_mark_add(obj, "weekend/day1", &t, ELM_CALENDAR_WEEKLY);
```
2. sunday
```
 struct tm t = { 0, 0, 12, 7, 0, 0, 0, 0, -1 };
 elm_calendar_mark_add(obj, "weekend/day2", &t, ELM_CALENDAR_WEEKLY);
```

Reviewers: CHAN, cedric, raster

Reviewed By: raster

Subscribers: raster, akanad, id213sin, cedric, jpeg

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

Revert the wobbly window code in wayland_egl engine - fun is over guys

So ok. haha. Amusing. But this is inappropriate.

1. It's not an INTENDED effect in theme so unless the theme ius doing
it - it shouldn't happen.
2. There are places to do this and it's NOT inside an engine. There
are high level filter and other mechanisms.
3. Actually this shouldn't be even done client-side. It should be done
compositor-side when moving a window around. Knowing velocity vector
etc. is useful to a client so the protocol can stay but doing the
animation client-side is "wrong". Some windows wobble and others do
not based on toolkit? really? sure we have to live with the CSD
difference but this? The compositor can do this JUYST fine. leave it
to compositor... OR do this properly with filters client-side. e.g. a
2d displacement map with evas filters would do the job as long as it
interpolates. If you want a way of forcing ALL objects in the canvas
to redirect to an intermediate buffer then do it up there at the
canvas level. It then works in GL and software, as opposed to only GL.

Also.... this is now causing issues for users:

<memeka> hi, from
https://git.enlightenment.org/core/efl.git/commit/?id=fb76fe55a52ac212b6870f1d74470a79ea5c5246
i run EFL_WAYLAND_DISABLE_WWW=1 terminology -> but it crashes in
wayland_egl/www.c in the setup_shaders function .... HELP?

so the fun was fune until we do a release (now) and until this causes
problems for users. Back to tried and tested code.

If you want to do this... do it right at the portable layers above.

...

Revert "wayland-egl: Fix use after free"
Revert "wayland_egl: Fix redirect to texture"
Revert "evas-wayland-egl: Add www protocol handling to wayland-egl engine"
Revert "gl_common: Add API for redirecting render to texture"

This reverts commit 2760afbb0e8c8adfb35b1dee016be7883fa07323.
This reverts commit c937248eac0d0996f3cbc31a348b92c8baac2ec3.
This reverts commit c67f50b40aaf2595e21373bf89b53bc1edec44a4.
This reverts commit 0f7f4b6de07707031482f6e3e7f7c69f7fb7e855.

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