Re: [PATCH weston 2/3] text: Don't track binding in struct text_backend

2015-09-24 Thread Bryce Harrington
On Wed, Sep 23, 2015 at 10:27:42AM -0500, Derek Foreman wrote: > On 17/09/15 03:45 PM, Bryce Harrington wrote: > > On Thu, Jun 25, 2015 at 03:49:48PM -0500, Derek Foreman wrote: > >> The binding is only used for preventing launch, and we've already got the > >> pid for that. > >> > >> With multiple

Re: [PATCH weston 2/3] text: Don't track binding in struct text_backend

2015-09-23 Thread Derek Foreman
On 17/09/15 03:45 PM, Bryce Harrington wrote: > On Thu, Jun 25, 2015 at 03:49:48PM -0500, Derek Foreman wrote: >> The binding is only used for preventing launch, and we've already got the >> pid for that. >> >> With multiple seats there are multiple bindings, so if the most recent >> binding unbind

Re: [PATCH weston 2/3] text: Don't track binding in struct text_backend

2015-09-17 Thread Bryce Harrington
On Thu, Jun 25, 2015 at 03:49:48PM -0500, Derek Foreman wrote: > The binding is only used for preventing launch, and we've already got the > pid for that. > > With multiple seats there are multiple bindings, so if the most recent > binding unbinds it will clear the pointer, so using it like this i

[PATCH weston 2/3] text: Don't track binding in struct text_backend

2015-06-25 Thread Derek Foreman
The binding is only used for preventing launch, and we've already got the pid for that. With multiple seats there are multiple bindings, so if the most recent binding unbinds it will clear the pointer, so using it like this is risky anyway. Signed-off-by: Derek Foreman --- src/text-backend.c |