RE: [PATCH] compositor: Clean up view output move and destroy listeners

2013-12-19 Thread Eoff, Ullysses A
This patch appears to stabilize the issue reported at https://bugs.freedesktop.org/show_bug.cgi?id=72845 I could not get Weston to crash in 20+ attempts to replicate. U. Artie Eoff > -Original Message- > From: wayland-devel-boun...@lists.freedesktop.org > [mailto:wayland-devel-boun...@

Re: Wayland design and source code documentation

2013-12-19 Thread Bryce Harrington
On Thu, Dec 19, 2013 at 06:13:03PM +0300, Artsiom Anikeyenka wrote: > That's absolutely right and I realize that very well. But you need to think > of total newcomers (as I am). I like graphics and I decided to look around > how that stuff works nowadays under the hood. To have a big picture, befor

Re: Wayland design and source code documentation

2013-12-19 Thread Bryce Harrington
On Thu, Dec 19, 2013 at 02:10:20PM -0800, Bryce Harrington wrote: > On Thu, Dec 19, 2013 at 06:13:03PM +0300, Artsiom Anikeyenka wrote: > > That's absolutely right and I realize that very well. But you need to think > > of total newcomers (as I am). I like graphics and I decided to look around > >

Trusted wayland client API (was: Limitations of Weston's screenshooter)

2013-12-19 Thread Maarten Baert
On 13/12/13 19:11, Jasper St. Pierre wrote: > I'm curious how your rogue program could modify the link-time > environment of a running Wayland compositor. It doesn't have to attack the compositor, it can attack the clients instead. To demonstrate that this is not just speculation on my part, I wen

Re: [PATCH wayland 0/3] Add a debug handler for printing protocol

2013-12-19 Thread Jason Ekstrand
Pekka, Thanks for looking at them! On Thu, Dec 19, 2013 at 2:17 AM, Pekka Paalanen wrote: > On Wed, 18 Dec 2013 20:56:17 -0600 > Jason Ekstrand wrote: > > > This series adds a wl_debug function and associated handler that can > > be set by the uesr. The first patch is merely a rename to prepa

[PATCH wayland 2/3] Add a wl_array_printf function for easy string formatting

2013-12-19 Thread Jason Ekstrand
Signed-off-by: Jason Ekstrand --- This version of the patch adds a wl_array_vprintf function if we want it. Both versions accomplish exactly the same thing as far as the series is concerned. However, wl_array_vprintf might be nice in the future. Feel free to use whichever one you prefer. src/

Re: [PATCH weston] westoy: Add an option to explicitly disable cairo on a widget

2013-12-19 Thread Pekka Paalanen
On Thu, 19 Dec 2013 16:17:12 + Neil Roberts wrote: > This problem was found while looking at the following bug: > > https://bugs.freedesktop.org/show_bug.cgi?id=72612 > > It turns out the patch doesn't help to fix the bug but I think it > would be a good thing to do anyway. > > --- >8

[PATCH] compositor: Clean up view output move and destroy listeners

2013-12-19 Thread Ander Conselvan de Oliveira
From: Ander Conselvan de Oliveira Remove those listeners when the output is destroyed, otherwise they'll point to invalid data that may lead to corruption when assigning a new output for the view. -- This is possibly related to bug 72845. I didn't have enough equipment to try and reproduce it.

[PATCH weston] westoy: Add an option to explicitly disable cairo on a widget

2013-12-19 Thread Neil Roberts
This problem was found while looking at the following bug: https://bugs.freedesktop.org/show_bug.cgi?id=72612 It turns out the patch doesn't help to fix the bug but I think it would be a good thing to do anyway. --- >8 --- (use git am --scissors to automatically chop here) The s

Re: Wayland design and source code documentation

2013-12-19 Thread Jasper St. Pierre
On Thu, Dec 19, 2013 at 10:13 AM, Artsiom Anikeyenka wrote: > That's absolutely right and I realize that very well. But you need to > think of total newcomers (as I am). I like graphics and I decided to look > around how that stuff works nowadays under the hood. To have a big picture, > before I s

Re: Wayland design and source code documentation

2013-12-19 Thread Artsiom Anikeyenka
That's absolutely right and I realize that very well. But you need to think of total newcomers (as I am). I like graphics and I decided to look around how that stuff works nowadays under the hood. To have a big picture, before I start implementing something on top of that. I don't know X11 and I do

[PATCH] compositor: Ensure views on hiden workspace are listening correct output

2013-12-19 Thread Xiong Zhang
The views on hiden workspace can't restored correctly when their output was unplugged. There are two reasons: First: the views on hiden workspace is restored through animation which will change y coordinate continually, if the y coordinate of invisible views are setted to first_output->height/4,

Re: [PATCH wayland 0/3] Add a debug handler for printing protocol

2013-12-19 Thread Pekka Paalanen
On Wed, 18 Dec 2013 20:56:17 -0600 Jason Ekstrand wrote: > This series adds a wl_debug function and associated handler that can > be set by the uesr. The first patch is merely a rename to prepare > for the third. The second adds a useful helper function used by the > third. > > Jason Ekstrand (