do not return when x11 damage event returns no rects

ensure first_damage flag is set in all cases

pulseaudio: pass big data as pass by reference

CID: 1308394

enlightenment: Make sure we have a zone before calling
_bgpreview_viewport_update

@fix CID1324753

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

enlightenment: Use 'fabs' to compute absolute value of floating point
types

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

enlightenment: #if 0 out an unused variable

NB: The function which actually uses this variable is also #if 0'd out

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

enlightenment: Use 'fabs' to get absolute value of floating point types

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

enlightenment: Fix E_CONFIG_LIMIT to use the proper min & max values

e_config->backlight.idle_dim is actually an unsigned char in the
structure, so use a 0 & 1 for min & max

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

enlightenment: Add missing field initializers for Eina_Rectangle

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

enlightenment: Add missing field initializers for Evas_Point

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

enlightenment: Add missing EINA_UNUSED for unused function paramaters

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

enlightenment: Add missing EINA_UNUSED for unused function parameters

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

enlightenment: Add missing EINA_UNUSED for unused function paramaters

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

subtract x11 client damage region even when forcing full override damage

fix T2785

force full damage for first frame of x11 override clients

it seems that damages for popup windows in some applications,
namely blink-based browsers, are reported incorrectly, resulting
in sometimes having a blank window

Fix make distcheck

mixer: start app from bin directory

before the app was installed into the modules directory, now its
installed to the bin directory

mixer: install icon and .desktop file

mixer: install app to bin directory

use correct param ordering for setting wayland client frame geometry

function uses l,r,t,b and not x,y,xx,yy

fix T2624

remove duplicate surface size update for wayland clients during commit

this has already happened in e_comp_wl_surface_attach()

don't forget wl_desktop_shell's header in tarball

Summary: Merge branch 'master' of git://git.enlightenment.org/core/enlightenment

Reviewers: zmike

Subscribers: cedric

Maniphest Tasks: T2776

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

e mixer - be more robust if pulse exits/crashes and def sink is null

ensure there are protections everywhere for when default sink is null
- mixer did crash when pulse crashed and a popup was to be shown...

explicitly disable screensaver for wayland urgency wakeups

make wake-on-urgent apply for all cases of window urgency

in the case that an urgent window is set as urgent again, wakeup should
still occur

apply _GTK_FRAME_EXTENTS for xwayland clients

not entirely sure this is worthwhile but may as well

Intl: fix potential stack buffer overflow.

CID 1298051

@fix

Client: fix potential out of bounds read.

This is an unsigned int underflow that could lead to an out
of bounds read.

CID 1291837

@fix

enlightenment: Update wayland readme

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

don't crash desktop gadget systrays when adjusting image sizing

desktop gadgets (gadman) do not have a shelf, so derefing that pointer
without checking will guarantee a crash

e comp data - actually ... NO visibility effect by default is right

so a bit of a config wipe had my comp config reset (some dev
shenanigans) and i got to see what current e "out of box" config is -
and it was horrid for menus. visibilitiy effect was broken. vertial.
default was none - always. forever. like empty. so go back to that. in
fact why do this visibility effect? that was the point of having a
different shadow in the theme - each has its own look + effect. this
makes things far more complex...

when removing VERTICAL maximize state, also remove LEFT and RIGHT states

LEFT and RIGHT are supersets of VERTICAL which serve no purpose on their own,
so remove them when removing VERTICAL

fix T2770

e comp - default config was rather broken - fix

default config didnt select menu style shadow - was selecting
vertical/still which meant animation was rather horrible. fix.

only use fast-path pixmap size updating after a resize for x11 clients

updating these values after other operations will result in the pixmap size
being set inaccurately, leading to rendering inconsistencies

update wayland readme

repopulate app list dialogs when efreet deskop cache updates

should improve usage when installing/uninstalling apps

attempt to load default e_bindings if initial load fails

do not set frame geometry on frame-possessing wayland clients

throw an ACK when trying to set comp frame geometry if frame exists

