eo_debug: Some more log improvements

Nothing fancy here...

eo_debug: Improve dangling xref debug logs even more

In case of manual free, as is heavily used by Evas, we can't really
print an ERR if there are still references before free has been
called.

This may not be ideal from a pure EO point of view but considering
how Evas uses manual free this is the best solution to avoid
polluting debug logs.

eo_debug: Improve one log

This changes the following message when the object is referencing
its own data. Also lowers from ERR to WRN and adds the class
name for the referenced data.

ERR<17450>:eo /home/jpeg/e/core/efl/src/lib/eo/eo_private.h:337
  _efl_unref_internal() in /home/jpeg/e/core/efl/src/lib/eo/eo.c:620:
  func '_efl_object_call_end' Data of object 0x400000021008db58 is
  still referenced by object 0x400000021008db58

Note that evas and elm have a few calls to efl_data_ref(obj, NULL)
which are imho quite ugly: not using the return value and not
specifying the data class. I'm keeping them as-is for now.

elm_widget: Fix order of operations (isa before data_get)

This fixes some of the occurences of the following error message
when using eo_debug:

ERR<23101>:eo /home/jpeg/e/core/efl/src/lib/eo/eo.c:1955
  efl_data_scope_get() Tried getting data of class 'Elm.Widget'
  from object of class 'Edje.Object', but the former is not a
  direct inheritance of the latter.

If we're going to call efl_isa, then let's at least do it before
efl_data_scope_get.

elc_multibuttonentry: input panel should be shown whenever mbe is clicked.

Summary:
When mbe got focus the input panel will be shown.
Otherwise, input panel will be hidden.

If the mbe still got focus then input panel hide by user action.
there is no way to reactivate the input panel in case.

Reviewers: woohyun, jpeg, cedric, Hermet

Reviewed By: jpeg

Subscribers: cedric, jpeg

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

ecore_input: add API to set/get deadzone of joystick event for an axis.

Summary:
The axis type joystick event could occur without user's control if joystick is too sensitive.
The deadzone prevents this unnecessary event. The default value is 200.
The event value for an axis is a signed integer between -32767 and +32767.

Test Plan: Using example

Reviewers: raster, cedric, jpeg

Reviewed By: jpeg

Subscribers: stefan_schmidt

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

build: Add our wayland protocols to the dist

Oops, without these in the distribution tarball you can't
build wayland support.

emotion: add since tag to new emotion_file_meta_artwork_get API

efl: add since tag to new invalid object error type

eo: add missing since tags for functions added during 1.19 cycle

ecore_x: add singce tags for ecore_x_stack_* functions added during 1.19 cycle

ecore_evas: add brief docs and since tags for new ecore_evas_shadow_geometry_*

These have been missing when introduced during the 1.19 cycle.

ecore-wl2: Fix setting input region on a window

Previous code here would store incorrect values into the
window->input rectangle. These values should be what is actually
getting set in the wl_region. This code also fixes an issue when
setting input region for transparent/alpha windows by clearing out
any pending input region (wl_surface_set_input_region(surface, null)).

Reviewed-By: Derek Foreman <derekf@osg.samsung.com>
@fix

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

ecore-wl2: Fix setting opaque region on a window

Previous code here would store incorrect values into the
window->opaque rectangle. These values should be what is actually
getting set in the wl_region. This code also fixes an issue when
setting opaque region for transparent/alpha windows by clearing out
any pending opaque region (wl_surface_set_opaque_region(surface, null)).

Reviewed-By: Derek Foreman <derekf@osg.samsung.com>
@fix

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

Revert "evas textblock: keep previous size when the calculation is skipped"

This reverts commit c39855a8ac6f86c2dd8b1856ed70f85e0e521f3e.

This actually breaks 1 dialog in e (app exited with error exit code).
it worked everywhere else so i thought it was good. seemingly not
after i saw one of these. revert D3595

Edje calc: Fix textblock size calculation logic

Summary:
In singleline textblock, using "text.min: 1 0" and min, max width,
Edje allows to use expandable text with ellipsis. It shows ellipsis
when only text's width reach the max width.
But, Edje couldn't support same feature on multiline textblock.
Edje dose not use max height or text.max properly if ellipsis is enabled.
This feature is very useful to make a layout with dynamically aligned text.
@fix

