wayland: whitespace cleanup

Reviewers: cedric, devilhorns, zmike

Subscribers: cedric

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

uuid: Check error on ftruncate call

Summary:
Fixes warning:

src/bin/e_uuid_store.c:71:4: warning: ignoring return value of
‘ftruncate’, declared with attribute warn_unused_result
[-Wunused-result]
    ftruncate(store->shmfd, TABLE_SIZE);
        ^

Signed-off-by: Bryce Harrington <bryce@osg.samsung.com>
Reviewers: zmike

Subscribers: cedric

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

wl compile fixes

wayland: Handle failure to init a display screen

Summary:
e_comp_wl_output_init will fail (silently) under a couple of
conditions.  Instead, make it propagate the error so it can be handled
at a higher level.  For now just issue an error message.

Signed-off-by: Bryce Harrington <bryce@osg.samsung.com>
Reviewers: cedric, devilhorns, zmike

Subscribers: cedric

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

bump modapi version to 15

e_util_comp_desk_count_get() -> e_desks_count()

also wtf is this double loop counting

remove e_util_zone_current_get()

remove e_util_zone_current_get() usage

remove E_Zone->comp member

use correct wayland ifdef for E_Client uuid generation

e_zone functions no longer require E_Comp param

e_client functions no longer require E_Comp param

e_comp functions no longer require E_Comp param

remove e_util_comp_current_get()

remove e_util_comp_current_get() usage

remove e_comp_evas_find()

remove e_comp_evas_find() usage

remove E_Drag->comp and E_Comp param from e_drag_new()

remove E_Event_Compositor_Resize struct

corresponding event no longer sends event info

comp canvas functions no longer require an E_Comp param

remove unnecessary E_Comp references in comp object callbacks

E_CLIENT_*FOREACH() macros no longer take a E_Comp param

remove E_Client->comp

remove E_Comp_Object->comp member

remove e_comp_util_evas_object_comp_get()

remove e_comp_util_evas_object_comp_get() usage

remove e_util_comp_zone_id_get() and e_util_comp_zone_number_get()

e_util_comp_zone_number_get() -> e_comp_zone_number_get()

remove e_comp_get() function

de-e_comp_get()ify e

mrw I remember how to regex http://i.giphy.com/dC9dujPVeAfIY.gif

fix wl shell client borders for internal, borderful clients...again

change DESKTOP env setting on startup

Summary:
From Enlightenment 17 to Enlightenment.

Seems e17 keeps popping up...
rEdb6aeef2b1a9c831fa0901e499dd8f73ff733836
rEec1cd25cfe666665b9e5641c91d3ad1fd2f9f11c

Reviewers: zmike

Subscribers: cedric

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

send client sizes for wl configure events when fullscreening/maximizing

remove unused wl variables

don't reset placement of internal wl clients on first commit

fixes placement of evry et al

don't crash when creating external shell surface clients

sadconditionals.jpg

correctly apply wl shell client border states only for non-internal clients

don't create new wl shell clients for internal windows

conditionally compile elm win trap wayland functionality

Revert "e - fix build when wayland not enabled"

This reverts commit 413ca8046e44786d086816a94d60492b92f6fac6.

broke wayland runtime: wrong define names, wrong usage

wl drm output module must set comp gl state more pedantically

disable wl client hide animations even with gl

broken until we start making textures for rendering clients

don't identify all wl shell clients as internal...you crackhead

fix ibar drop calc to not unexpectedly trigger epileptic seizures in users

when attempting to drop on an icon, the drop area geometry was never included
in the calc, resulting in the drop changing between positions on every frame

slight ibar dnd optimization when checking taskbar apps for drop site

Updating italian translation

e clients - make sure we handle client with a NULL zone

don't modify pixmap hash when changing pixmap parent window

moderately certain I originally wrote this to work in the other direction
and then failed to remove it when I switched to setting parents instead of
children. regardless, pixmap hash should not be changed here

CRI on attempting to change a pixmap's client when pixmap already has client

this should never occur, so do something if it does since it means there's a big problem

always create new client for wl shell surfaces

this should be a more correct way of handling (shell) surfaces since
there's never a time when we won't want to create a new client

don't reuse wl surface pixmaps without adding a ref

don't attempt to find wl shell E_Client when one cannot exist

Wayland: more checks to avoid consuming serials needlessly

Reviewers: devilhorns, zmike

Subscribers: cedric

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

fix invalid eina list usage when merging wl surface frame list

Wayland: Don't consume serials when not sending events

Reviewers: zmike, devilhorns

Subscribers: cedric

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

wayland: Check for potential OOM from wl_array_add()

Summary:
wl_array_add() returns NULL if it can't malloc/realloc.  This
is probably unlikely but check for it and issue a meaningful error if it
occurs.

Reviewers: cedric, devilhorns, zmike, raster

Reviewed By: raster

Subscribers: raster, cedric

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

Send modifiers to wayland clients on focus in

Summary:
If we don't update the modifiers on focus in we can end up with stuck
modifiers if a modifier is held when starting a client.

Reviewers: zmike, devilhorns

Reviewed By: devilhorns

Subscribers: cedric

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

Break e_comp_wl_input_keyboard_modifiers_update into two functions

Summary:
Separate the state serialization from the send so we can use the
serialization function elsewhere.

Reviewers: zmike, devilhorns

Subscribers: cedric

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

Don't update xkbstate when no app has focus

Summary:
We still update the keyboard array.  focus_in adds everything in the
array to xkbstate anyway, so adding it when there's no focus results
in a double update of state and potentially stuck modifiers.

Reviewers: zmike, devilhorns

Subscribers: cedric

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

redo ibar icon label position signalling to emit left/center/right