clear menu cache when saving a menu file

force regeneration of menus after editing them

ref T2271

make icon selection dialog resizable in .desktop editor

ref T2271

move configure init after intl init has completed

gettext calls will not return the correct language string until after
setup has completed, leading to untranslatable strings

fix T2760

clear the tiler for input region after setting to comp object.

Summary:
there is no need to set repeatedly input region even if it's already applied.
and this patch remove the code to clear tiler from client's unmapped case.
this fixes that tiler for input region is removed before applying it to comp object in case client is unmmaped yet.

Reviewers: devilhorns, zmike

Subscribers: cedric

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

Add support for the ISO 8601 date format to the clock module

Reviewers: zmike

Subscribers: cedric

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

 e fm: Use correct string length when copying path.

 Sometimes, trying to go to the parent directory wouldn’t work and end up showing some garbage. Unfortunately, the code ended up not NULL-ending the path, which made searching for the path separator buggy.

Add "(drag only)" to title for drag only edge bindings

Reviewers: zmike

Subscribers: cedric

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

 e menus: Fix invalid comparisons.

 Fixes CID 1267215 and 1267216.

fix systray to work with spec-breaking apps, eg. steam

according to the StatusNotifierItem specification, applications
register "service org.freedesktop.StatusNotifierItem-PID-ID" on the
session bus, and then "must register the unique instance name
to the StatusNotifierWatcher".

some applications, such as steam, instead register the path that they
will run on (/org/ayatana/NotificationItem/steam) and then expect the
watcher to register the method call's send id bus: this is totally bogus.

to catch this, when registering the new item the enlightenment watcher must
first determine if the item is spec-conforming. if yes, proceed as normal.
if no, pretend the application knows what it's doing and try to make things
work as expected anyway

for more details, read the full spec here
http://www.freedesktop.org/wiki/Specifications/StatusNotifierItem

fix T2763

move client post-update adding into render function

in some cases, it's possible for a client which expects to render on
the next frame to actually render on the frame after. in these cases,
the compositor must not clear the pixmap image until after the render
has occurred in order to avoid inaccuracies. for this reason, the best
place to flag a client for post-render work is at the time of the client's
render

ref T2762
ref D3120

unify comp render returns for argb and non-argb clients

fix comp render return value for failed draws of non-argb clients

fix return value of comp render for wayland clients

add RENDER_DEBUG for forced renders when setting dirty flag

should remove a frame resource from pending.frames list also

Summary: Signed-off-by: Boram Park <boram1288.park@samsung.com>

Reviewers: stefan_schmidt, gwanglim, raster, zmike, devilhorns

Reviewed By: zmike, devilhorns

Subscribers: cedric

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

md5: fix wrong sizeof argument (SIZEOF_MISMATCH)

Summary: fixes CID: 222382

Reviewers: raster, zmike, cedric, devilhorns

Reviewed By: devilhorns

Subscribers: cedric, seoz, sachin.dev

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

unset release mode

20.0-alpha NEWS updates

20.0-alpha release

always apply all randr changes after a screen event

fixes case where plugging an external monitor would cause the screen
to expand onto that display without the display being effectively usable
until after a restart

wl_desktop_shell: return void

Summary:
_e_input_panel_cb_surface_get() returns void.

Signed-off-by: Boram Park <boram1288.park@samsung.com>
Reviewers: zmike, stefan_schmidt, gwanglim, raster, devilhorns

Subscribers: cedric

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

remove remote colorclass editor usage

match current elm function signatures

merge E19 branch NEWS items

update wayland readme

redo wayland client buffer referencing

the previous methodology was effectively:
attach -> ref(new buffer) x2 / unref(old buffer) x2
...
...
attach -> ref(new buffer) x2 / unref(old buffer) x2