Reviewers: cedric, tasn, woohyun, raster, herdsman

Subscribers: z-wony, eagleeye, jpeg

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

evas textblock: keep previous size when the calculation is skipped

Summary:
When a size calculation is skipped because of some reasons,
Evas Textblock should keep same size with the previous size.
@fix

Test Plan: N/A

Reviewers: raster, herdsman, cedric, jpeg

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

ecore_evas: Avoid safety check error log

If an engine can not be used (eg. GL inside a standard Xephyr),
some ERR logs will be printed out by ecore_evas_x. This patch
avoids extra eina_safety error logs from using a NULL pointer.

evas key: Add null safety to EAPIs

This avoids a crash with Bryces. This does not fix the problem
with the Bryce itself. Ping @zmike

See T5157

evas: Fix double free in post events

A use of the post event callback in Bryce triggered a double free
as an event was being processed but not yet removed from its list,
leading to an invalid state of the Eina_List.

Also remove obj from struct as it is unused.

Fixes T5157

evas gl: Fix memory tracking

See b16c961680579ad13 which by "fixing" indentation changed the
logic of the code. Yes the indentation was misleading (because of
the #if / #else) but the logic was correct. Anyway this is was not
a harmful issue as those memory trackers aren't really used.

ecore-evas-drm: Fix setting rotation during async render

If we are in async render and a call to rotation_set has been made,
then we should delay setting that rotation until rendering is
completed. This patch fixes that issue by setting ee->delayed.rotation
and handling it during _drm_render_updates

@fix

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

ecore-evas-wayland: Fix setting rotation during async render

If we are in async render when a call to set rotation happens, we
should not be setting the engine info->rotation until we have actually
handled the rotation. Old code here would set delayed.rotation on the
ecore_evas itself, but then it would tell the engine to rotate. What
we actually want here is If in async render, set the delayed rotation
property and let render_updates deal with any delayed setting.

@fix

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

evas-gl-common: Fix misleading indentation

gcc reports that the 'else' here does not guard the latter
statement...seems like a cause of some missing parens so fix.

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

release: Update NEWS and bump version for 1.19.0-beta1 release

ecore-drm2: Fix debug messages to use proper type

Small patch which fixes some FB flipping messages to use the proper
type (ie: some messages were ERR when should be DBG or WRN, etc).

NB: No functional changes

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

eina: make the ein_file_unlink symbol availbale form the ein_file header

It was only defined in the c file. Without any documentation, since tag, etc.

tests/eina/eina_test_file.c:855:4: warning: implicit declaration of function ‘eina_file_unlink’
[-Wimplicit-function-declaration]

tests: eina_file: check if the fd is really 0 when we set close file to true

The return value from create_file_not_empty was saved but never checked if it
really is 0. Whcih should be the case if we call the function with file_close
true.

tests: eina_file: fix coding style in file_unlink test case

tests: eina_file: remove unused variable

elm_index: Skip box auto fill when index size is invalid.

Summary:
_index_box_auto_fill will be called when user calling elm_index_level_go(), elm_index_omit_enabled_set().

item size re-calculated and each item style reset even index size is invalid.
It can make performance issue during index creation time.

Test Plan:
elementary_test
index sample

Reviewers: taxi2se, jpeg, cedric

Subscribers: cedric, jpeg

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

evas gl: Partially fix MSAA for GLES 3 drivers

MSAA (multisampled_render_to_texture) support was implemented with
GLES 2 in mind, but for GLES 3 it comes as a core feature, not as
an extension. Also it relies on renderbuffers, not textures. GL
is awesome.

Note: MSAA still doesn't work!

win: Make ELM_ACCEL work with msaa, depth & stencil

This makes the env var override and the elementary config
global override on accel preference work for not only "gl"
but also advanced configs such as "gl:msaa_high:depth:stencil"

doxygen: Fix typos & invalid statements in ecore

This covers ecore audio, avahi, buffer and con.

Summary: There are some typos and cacologigue statements in
doxygen of ecore_audio, ecore_avahi, ecore_buffer, and ecore_con.

Test Plan: API Doxygen Revision

Reviewers: stefan, cedric, raster, Jaehyun_Cho

Subscribers: jpeg, conr2d

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

doxygen: Fix typos & invalid statements in Ecore_Getopt and Ecore_Legacy

Summary: There are some typos and calogique statements in doxygen
of Ecore_Getopt and Ecore_Legacy so I had fixed typos and cacologique statements.

Test Plan: Doxygen revision

Reviewers: stefan, cedric, raster, Jaehyun_Cho

Subscribers: jpeg, conr2d

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

EvasGL: For depth&stencil&msaa, use renderbuffer instead of texture

Summary:
glFramebufferTexture2DMultisampleEXT() supports color attachment only.
so depth&stencil attachment should be bound via renderbuffer not texture.

According to http://www.khronos.org/registry/OpenGL/extensions/EXT/EXT_multisampled_render_to_texture.txt
In order to allow the use of multisampled depth and stencil buffers
when performing multisampled rendering to a texture, the extension
also adds the command RenderbufferStorageMultisampleEXT.
target, textarget, texture, and level correspond to the same
parameters for FramebufferTexture2D and have the same restrictions.
attachment must be COLOR_ATTACHMENT0.

Test Plan: Evas GL on GLES 3

Reviewers: jpeg, raster, dkdk, wonsik

Tags: #efl

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

efl net proxy helper - use infinite for and sleeps instead of pause

so the process was waiting to be killed (waiting forever) and i used
pause() but this isnt windows-friendly, so since this process expects
to be killed by the parent process, then use a for (;;) with lon long
long sleeps... instead.

this shoud fix T5174

configure - eolian seemingly still needs evil due to eina

this should fix T5172

vpath file - add a keey property to know if obj should be kept around

if its a file downloading (to a tmp location) or a zip/tar/whatever
file being extracted also to a tmp location and that tmp file needs to
be removed after ...you need to keep the obj around to know when to
delete the file. this makes the keeping optional and you query if the
vpath obj is meant to be kept or not. if it's not it's safe to del
immediately.

this cuts down 1 obj per image obj/edje obj when generally unused.
save some mem.

evas: fix bugs in gif image loader

gif's logical screen size (which is considered the image size)
might be different from the size of each frame.
when decoding a frame, the width and height of the decoded data should be
based on the size of the frame, not on the size of the logical screen size.
if a frame is decoded into a buffer of screen size, this might happen

(frame = 6 X 3, logical screen = 5 X 3)
OOOXXX      OOOXX
OOOXXX  =>  XOOOX
OOOXXX      XXOOO

@fix

evas gl common - make render op explicitly signed as we set it to -1

this just is cleaner and now also matches types for render_op in gl
common context.

eina inlist - cast container via void ptr to avoid warnings

we really can't do much here but our direct casting causes warnings in
apps or anyone using this macro so keep things silent as our pointer
tricks are actually ok but the compiler can't figure it out.

evas_table: abort when there is already the option struct

otherwise we would just reuse that struct. This results in
freeing/writing/reading the memory twice.

This error message should give the dev a tip on what went wrong, instead
of leaving cryptic valgrind messages and crashes.

This fixes the sysinfo desktop gadget in enlightenment.

ref T5173

evas: fix child position when the container is moved

when the position of the container is changed the children should also be
repositioned, so setting the changed flag.

The visual effect where you saw that was in luncher where items of the
bar did not get fully up.

evlog/log bt - fix size casting tot stop warning when printing errs

evas color copy neon asm - declare tmp as output not input for warning--

less warnings - use output not input for tmp var. much better.

eo base - fix warnings for debug logs to get format string types happy

gcc is very unhappy with these log prints - specifically on arm 32bit.
this fixes it so we can focus on real warnings/issues.

evas generic loader ps - fix warning about printf format mismatch type

it's right. types are uint but format exects unsigned long. fix.

evas wayland egl engine - fix warning about struct wl_resource not used

this struct is not actually used other than in this func ptr decl and
it's warning-land for gcc, so .. make it void * and do things like
pretty much all other gl extn func ptrs... be very generic

evas gl drm engine - fix warning about struct wl_resource not used

this struct is not actually used other than in this func ptr decl and
it's warning-land for gcc, so .. make it void * and do things like
pretty much all other gl extn func ptrs... be very generic

evas smooth scaler - fix warning where indenting is misleading

indeed the indenting is misleading. fix.

evas blit main - put mmx funcs in ifdefs as they are not used on arm

fixes warning

evas draw func get - make pixels as unused as it is in neon asm cases

also a little shitespace cleanup and correct ()ing of ops to clean
this function up a bit.

evas convert color - neon asm - fix warning by casting - no side effects

it's a warning one way or another so reduce noise with a harmless case
as passing in a pit ro a 32bit type is more restrictive than the ptr
it accepts (an 8bit type)

efl base class - fix warning about comparing differing sizes

yes - we compare a difference between 2 ptrs and an index which is a
uint. the safe thing here is to promote the unit to the ptrdiff_t
type. reality is we cant have more than 2^32 cb's on an object
anyway... so this should be ok.

efl io - fix using wrong type for return values pointed to

this potentially could be a bug on 32bit systems. this fixes that and
addresses the warning that pointed it out

ecore pipe - fix warning of comparing integers of differing sizes

elm code - silence possibly uninitialied compiler warnings

every time i build efl i see all these warnigns spew by about elm_code
and possibly uninited vars being used. yes - they're filled in via ptr
ref bya  func but gcc definitely seems to think they MIGHT not be
filled in in some cases... and either way tyhe cost of setting to 0 is
minimal and less noise is better so we see real warnings more
easily... :)

