Re: [PATCH v2] server: add wl_display_destroy_clients()

2018-02-09 Thread Daniel Stone
Hi Simon, On 13 December 2017 at 10:51, emersion wrote: > Bug [1] reported that wl_display_destroy() doesn't destroy clients, so > client socket file descriptors are being kept open until the compositor > process exits. > > Patch [2] proposed to destroy clients in wl_display_destroy(). The > patc

Re: [PATCH v2] server: add wl_display_destroy_clients()

2017-12-13 Thread Simon Ser
> Original Message >Subject: Re: [PATCH v2] server: add wl_display_destroy_clients() >Local Time: December 13, 2017 1:39 PM >UTC Time: December 13, 2017 12:39 PM >From: ppaala...@gmail.com >To: Simon Ser >wayland-devel@lists.freedesktop.org > >On Wed,

Re: [PATCH v2] server: add wl_display_destroy_clients()

2017-12-13 Thread Pekka Paalanen
On Wed, 13 Dec 2017 06:59:10 -0500 Simon Ser wrote: > > Original Message > >Subject: Re: [PATCH v2] server: add wl_display_destroy_clients() > >Local Time: December 13, 2017 12:11 PM > >UTC Time: December 13, 2017 11:11 AM > >From: ppaala...@gmail.c

Re: [PATCH v2] server: add wl_display_destroy_clients()

2017-12-13 Thread Simon Ser
> Original Message >Subject: Re: [PATCH v2] server: add wl_display_destroy_clients() >Local Time: December 13, 2017 12:11 PM >UTC Time: December 13, 2017 11:11 AM >From: ppaala...@gmail.com >To: emersion >wayland-devel@lists.freedesktop.org > >On Wed,

Re: [PATCH v2] server: add wl_display_destroy_clients()

2017-12-13 Thread Pekka Paalanen
On Wed, 13 Dec 2017 11:51:19 +0100 emersion wrote: > Bug [1] reported that wl_display_destroy() doesn't destroy clients, so > client socket file descriptors are being kept open until the compositor > process exits. > > Patch [2] proposed to destroy clients in wl_display_destroy(). The > patch wa

[PATCH v2] server: add wl_display_destroy_clients()

2017-12-13 Thread emersion
Bug [1] reported that wl_display_destroy() doesn't destroy clients, so client socket file descriptors are being kept open until the compositor process exits. Patch [2] proposed to destroy clients in wl_display_destroy(). The patch was not accepted because doing so changes the ABI. Thus, a new wl_