Re: [PATCH wayland 8/8] client: Use read preparation API in wl_display_dispatch_queue()

2015-10-09 Thread Jonas Ådahl
On Fri, Oct 09, 2015 at 12:50:24PM +0300, Pekka Paalanen wrote: > On Fri, 9 Oct 2015 16:42:04 +0800 > Jonas Ådahl wrote: > > > On Thu, Oct 08, 2015 at 04:29:03PM +0300, Pekka Paalanen wrote: > > > On Fri, 2 Oct 2015 17:32:59 +0800 > > > Jonas Ådahl wrote: > > > > > > > Instead of doing things

Re: [PATCH wayland 8/8] client: Use read preparation API in wl_display_dispatch_queue()

2015-10-09 Thread Pekka Paalanen
On Fri, 9 Oct 2015 16:42:04 +0800 Jonas Ådahl wrote: > On Thu, Oct 08, 2015 at 04:29:03PM +0300, Pekka Paalanen wrote: > > On Fri, 2 Oct 2015 17:32:59 +0800 > > Jonas Ådahl wrote: > > > > > Instead of doing things that do the equivalent of using > > > wl_display_prepare_read() and friends, jus

Re: [PATCH wayland 8/8] client: Use read preparation API in wl_display_dispatch_queue()

2015-10-09 Thread Jonas Ådahl
On Thu, Oct 08, 2015 at 04:29:03PM +0300, Pekka Paalanen wrote: > On Fri, 2 Oct 2015 17:32:59 +0800 > Jonas Ådahl wrote: > > > Instead of doing things that do the equivalent of using > > wl_display_prepare_read() and friends, just use the public API. The > > only semantical difference is that we

Re: [PATCH wayland 8/8] client: Use read preparation API in wl_display_dispatch_queue()

2015-10-08 Thread Pekka Paalanen
On Fri, 2 Oct 2015 17:32:59 +0800 Jonas Ådahl wrote: > Instead of doing things that do the equivalent of using > wl_display_prepare_read() and friends, just use the public API. The > only semantical difference is that we will now unlock and lock the mutex > more times compared to before. > > Si

Re: [PATCH wayland 8/8] client: Use read preparation API in wl_display_dispatch_queue()

2015-10-05 Thread Bryce Harrington
On Fri, Oct 02, 2015 at 01:50:09PM -0700, Bryce Harrington wrote: > On Fri, Oct 02, 2015 at 01:42:03PM -0700, Bryce Harrington wrote: > > On Fri, Oct 02, 2015 at 05:32:59PM +0800, Jonas Ådahl wrote: > > > Instead of doing things that do the equivalent of using > > > wl_display_prepare_read() and fr

Re: [PATCH wayland 8/8] client: Use read preparation API in wl_display_dispatch_queue()

2015-10-04 Thread Jonas Ådahl
On Fri, Oct 02, 2015 at 01:42:03PM -0700, Bryce Harrington wrote: > On Fri, Oct 02, 2015 at 05:32:59PM +0800, Jonas Ådahl wrote: > > Instead of doing things that do the equivalent of using > > wl_display_prepare_read() and friends, just use the public API. The > > only semantical difference is that

Re: [PATCH wayland 8/8] client: Use read preparation API in wl_display_dispatch_queue()

2015-10-02 Thread Bryce Harrington
On Fri, Oct 02, 2015 at 01:42:03PM -0700, Bryce Harrington wrote: > On Fri, Oct 02, 2015 at 05:32:59PM +0800, Jonas Ådahl wrote: > > Instead of doing things that do the equivalent of using > > wl_display_prepare_read() and friends, just use the public API. The > > only semantical difference is that

Re: [PATCH wayland 8/8] client: Use read preparation API in wl_display_dispatch_queue()

2015-10-02 Thread Bryce Harrington
On Fri, Oct 02, 2015 at 05:32:59PM +0800, Jonas Ådahl wrote: > Instead of doing things that do the equivalent of using > wl_display_prepare_read() and friends, just use the public API. The > only semantical difference is that we will now unlock and lock the mutex > more times compared to before. >

[PATCH wayland 8/8] client: Use read preparation API in wl_display_dispatch_queue()

2015-10-02 Thread Jonas Ådahl
Instead of doing things that do the equivalent of using wl_display_prepare_read() and friends, just use the public API. The only semantical difference is that we will now unlock and lock the mutex more times compared to before. Signed-off-by: Jonas Ådahl --- There is no real need to merge this p