evas wl shm - fix warning with unused variable

evas: use the right structure to not over allocate memory.

Eina_Trash is designed for storing cached pointer without any memory
consumption. Please be careful with EFL memory consumption.

ecore-wl2: roundtrip during client disconnect

ensure all pending requests are processed by the compositor and fixes
session recovery destroy

@fix

elm_code: Fix syntax crash on trailing newline in multiline macro

@fix

ecore: Don't call _ecore_fd_valid() in release builds

Not sure _ecore_fd_valid() is all that useful anymore, as the
commit that introduced it said it would be removed "before release"
a long time ago - it's a debug assist that probably doesn't need
to be in release builds.

(I'm counting syscalls on rpi3 - still, calling this an optimization
seems like a bit of a stretch.)

Sysinfo gadget themes: Account for vertical theme.

efl_object: document when callbacks are called

ecore_wl2: Update compositor bind now that damage_buffer is released

We had to do some weird special case stuff when binding wl_compositor
because we implemented damage buffer before depending on a wayland
release that provided it.

Now our wayland dep is newer than damage buffer, so the test can be
more conventional.

Should have no functional change.

eolian: silence static analyzer null check (CID 1369018)

eolian: fix memory leak in doc tokens (CID 1367505)

vpath usage - simplify to bare minimum to make gustavo happy