this resulted in buffer management failures and crashing. now the
buffer gets 1x ref before render and 1x unref after render, ensuring
that the lifetime is accurate (assuming evas doesn't lie to us)

now we still have random crashing during resize, but not as much as
before

redo CSD for wayland clients to use comp object frame geometry

this fixes issues with client placement and is a more native solution
than the previous hacks

never mark wl client pixmaps as dirty or refresh them

this has no meaning for a wl pixmap in the current implementation

free x11 damage region rects

do not roundtrip for x11 client message DBG if it will not be visible

also free fetched atom name

unset E_Client->internal_elm_win before deleting the client in e_win

==24509== Invalid write of size 8
==24509==    at 0x502D00: _e_elm_win_trap_del (e_win.c:39)
==24509==    by 0x509BFC2: _elm_win_evas_object_smart_del (elm_win.c:1886)
==24509==    by 0x91F4643: evas_obj_smart_del (in /usr/lib/libevas.so.1.15.99)
==24509==    by 0x91F5B5C: evas_object_smart_del (evas_object_smart.c:1021)
==24509==    by 0x91E9107: _evas_object_eo_base_destructor (evas_object_main.c:739)
==24509==    by 0xE54A8A3: eo_destructor (in /usr/lib/libeo.so.1.15.99)
==24509==    by 0x5086715: _elm_widget_eo_base_destructor (elm_widget.c:5744)
==24509==    by 0xE54A8A3: eo_destructor (in /usr/lib/libeo.so.1.15.99)
==24509==    by 0xE5443EC: _eo_del_internal (eo_private.h:221)
==24509==    by 0xE5443EC: _eo_unref (eo_private.h:295)
==24509==    by 0xE5443EC: _eo_do_end (eo.c:546)
==24509==    by 0x4D5B1A: _e_obj_dialog_free (e_obj_dialog.c:125)
==24509==    by 0x4D61FB: e_object_free (e_object.c:152)
==24509==    by 0x4D61FB: e_object_unref (e_object.c:152)
==24509==    by 0x4EDC54: _e_sys_logout_after (e_sys.c:750)
==24509==    by 0x4ED7AC: _e_sys_action_do (e_sys.c:925)
==24509==    by 0x4EE348: e_sys_action_raw_do (e_sys.c:311)
==24509==    by 0x4EE43F: _e_sys_comp_done_cb (e_sys.c:66)
==24509==    by 0x6097348: _edje_emit_cb (edje_program.c:1476)
==24509==    by 0x6097348: _edje_emit_handle (edje_program.c:1405)
==24509==    by 0x60924EE: _edje_message_queue_process (edje_message_queue.c:787)
==24509==    by 0x60926A6: _edje_job (edje_message_queue.c:154)
==24509==    by 0xCC5087A: _ecore_job_event_handler (ecore_job.c:121)
==24509==    by 0xCC4B204: _ecore_call_handler_cb (ecore_private.h:390)
==24509==    by 0xCC4B204: _ecore_event_call (ecore_events.c:565)
==24509==    by 0xCC52AE7: _ecore_main_loop_iterate_internal (ecore_main.c:1927)
==24509==    by 0xCC52CD6: ecore_main_loop_begin (ecore_main.c:983)
==24509==    by 0x4383F4: main (e_main.c:1047)
==24509==  Address 0x14fb1a28 is 1,176 bytes inside a block of size 1,352 free'd
==24509==    at 0x4C2A65B: free (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==24509==    by 0x4D61FB: e_object_free (e_object.c:152)
==24509==    by 0x4D61FB: e_object_unref (e_object.c:152)
==24509==    by 0x502CED: _e_elm_win_trap_del (e_win.c:37)
==24509==    by 0x509BFC2: _elm_win_evas_object_smart_del (elm_win.c:1886)
==24509==    by 0x91F4643: evas_obj_smart_del (in /usr/lib/libevas.so.1.15.99)
==24509==    by 0x91F5B5C: evas_object_smart_del (evas_object_smart.c:1021)
==24509==    by 0x91E9107: _evas_object_eo_base_destructor (evas_object_main.c:739)
==24509==    by 0xE54A8A3: eo_destructor (in /usr/lib/libeo.so.1.15.99)
==24509==    by 0x5086715: _elm_widget_eo_base_destructor (elm_widget.c:5744)
==24509==    by 0xE54A8A3: eo_destructor (in /usr/lib/libeo.so.1.15.99)
==24509==    by 0xE5443EC: _eo_del_internal (eo_private.h:221)
==24509==    by 0xE5443EC: _eo_unref (eo_private.h:295)
==24509==    by 0xE5443EC: _eo_do_end (eo.c:546)
==24509==    by 0x4D5B1A: _e_obj_dialog_free (e_obj_dialog.c:125)
==24509==    by 0x4D61FB: e_object_free (e_object.c:152)
==24509==    by 0x4D61FB: e_object_unref (e_object.c:152)
==24509==    by 0x4EDC54: _e_sys_logout_after (e_sys.c:750)
==24509==    by 0x4ED7AC: _e_sys_action_do (e_sys.c:925)
==24509==    by 0x4EE348: e_sys_action_raw_do (e_sys.c:311)
==24509==    by 0x4EE43F: _e_sys_comp_done_cb (e_sys.c:66)
==24509==    by 0x6097348: _edje_emit_cb (edje_program.c:1476)
==24509==    by 0x6097348: _edje_emit_handle (edje_program.c:1405)
==24509==    by 0x60924EE: _edje_message_queue_process (edje_message_queue.c:787)
==24509==    by 0x60926A6: _edje_job (edje_message_queue.c:154)
==24509==    by 0xCC5087A: _ecore_job_event_handler (ecore_job.c:121)
==24509==    by 0xCC4B204: _ecore_call_handler_cb (ecore_private.h:390)
==24509==    by 0xCC4B204: _ecore_event_call (ecore_events.c:565)
==24509==    by 0xCC52AE7: _ecore_main_loop_iterate_internal (ecore_main.c:1927)
==24509==    by 0xCC52CD6: ecore_main_loop_begin (ecore_main.c:983)
==24509==    by 0x4383F4: main (e_main.c:1047)

do not roundtrip for x11 client message DBG if it will not be visible

also free fetched atom name

remove object event callback on notification popdown

==24509== Invalid write of size 8
==24509==    at 0x2C5ED500: _notification_popup_del_cb (e_mod_popup.c:404)
==24509==    by 0x91AAE93: _eo_evas_object_cb (evas_callbacks.c:92)
==24509==    by 0xE54DDEA: _eo_base_event_callback_call (eo_base_class.c:715)
==24509==    by 0xE54B67A: eo_event_callback_call (in /usr/lib/libeo.so.1.15.99)
==24509==    by 0x91AB323: evas_object_event_callback_call (evas_callbacks.c:264)
==24509==    by 0x91E8DE3: _evas_object_eo_base_destructor (evas_object_main.c:691)
==24509==    by 0xE54A8A3: eo_destructor (in /usr/lib/libeo.so.1.15.99)
==24509==    by 0x60A3019: _edje_object_eo_base_destructor (edje_smart.c:43)
==24509==    by 0xE54A8A3: eo_destructor (in /usr/lib/libeo.so.1.15.99)
==24509==    by 0xE5443EC: _eo_del_internal (eo_private.h:221)
==24509==    by 0xE5443EC: _eo_unref (eo_private.h:295)
==24509==    by 0xE5443EC: _eo_do_end (eo.c:546)
==24509==    by 0x6091146: edje_match_callback_exec_check_finals (edje_match.c:556)
==24509==    by 0x6091146: edje_match_callback_exec (edje_match.c:711)
==24509==    by 0x60974AE: _edje_emit_cb (edje_program.c:1453)
==24509==    by 0x60974AE: _edje_emit_handle (edje_program.c:1405)
==24509==    by 0x60924EE: _edje_message_queue_process (edje_message_queue.c:787)
==24509==    by 0x60926A6: _edje_job (edje_message_queue.c:154)
==24509==    by 0xCC5087A: _ecore_job_event_handler (ecore_job.c:121)
==24509==    by 0xCC4B204: _ecore_call_handler_cb (ecore_private.h:390)
==24509==    by 0xCC4B204: _ecore_event_call (ecore_events.c:565)
==24509==    by 0xCC52AE7: _ecore_main_loop_iterate_internal (ecore_main.c:1927)
==24509==    by 0xCC52CD6: ecore_main_loop_begin (ecore_main.c:983)
==24509==    by 0x4383F4: main (e_main.c:1047)
==24509==  Address 0x23b3dcb0 is 16 bytes inside a block of size 80 free'd
==24509==    at 0x4C2A65B: free (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==24509==    by 0x2C5ED606: _notification_popdown (e_mod_popup.c:747)
==24509==    by 0x2C5ED952: _notification_reshuffle_cb (e_mod_popup.c:700)
==24509==    by 0x91AAE93: _eo_evas_object_cb (evas_callbacks.c:92)
==24509==    by 0xE54DDEA: _eo_base_event_callback_call (eo_base_class.c:715)
==24509==    by 0xE54B67A: eo_event_callback_call (in /usr/lib/libeo.so.1.15.99)
==24509==    by 0x91AB323: evas_object_event_callback_call (evas_callbacks.c:264)
==24509==    by 0x91E8DE3: _evas_object_eo_base_destructor (evas_object_main.c:691)
==24509==    by 0xE54A8A3: eo_destructor (in /usr/lib/libeo.so.1.15.99)
==24509==    by 0x60A3019: _edje_object_eo_base_destructor (edje_smart.c:43)
==24509==    by 0xE54A8A3: eo_destructor (in /usr/lib/libeo.so.1.15.99)
==24509==    by 0xE5443EC: _eo_del_internal (eo_private.h:221)
==24509==    by 0xE5443EC: _eo_unref (eo_private.h:295)
==24509==    by 0xE5443EC: _eo_do_end (eo.c:546)
==24509==    by 0x504CEF: _e_zoomap_smart_del (e_zoomap.c:266)
==24509==    by 0x91F5ABC: evas_object_smart_del (evas_object_smart.c:1019)
==24509==    by 0x91E9107: _evas_object_eo_base_destructor (evas_object_main.c:739)
==24509==    by 0xE54A8A3: eo_destructor (in /usr/lib/libeo.so.1.15.99)
==24509==    by 0xE5443EC: _eo_del_internal (eo_private.h:221)
==24509==    by 0xE5443EC: _eo_unref (eo_private.h:295)
==24509==    by 0xE5443EC: _eo_do_end (eo.c:546)
==24509==    by 0x458C7A: _e_comp_object_util_del (e_comp_object.c:2402)
==24509==    by 0x91AAE93: _eo_evas_object_cb (evas_callbacks.c:92)
==24509==    by 0xE54DDEA: _eo_base_event_callback_call (eo_base_class.c:715)
==24509==    by 0xE54B67A: eo_event_callback_call (in /usr/lib/libeo.so.1.15.99)
==24509==    by 0x91AB323: evas_object_event_callback_call (evas_callbacks.c:264)
==24509==    by 0x91E8DE3: _evas_object_eo_base_destructor (evas_object_main.c:691)
==24509==    by 0xE54A8A3: eo_destructor (in /usr/lib/libeo.so.1.15.99)
==24509==    by 0x60A3019: _edje_object_eo_base_destructor (edje_smart.c:43)
==24509==    by 0xE54A8A3: eo_destructor (in /usr/lib/libeo.so.1.15.99)
==24509==    by 0xE5443EC: _eo_del_internal (eo_private.h:221)
==24509==    by 0xE5443EC: _eo_unref (eo_private.h:295)
==24509==    by 0xE5443EC: _eo_do_end (eo.c:546)
==24509==    by 0x6091146: edje_match_callback_exec_check_finals (edje_match.c:556)
==24509==    by 0x6091146: edje_match_callback_exec (edje_match.c:711)
==24509==    by 0x60974AE: _edje_emit_cb (edje_program.c:1453)
==24509==    by 0x60974AE: _edje_emit_handle (edje_program.c:1405)
==24509==    by 0x60924EE: _edje_message_queue_process (edje_message_queue.c:787)
==24509==    by 0x60926A6: _edje_job (edje_message_queue.c:154)
==24509==    by 0xCC5087A: _ecore_job_event_handler (ecore_job.c:121)
==24509==    by 0xCC4B204: _ecore_call_handler_cb (ecore_private.h:390)
==24509==    by 0xCC4B204: _ecore_event_call (ecore_events.c:565)
==24509==    by 0xCC52AE7: _ecore_main_loop_iterate_internal (ecore_main.c:1927)
==24509==    by 0xCC52CD6: ecore_main_loop_begin (ecore_main.c:983)
==24509==    by 0x4383F4: main (e_main.c:1047)

reject client resize attempts for clients with dirty pixmaps

these clients will resize/render correctly only after the next
render loop, so defer until then to avoid #RenderFail

fix T2754

s/EAPI/E_API/ ...again

use client window coords for resize-moving without a frame_object

improves placement of csd windows

ref T2750

do not set client window coordinates during a move until after resizing check

fixes case where values would be erroneously updated when they should instead
have been rejected

ref T2750

use more accurate math to determine whether to apply a client maximize change

unset the requested maximize state when receiving a client unmaximize request

not sure why it was always removing VERTICAL here but that's a bug

allow client resizes on axes currently unrestricted by maximize state

if this isn't explicitly blocked by config options then allowing resizes
on the unmaximized axes is necessary in order to avoid accidentally
queuing a full unmaximize

unset E_Client->changes.need_unmaximize after calling unmaximize

...just in case

remove client maximize states during unmaximize only if they are active

do not accidentally remove untested states using bad math operations

force adjustment of client geometry when changing csd frame size

fix T2750

_GTK_FRAME_EXTENTS changes using a property, not a client message

reject identical calls to e_comp_object_frame_geometry_set()

e - efm fix. fix eio error handler that is called after eio cancel

so you have to have an error cb - so make it empty as even after a
canel, it is called, and thus accesses an invalid icon we've freed.

efm - handle cancel of async eio if icon is freed and eio still active

@fix

force pixmap clear when hiding an x11 window

failure to refetch this upon remapping the window will result in
a broken pixmap which cannot be rendered

don't call e_client_unignore() when un-withdrawing an x11 window

these clients were not ignored on creation, so they've already triggered
all the relevant events

bump required efl version

map/unmap x11 client windows when toggling iconic state

ICCCM 4.1.4

initialize pixmap size vars in comp resize interceptor

these are never uninitialized when they are used, but this
is not obvious to the reader

CID 1324956, 1324956

set x11 supported atom array size correctly

CID 1324958

fix handling of x11 ICCCM WithdrawnState

according to ICCCM 4.1.4:
Only the client can effect a transition into or out of the Withdrawn state

withdrawn windows cannot be shown under any circumstances. the best that can
be done is to try mapping the window and hope it decides to appear.

to prevent any inadvertent showing of the window before it leaves the
withdrawn state, we play games with the E_Client->ignored flag in order
to skip client evals until we get notified that maybe we want to stop
skipping those evals

ref T2745

check warp_client existence during pointer warp before dereferencing it

ref T2679

Revert "Revert "Revert "Revert "deskmirror - fix dangling reference to mirror by refcounting it""""

This reverts commit 2373b69c61d7d6ab74675a4facff54601831ce9a.

stop crashing on restart

add csd case for compositor frame adjust messages

not 100% perfect, but close enough

move compositor frame adjust messages to pixels_get callback

native surfaces don't call e_comp_object_render()

set comp object client_inset.calc based on frame_object existence

block border-affecting client functions for csd clients

block border-affecting entries from showing up in client menu with csd

fix e_shelf_position_calc() to not set struct members before applying them

fix T2743

add util function for determining if a client is allowed to have a frame

returns false if mwm borderless is set or if csd exists