Add display event to acknowledge ID deletion

We need to make sure the client doesn't reuse an object ID until the
server has seen the destroy request.  When a client destroys an ID
the server will now respond with the display.delete_id event, which lets
the client block reuse until it receives the event.

util: Add wl_list_insert_list()

hash: Improve double hashing

Instead of artificially introducing collisions in the step value by
replacing 0 with 1 (which causes the value 1 to have twice the
frequency of any other value), the step value can simply be computed
as an uniformly distributed value in the range [1, rehash], extremes
included.

This is safe because any step value smaller than the hash modulus is
co-prime with it, hence induces an orbit which includes every integer
in [0, size - 1].

scanner: Check name == NULL for enum entries

From a clang static analysis warning.

connection: Check object types in message parameters

event-loop: Fix idle handler dispatch corner case

When the last idle handler queues another idle handler, we fail to
dispatch that last handler.  The wl_list_for_each_safe loop looks up
the next pointer before running the handler, and at that point it points
to the head of the list and the loop terminates.

Instead, just loop until the list is empty.

scanner: Fix typo in error message

protocol: Change global_remove arg to be 'name' insteadl of 'id'

Consistent with global args.

scanner: Catch more invalid attribute combinations for arg

Spotted by Jakob Bornecrantz.

debug: Print debug message when client wakes up

This helps spot and remove extraneous round trips and context switches.

Silence a valgrind warning

wayland-server: Track input_device resource for focused surface

We no long track the focused surface, but expect the compositor to set
focus when the focus changes.  We do track the resource for the input
device that corresponds to the current surface, in case that goes away.

wayland-server: Always use the display_resource for posting wl_display errors

wayland-server: fix resource destroy

Object ID was read from freed memory.

wayland-server: fix crash on invalid resource

Using display_resource to post error about invalid resource.

Reviewed-by: Tiago Vignatti <tiago.vignatti@intel.com>

scanner: Fix indent for *_add_listener

Rename motion_grab to implicit_grab

Let's just use the same terms as X.

Add back get/set user data wrappers for wl_display

server: Make error posting functions take a resource instead of a client

Remove the wl_visual interface

The visual interface was meant to be a generic mechanism for
specifying the content of a buffer.  It goes back to before we had the
buffer factory interfaces (like wl_drm and wl_shm) and we wanted to
keep it open-ended enough that yuv, png or even svg buffer or so would
be possible.

Now that we have the buffer abstraction, we can add different buffer
types by introducing new interfaces that create buffers.  It only
makes sense to leave it to those interfaces to specify the contents of
the buffers.

For wl_shm, this means that we now just specify the pixel format using
an enum.  For EGL buffers, the exact pixel formats are controlled by
the implementation (part of wl_drm and similar), and from the client
point of view, everything is controlled using EGLConfigs.

Update TODO

server: Disconnect client on error

shm: Compare resource interface when validating visual

server: Pass struct wl_resource for object references

Don't special case object ID 1 lookup

server: wl_display and wl_input_device are no longer resources

Update TODO

Store objects in wl_map data structure

The wl_map data structure is just an array with a free-list that lets the
client recycle unused client IDs and keep range of client IDs under control.

Remove range protocol

Destroy client resource by walking client hash table

All resources are now in the client hash table and we can clean up
just by walking the hash.

Always allocate a new wl_resource when binding an object

Previously we would bind some resources into multiple client hash tables.

Add wl_hash_table_for_each()

Bind globals to client provided object IDs

server: Hand code all wl_display client stubs

server: Assign global id as part of wl_display_add_global()

server: Make object hash table per-client

Rename global 'func' to 'bind'

server: Make everything in the object hash a wl_resource

Use a callback object instead of ad-hoc lists for sync and frame events

So obvious in retrospect.  The object system can do all the work for us
and keep track of pending calls as regular objects and we don't need to
abuse the resource system to get them cleaned up on client exit.  We
don't need the custom key management or (broken) lookup, we just sue
object IDs.  And last but not least, anybody can receive the callback,
not just display listeners.

connection: fix libffi usage, our functions return void

All clients were segfaulting on my machine.

Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>

connection: Object ID 0 is not a valid new object ID

event-loop: Quiet a few warnings

There's no good error recovery possible in these cases though.

Make expat a dependency to the scanner

Rename source subdir from wayland to src

Change scanner.c license to MIT

scanner: Silence warning

scanner: Initialize protocol.copyright to NULL

protocol: Delete screenshooter.xml

The protocol interface is on compositor's side now.

Signed-off-by: Tiago Vignatti <tiago.vignatti@intel.com>

scanner: Take copyright notice from xml file

connection: Use static strings instead of sprintf and buffer overflow

Spotted by Samuel Rødal <samuel.rodal@nokia.com>

