[PATCH v4] screenshooter: Properly handle multiple outputs.

2012-04-04 Thread Scott Moreau
--- In this version: - struct is named screenshooter_output - global struct instance dropped - use 'output' as name for local instances - properly brace functions clients/screenshot.c | 69 -- src/compositor-drm.c | 15 ++ src/c

[PATCH v3] screenshooter: Properly handle multiple outputs.

2012-04-04 Thread Scott Moreau
--- Changed in this version: The struct is named screenshooter_output while instances are named ss_output. Fixed whitespace errors. clients/screenshot.c | 69 -- src/compositor-drm.c | 14 + src/compositor-wayland.c | 14 +

[PATCH v2] screenshooter: Properly handle multiple outputs.

2012-04-04 Thread Scott Moreau
--- clients/screenshot.c | 69 -- src/compositor-drm.c | 14 + src/compositor-wayland.c | 14 + src/compositor-x11.c | 14 + src/compositor.h |1 + src/screenshooter.c | 29 ++

Re: [PATCH] screenshooter: Properly handle multiple outputs.

2012-04-04 Thread Scott Moreau
> > > >> > } else if (strcmp(interface, "wl_shm") == 0) { >> > shm = wl_display_bind(display, id, &wl_shm_interface); >> > } else if (strcmp(interface, "screenshooter") == 0) { >> > @@ -144,6 +171,8 @@ int main(int argc, char *argv[]) >> > struct wl_display *display;

GTK without dependency on Cairo-GL

2012-04-04 Thread Rob Bradford
This branch makes Cairo GL an optional dependency (and uses SHM buffer instead): https://github.com/rbradford/gtk/tree/wip/wayland-render-changes I'll be pushing this to GTK master soon (once 3.4 is branched off) but please feel free to test. Cheerio, Rob

Re: [PATCH] screenshooter: Properly handle multiple outputs.

2012-04-04 Thread Scott Moreau
On Wed, Apr 4, 2012 at 6:37 AM, Kristian Hoegsberg wrote: > On Wed, Apr 04, 2012 at 01:51:08AM -0600, Scott Moreau wrote: > > --- > > Looks pretty good. There are a few small comments below, but the > overall approach is good. > > > Tested: Side-by-side and stacked output configurations, in x11 a

[PATCH] shell: handle surface destruction during move, resize and rotate grabs

2012-04-04 Thread Ander Conselvan de Oliveira
When the surface being moved, resized or rotated was destroyed, the compositor would crash. Fix this by using a destroy listener on the referenced surface. To reduce code duplication, the surface reference and the destroy listener is added to a new struct shell_grab. --- src/shell.c | 151 ++

Re: [PATCH] screenshooter: Properly handle multiple outputs.

2012-04-04 Thread Kristian Hoegsberg
On Wed, Apr 04, 2012 at 01:51:08AM -0600, Scott Moreau wrote: > --- Looks pretty good. There are a few small comments below, but the overall approach is good. > Tested: Side-by-side and stacked output configurations, in x11 and drm. > Shooting under drm glitches sometimes and either or both outp

Re: [PATCH 2/6 weston] xserver: glue it with shell_surface

2012-04-04 Thread Ander Conselvan de Oliveira
On 04/03/2012 04:58 PM, Tiago Vignatti wrote: Signed-off-by: Tiago Vignatti --- src/Makefile.am|4 +++- src/shell.c| 17 +++-- src/shell.h| 34 ++ src/xserver-launcher.c | 14 ++ 4 files change

[PATCH] screenshooter: Properly handle multiple outputs.

2012-04-04 Thread Scott Moreau
--- Tested: Side-by-side and stacked output configurations, in x11 and drm. Shooting under drm glitches sometimes and either or both output data contains garbage. clients/screenshot.c | 69 -- src/compositor-drm.c | 14 + src/compos