since these are only local path resolves, the do and wait are
technically not needed. also remove any other tmp strings
and use the vpath string resolving feature to avoid
printfs/strjoins/cats etc. etc. as well.

also remember to remove old name string from evas shm code - it worked
for me. i guess i was lucky and it happened to be NULL thus free was
fine.

build: add mapping between ecore_file and ecore-file for pkgconfig files

elua.pc was depending on a ecore_file pc file it is named ecore-file. We have
mapping for similar cases. This one was just forgotten it seems.

Fixes T5075

src/utils: remove no longer used gitigonre and directories

When the XCB support was removed end of last year the .gitignore file left
behind. It was the only file still sitting in the src/utils tree so git removes
the whoile subtree now.

net_proxy_helper: fix typo in dependencies

Fixes:
error: cannot find the library 'lib/eina/libeina.la' or unhandled argument 'lib/eina/libeina.la'

Signed-off-by: Romain Naour <romain.naour@gmail.com>

eina_modinfo: add missing dependency on libeina.la

Fixes:
libtool:   error: cannot find the library 'lib/eina/libeina.la' or unhandled
argument 'lib/eina/libeina.la'

Signed-off-by: Romain Naour <romain.naour@gmail.com>

elementary-cxx: Make C++ test compile a not used archive so linking can be avoided

Fixes linking errors for some older combos of C++ compiler and linker.

T2838

release: Update NEWS and bump version for 1.19.0-alpha1 release

Revert "vpath usage - simplify to bare minimum to make gustavo happy"

This reverts commit 2037474dc0fd2b360452f2a15abcbe533b57ca37.

This causes the wayland_shm engine to seg fault immediately at
startup when attempting to create shm buffers.

