[PULL] hw/xwin warning fixes

Hi Keith, Some patches to fix format warnings in hw/xwin. Please consider pulling into master. I'm not sure what the status is of 1.18, so I can resend this later, if you prefer. Thanks. The following changes since commit 732e3b9c08532f40656010eac9d128601cc88c3f: Instead of one

Re: [PULL] hw/xwin warning fixes

Jon Turney jon.tur...@dronecode.org.uk writes: Jon TURNEY (13): hw/xwin: Remove GetTickCount() from various pieces of debugging output hw/xwin: Ensure format warnings in winclipboard/ hw/xwin: printf format fixes for HWND type hw/xwin: printf format fixes for XID type

Re: [PATCH v3] configurable maximum number of clients

Adding the dependency on xproto 7.0.28 that has been released last Wednesday which includes the patch to increase the number of file descriptors. This is required for the second (dependent) patch which has already been reviewed by Adam, thus adding the R-b: Adam Jackson a...@redhat.com to that

Re: [PATCH] Fix NO_LOCAL_CLIENT_CRED build

Hi, On Mon, Jul 6, 2015 at 6:08 PM, Keith Packard kei...@keithp.com wrote: I liked your first version a lot better; looks a lot simpler. An autoconf test might make sense if there was some reason to override it? To be clear, I primarily gave feedback because touched it last. I think it makes

[PATCH 2/2] configurable maximum number of clients

Make the maximum number of clients user configurable, either from the command line or from xorg.conf This patch works by using the MAXCLIENTS (raised to 512) as the maximum allowed number of clients, but allowing the actual limit to be set by the user to a lower value (keeping the default of

[PATCH 1/2] Require at least xproto 7.0.28

To make sure we have enough file descriptors to manage at most 512 clients. Signed-off-by: Olivier Fourdan ofour...@redhat.com --- configure.ac | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure.ac b/configure.ac index 328563e..16d2123 100644 --- a/configure.ac +++

[PATCH 4/4] glamor: Drop a bunch of glamor_priv == NULL checks.

Now that it's always non-null when the pixmap is non-null, we don't need so much of this. glamor_get_pixmap_private() itself still accepts a NULL pixmap and returns NULL, because of glamor_render.c Signed-off-by: Eric Anholt e...@anholt.net --- glamor/glamor.c | 17 ++---

[PATCH 2/4] glamor: Drop dead glamor_is_large_picture().

It died as of keithp's new glyphs code. Signed-off-by: Eric Anholt e...@anholt.net --- glamor/glamor_utils.h | 12 1 file changed, 12 deletions(-) diff --git a/glamor/glamor_utils.h b/glamor/glamor_utils.h index cef62c5..0a7de82 100644 --- a/glamor/glamor_utils.h +++

[PATCH 3/4] glamor: Ask the server to always allocate our private.

This avoids a lot of screwing around to attach our privates later. It means that non-glamor pixmaps now gain 120 bytes of glamor privates on 64-bit (which has quite a bit of fixable bloat), and glamor pixmaps take one less pointer of storage (not counting malloc overhead). Note that privates

[PATCH 1/4] glamor: Reuse the glamor_is_memory helper.

Signed-off-by: Eric Anholt e...@anholt.net --- glamor/glamor_composite_glyphs.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/glamor/glamor_composite_glyphs.c b/glamor/glamor_composite_glyphs.c index cd88524..389c8f4 100644 --- a/glamor/glamor_composite_glyphs.c +++

glamor: Always allocate pixmap privates

The commit message in #3 has the meat of the series, but for further explanation, note that this reduces the glamor library by 8588 bytes (4.8%) on my x86-64 build, and I've only got 94 client-allocated pixmaps on my desktop at the moment according to xrestop.

Re: [PATCH 1/1] randr: make RROutputChanged change the main protocol screen not the gpu screen

On 14.01.2014 00:37, Alberto Milone wrote: On 16/12/13 09:52, Alberto Milone wrote: On 12/12/13 10:00, Alberto Milone wrote: We only set changes on the main protocol screen as, for example in RRSetChanged() and RRTellChanged(), therefore we should follow the same logic when reporting that an

[PATCH xf86-input-libinput] Add a property for tap drag lock

Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- configure.ac | 2 +- include/libinput-properties.h | 6 +++ man/libinput.man | 9 + src/libinput.c| 94 +++ 4 files changed, 110 insertions(+), 1

Re: [PATCH 4/4] glamor: Drop a bunch of glamor_priv == NULL checks.

On 08.07.2015 09:26, Eric Anholt wrote: diff --git a/glamor/glamor_utils.h b/glamor/glamor_utils.h index 0a7de82..a923b7a 100644 --- a/glamor/glamor_utils.h +++ b/glamor/glamor_utils.h @@ -756,8 +756,8 @@ glamor_translate_boxes(BoxPtr boxes, int nbox, int dx, int dy)

Re: [PATCH] Fix NO_LOCAL_CLIENT_CRED build

On 06/07/2015 23:08, Keith Packard wrote: Jon TURNEY writes: On 04/07/2015 05:21, Ray Strode wrote: On Thu, Jul 2, 2015 at 12:40 PM, Jon TURNEY wrote: Makes sense. Revised patch attached. LGTM I'm going to choose to assume that is a Reviewed-by: Keith, Please consider applying to master.