also calc based on icon position, not icon orientation

fix crash when using e_comp_object_util_center_pos_get() with non comp_objects

fix horizontal/vertical smart maximize

fix T2170

use current zone for shelf cfd setup

fix T2168

Revert "disable fileman popups in wl-only mode"

This reverts commit e6c47e8d90e75182cffd1d8cbdbb9e3821a4dbf1.

these no longer crash so reenable them

remove unnecessary wl client pixmap checks

these are direct object callbacks so they will only be triggered on wl clients

use stderr messages when failing to init X compositor

can't show error dialogs when we're about to exit, may as well print something useful

fix T2182

exit immediately if any unrecognized arg is passed on startup

ref T2182

disable wl clients in X

this is untested and undeveloped, no point in leaving it around to crash randomly

Fix stuck modifiers on client exit

Summary:
We still need to remove the active keys when focus_out happens, even
if the pixmap is already gone.  If we don't and a modifier was held
down during exit then the xkb state will have that modifier stuck
forever.

Reviewers: zmike, devilhorns

Subscribers: cedric

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

don't require client mirror object existence to hide dead comp object mirrors

only hide comp object images for dead clients on smart hide

flag non-internal wl shm clients as dead and delete mirrors on hide

no more crashing?

trivial move comp object function

add E_Client->dead to mark clients that should immediately be removed from the compositor

theme_shadow was unused

force non-internal wl clients to hide when gl is not available

hide comp object image on smart hide

ensure that this is not being rendered anywhere at the time of hide

Revert "when we get a request to close client app, defer resource deletion"

This reverts commit 1cf7c1f1511c446413fdc6fae35d17870bdd1c4e.

didn't fix the issue and obfuscated the code unnecessarily

re-setting same cursor object updates hotspot as expected

never resize client clip object

set ec->fullscreen before applying fullscreen geometry

reapply wl surface focus during commit when necessary

grab wl shell surface geom and use for adjusting resize operations

I think this still may not be 100% perfect,
but it's more like 95% perfect now instead of 50%

enlightenment: Implement support for wl_data_device_interface "release" event

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

enlightenment: Bump required Wayland version to 1.7.0

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

don't install SIGBUS handler if we are running wayland-only

Wayland shm sets up a sigbus handler for catching invalid shm region
access. If we setup our sigbus handler here, then the wl-shm sigbus
handler will not function properly

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

when we get a request to close client app, defer resource deletion
until After close animation has been run

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

remove ec->awaiting_hide_event

no longer used

improve wl grab resize calcs

update+clamp wl surface client size to buffer size during commit

this is the same as the X render loop behavior, just in a different place

send different values to wl clients for configure when performing a resize action

the calculated values here seem to not be sufficiently accurate to manage a calculated resize.
instead, calc relative pointer coords and send them

 #barrel

set output->scale to e_scale, and when we wl_output_send_scale
actually send output->scale

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

wayland: Make sure we're initializing output scale sensibly

Summary:
By default the E_NEW() will create our output objects with a 0.0 scale,
which doesn't make any sense and if anything will lead to crashes.
Instead use a scale factor of 1.0.

When updating output details, if the scale setting is invalid then set
it to 1.0 as a sensible value.

Note this doesn't actually enable scaling, just helps make sure we're
not injecting invalid scale parameters from the start.

Reviewers: zmike, devilhorns, cedric

Subscribers: cedric

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

wayland: Refactor output initialization code

Summary:
The code to initialize outputs has duplicate stanzas for updating the
data members of the output structure.  This set of refactoring steps
shuffles things around to eliminate the redundant code.  I think this
also makes the code easier to follow too.

This also breaks out a new internal routine for looking up outputs by
id.  I suspect this will have general usefulness later.

Reviewers: devilhorns, cedric, zmike

Subscribers: cedric

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

remove wl client resize idler, send ec->w/h instead of ec->client.w/h on resize

this idler makes sense in X where there is a third party display server but not in wl

noop when setting pointer object if re-setting same object

disable fileman popups in wl-only mode

these crash for now and I'm annoyed

remove missed use of E_COMP_WL_PIXMAP_CHECK macro

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

wayland:  Document the public API

Summary: Add doxygen comments for public API members

Reviewers: zmike, cedric, devilhorns

Reviewed By: devilhorns

Subscribers: cedric

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

wayland: Eliminate extraneous E_COMP_WL_PIXMAP_CHECK macro

Summary:
Many of the functions in the compositor do some preliminary checks, to
ensure objects are valid and the right type.  Most of the checks follow
a consistent pattern, for example:

    if (!(ec = data)) return;
    if (e_object_is_del(E_OBJECT(ec))) return;
    if (ec->iconic) return;

This style makes it pretty straightforward to review the pre-conditions
of the function.

The one exception is the check to see if an object is a wayland pixmap;
this is implemented as a C macro.

     E_COMP_WL_PIXMAP_CHECK;

The name of the macro is descriptive enough, but it conceals what is
actually being checked.  I think that it is clearer and fits better with
the style of the rest of the pre-condition checks for this check to just
be shown as regular C code:

     if (e_pixmap_type_get(ec->pixmap) != E_PIXMAP_TYPE_WL) return;

Reviewers: zmike, cedric, devilhorns

Reviewed By: devilhorns

Subscribers: cedric

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

wayland: Fix missing check on E_NEW return

Summary:
All other E_NEW calls in this file check the return value, except this
one place.  In this function, other calls are being checked for NULL and
handled so this one should as well.  The other handlers in this function
follow the style of issuing an error message, freeing objects, and
returning false; we don't need to free anything so just do the error
message and return.

Reviewers: zmike, cedric, devilhorns

Subscribers: cedric

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