Please make sure when committing changes to the wayland_shm engine to
test on intel, exynos, and at least one platform without dmabuf
capabilities - or using the EVAS_WAYLAND_SHM_DISABLE_DMABUF env var
to disable dmabuf on intel or exynos.

Anyone without the time or hardware to fully test changes to wayland_shm
can submit a patch to phabricator and assign it to me so I can fully
test it before landing.

docs: avoid possible duplicates when figuring out all impls of a func

docs: list all existing implements of a method/property

docs: reverse inheritance hierarchy api in doctree

elm: examples: add new layout_model_connect binary to gitignore

docs: eina: add doc for new eina value type

docs: elm: document new efl UI image factory class

docs: efl mvc: document all new model view controller related eo files

docs: ecore: document newly added efl model composite classes

docs: efl filters: document all new data structures for efl filters

vpath usage - simplify to bare minimum to make gustavo happy

since these are only local path resolves, the do and wait are
technically not needed. also remove any other tmp strings and use the
vpath string resolving feature to avoid printfs/strjoins/cats etc.
etc. as well.

efl vpath - for runtime dir dont fall back to tmp but instead abort

if we can't create a runtimedir maybe its best to abort. also ensure
it is created as the effective user id.

efreetd - remove unused var

eina utils and vpath - use euid not uid if setuid for correct user

was using uid not euid for "if setuid use safe way to get location" -
ie the user that effectively owns the process, not nominally.

@fix

elm config - handle setuid apps correctly and dont create junk dirs in .

this fixes a bug where elm_cofnig would firstly use
~/.config/elementary which isnt what we have used before and so would
break older config. also it'd create junk if it was setuid. fix this.

@fix

evas fb dev env var - allow in setuid processes with sanitizing

this allows only /dev/fb[0-0] or /dev/fb/something where somthing does
not begin with a . - thus no way to break out of the fb subdir... so
it should be ok... this keeps setuid safety and allows this env var to
work now as intended in this situation.

efreetd cache - use eina home env get to centralize homedir fetching

this cleans up and improves a getenv that should really use eina

efreetd - dont use XDG_RUNTIME_DIR or EFREET_LOG env vars for setuid

if a setuid app needs efreet - it will not be able to contact the
users' efreetd and thus may spawn it's own... and thus we cant have
this now spawned efreetd using env vars inherited from the
unpriveleged etc. user, so ignore them.

efreet - use vpath to get XDG_RUNTIME_DIR so we have a single impl

having too many places that get XDG_RUNTIME_DIR makes it harder to
keep secure etc, so make it a single location for efreet too.

evas wayland shm engine - use vpath for getting xdg runtime dir

this should make the wayland shm engine safe to use from setuid
processes. it shouldnt depend on getenv of specific file paths if
setuid. vpatrh takes care of this in one central place

ecore buffer module - use vpath for xdg runtime for setuid safety

this should make ecore buffer module safe to use in setuid processes

evas cserve - remove unused if 0'd out code that is misleading my greps

looking for XDG_RUNTIME_DIR usage to lock down for setuid apps... this
msled me... remove it as it's unused.

elm widget item: add guide about when to use item_signal_emit

genlist/gengrid now uses cache for view objects of items and
re-associate edje objects to different obj items whenever needed.
problem here is if app user changes the state of view object
(by sending edje signal, elm_object_item_signal_emit()) and
forget to reset it in unrealized, the old edje object state will link to other
object item, so its application user responsibility to reset
the edje state to normal in "unrealized" callback.

Signed-off-by: Amitesh Singh <amitesh.sh@samsung.com>

evas: Fix invalid current state (invalid bool value)

newframe() resets some of the gl context properties, so match them
to our shadow copy in the gc state.

target_surface_set() also resets some of those properties but only
in our shadow copy, not in the actual GL context. As a consequence
we can't rely on setting a bool to 0 or 1 unless we also call the
equivalent GL function. Expand bitfields to 2 bits allows us to
set a bool to the invalid value of -1 (yes, that's not a real bool).

Also there is no need to reset the target surface to NULL during
newframe. It will be reset during target_surface_set.

This fixes some issues I encounter while working on GL filters.

elput - use vpath to get xdg runtime to also be setuid safe

this time around... use a / at the end of the (:run:) virtual dir
path to make it work.