Link to -lrt for clock_gettime()

Pass object ID not pointer when sending a global announce event

When the type for the first argument of the global event changed from new_id to
uint, wl_connection_vmarshal started expecting an integer argument rather than
an object argument. As a result we were sending the client a chunk of pointer
rather than a useful global identifier.

conection: Handle demarshal errors a little less dramatically

Fix segfault in client when demarshalling fails

protocol: fix key event and related comments

debug: add timestamps when logging

Signed-off-by: Tiago Vignatti <tiago.vignatti@intel.com>

protocol: Fix type of visual arg in token_visual event

scanner: Output type info for new_id arguments

protocol: Make id arg of display.global event be just a uint

Global names are different from object IDs.

scanner: map 'int' to int32_t.

Add touch events to protocol.

server: Add wl_client_flush()

Flush output queue up for the specified client.

event-loop: Make idle handlers work again

Update TODO

Merge remote-tracking branch 'bnf/buffer.release'

Drop wl_compoositor parameter in wl_buffer

Add busy_count and client parameter to wl_buffer

Can be used to implement release event.

Add buffer.release event

Add more protocol for communicating screen geometry

Add wl_display_remove_global.

Change 4453ba084aae5a00318b9dfdeda95e8eaa17494c disallows using
post_global with objects not on the global list. Therefore selection
and drag offers have to be added to the global list from now on.
However these may often get replaced by a newer object and thus need a
way to remove a global from the global list.

Do not leak globals and listeners.

 From 3fa2ad7dec42f85795449f33b2a4c9461aa4924d Mon Sep 17 00:00:00 2001
From: Laszlo Agocs <laszlo.p.agocs@nokia.com>
Date: Tue, 14 Jun 2011 11:35:28 +0200
Subject: [PATCH] Do not leak globals and listeners.

Move map functionality into shell

Initial surface.attach makes the surface visible and clients
can set the surface type using shell.set_transient etc or
other interfaces prior to that.

Call a new global listener for each existing global

Signed-off-by: Corentin Chary <corentin.chary@gmail.com>

Flush the protocol output buffer when it fills up

Original patch from bnf.

Signed-off-by: Corentin Chary <corentin.chary@gmail.com>

Remove left-over visual prototypes

connection: Fix pointer arithmetic error

Pointed out by Nicolas Pouillon in irc.

Fall back to accept() on systems where accept4() is not implemented.

wayland-shm: Check visual for being NULL

TODO: Update

Add event to associate visuals with a pixel format token

server: Send error if client tries to bind to non-existant object

Define shm specific errors

We stop abusing the wl_display error codes for shm purposes.

Unify error events

Consolidate the different error events into one.  This event will also be
usable for other interaces.

TODO: Consolidate and update

connection.c: Use sizeof int for allocating fd extra storage

display: Adds wl_display_flush to send buffered commands

Previously for wayland clients to flush buffered connection data to the
compositor they needed to first register an event-mask-update-callback
via wl_display_get_fd() to determine if there is anything writeable
pending.  (NB: It's considered an error to iterate connection data with
an invalid mask)

Since it's only possible to register a single update callback currently
it's a bit awkward if you want to allow multiple orthogonal components
to flush the connection data, such as Cogl and Clutter which both want
to interact with wayland and may want to flush commands at different
times.

This adds a mechanism, wl_display_flush(), which makes it possible to
flush writeable connection data without first checking the event mask.

wayland-util: Drop unused struct wl_argument

Add configure option to disable scanner compilation

When cross-compiling Wayland, wayland-scanner should be picked up from
the host system instead of compiling and trying to run e.g. ARM
wayland-scanner on X86.

This patch adds --disable-scanner option for disabling the scanner
from the build and using existing wayland-scanner instead (from PATH)
which should help in most cross-compilation systems.

By default everything works as before.

Signed-off-by: Jani Uusi-Rantala <jani.uusi-rantala@nokia.com>

Add a destroy_listener to wl_resource

Drop the one in wl_surface.

connection: Define _GNU_SOURCE for F_DUPFD_CLOEXEC

scanner: Fail if there's a 'destroy' request not marked as a destructor

scanner: Initialize wl_message structs completely

connection: Handle fd passing in an CLOEXEC safe way

mmap shm buffers read/write

Add some things related to wayland-scanner to .gitignore

event-loop: Drop unused idle_list

Idle sources are now just a special case of a checked source.

Don't define wl_shell in wayland-server.h

event-loop: Idle callback are a special case of a checked source

event-loop: Dont use a check callback just dispatch again

We pass the poll mask so the dispatch callback can know if there is
data on the fd or not.  The dispatch function must return true if
it dispatched something, in which case we need to do another round of
post-dispatch callbacks."

Add common shm-server code