Re: [RFC] DeepColor Visual Class Extension

2017-08-14 Thread Alex Goins
We brainstormed all of the suggestions amongst ourselves, and have a revised version of the spec. It also includes some changes of our own. Unfortunately, we weren't able to bottom out on everything (most notably the masquerading DeepColor/TrueColor visuals), but the suggestions we haven't yet

Re: [PATCH xserver v2] meson: Fix epoll detection

2017-08-14 Thread Keith Packard
Peter Harris writes: > HAVE_OSPOLL is only defined inside this file, not by autoconf/meson, so > it's always defined to 1. > > The rest of os/ospoll.c uses #if instead of #ifdef everywhere. I'd be > inclined to update all of them if I were to paint that particular >

Re: [PATCH xserver v2] meson: Fix epoll detection

2017-08-14 Thread Peter Harris
On 2017-08-14 4:45 PM, Keith Packard wrote: > Peter Harris writes: >> -#if !HAVE_OSPOLL && HAVE_EPOLL_CREATE1 >> +#if !HAVE_OSPOLL && defined(HAVE_EPOLL_CREATE1) > > Should be using defined for HAVE_OSPOLL as well? HAVE_OSPOLL is only defined inside this file, not by

Re: [PATCH xserver v2] meson: Fix epoll detection

2017-08-14 Thread Eric Anholt
Peter Harris writes: > The epoll code depends on epoll_create1, not epoll_create. Reviewed and pushed. Thanks! signature.asc Description: PGP signature ___ xorg-devel@lists.x.org: X.Org development Archives:

Re: [PATCH xserver v2] meson: Fix epoll detection

2017-08-14 Thread Keith Packard
Peter Harris writes: > The epoll code depends on epoll_create1, not epoll_create. > > Signed-off-by: Peter Harris Thanks! > -#if !HAVE_OSPOLL && HAVE_EPOLL_CREATE1 > +#if !HAVE_OSPOLL && defined(HAVE_EPOLL_CREATE1) Should be using defined for

Re: [PATCH xserver 2/3] glamor: Scissor CopyArea to the bounds of the drawing.

2017-08-14 Thread Eric Anholt
Mark Marshall writes: > On 1 August 2017 at 22:59, Eric Anholt wrote: >> Like the previous fix to rectangles, this reduces the area drawn on >> tiled renderers by letting the CPU-side tile setup know what tiles >> might be drawn at all. >> >>

Re: [PATCH xserver] meson: Fix epoll detection

2017-08-14 Thread Eric Anholt
Peter Harris writes: > On 2017-08-11 5:50 AM, Peter Hutterer wrote: >> On Tue, Aug 08, 2017 at 11:16:13AM -0400, Peter Harris wrote: >>> The epoll code depends on epoll_create1, not epoll_create. >>> >>> The trinary " ? 1 : false" is used because HAVE_EPOLL_CREATE1 is

Re: [PATCH libX11 1/3] Make _XCloseLC thread safe.

2017-08-14 Thread Jacek Caban
On 14.08.2017 17:11, Adam Jackson wrote: > On Thu, 2017-08-10 at 23:04 +0200, Jacek Caban wrote: >> Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=55678 >> Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=68538 >> Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=69088 >> Signed-off-by:

Re: [PATCH xserver 0/7] xwin: misc small fixes

2017-08-14 Thread Jon Turney
On 03/08/2017 20:15, Emil Velikov wrote: Hi all, Here's a few small cleanups that I had lying around for months. Patches are dead trivial, although completely untested. Please review, Thanks very much for doing this. Getting rid of those conditionals has been on my todo list for a long

Re: [PATCH xserver 4/7] xwin: remove always true/set XWIN_CLIPBOARD conditional/define

2017-08-14 Thread Jon Turney
On 03/08/2017 20:15, Emil Velikov wrote: --- a/hw/xwin/winscrinit.c +++ b/hw/xwin/winscrinit.c @@ -264,9 +264,7 @@ winFinishScreenInitFB(int i, ScreenPtr pScreen, int argc, char **argv) winScreenInfo *pScreenInfo = pScreenPriv->pScreenInfo; VisualPtr pVisual = NULL; -#if

[PATCH libX11 3/3] Make conv_list thread safe.

2017-08-14 Thread Jacek Caban
Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=55678 Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=68538 Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=69088 Signed-off-by: Jacek Caban --- src/locking.c | 13 + src/xlibi18n/lcConv.c | 25

[PATCH libX11 2/3] Don't cache last lcd in _XlcCurrentLC.

2017-08-14 Thread Jacek Caban
Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=55678 Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=68538 Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=69088 The way it's currently cached is not thread safe. As long as locale doesn't change, the same object is reused anyway.

[PATCH libX11 1/3] Make _XCloseLC thread safe.

2017-08-14 Thread Jacek Caban
Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=55678 Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=68538 Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=69088 Signed-off-by: Jacek Caban --- src/xlibi18n/lcWrap.c | 4 1 file changed, 4 insertions(+)

Re: [PATCH xserver] Make PixmapDirtyUpdateRec::src a DrawablePtr

2017-08-14 Thread Adam Jackson
On Tue, 2017-04-18 at 19:07 +0900, Michel Dänzer wrote: > From: Michel Dänzer > > This allows making the master screen's pixmap_dirty_list entries > explicitly reflect that we're now tracking the root window instead of > the screen pixmap, in order to allow Present page

Re: [PATCH libX11 1/3] Make _XCloseLC thread safe.

2017-08-14 Thread Adam Jackson
On Thu, 2017-08-10 at 23:04 +0200, Jacek Caban wrote: > Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=55678 > Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=68538 > Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=69088 > Signed-off-by: Jacek Caban > --- >

Re: [PATCH libX11 1/3] Make _XCloseLC thread safe.

2017-08-14 Thread walter harms
hi, can you provide a simple program does needs the patch to work ? re, wh Am 10.08.2017 23:04, schrieb Jacek Caban: > Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=55678 > Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=68538 > Fixes:

Re: [PATCH] xwayland: Avoid repeatedly looping through window ancestor chain

2017-08-14 Thread Pekka Paalanen
On Sat, 12 Aug 2017 03:45:39 +0200 Roman Gilg wrote: > On Thu, Aug 10, 2017 at 2:49 PM, Pekka Paalanen wrote: > > Assuming the theory is sound, how about you kept the old names and > > semantics of xwl_window_get() and xwl_window_from_window() but just >

Re: Second Feedback request for my GSoC project to improve Present support in Xwayland

2017-08-14 Thread Michel Dänzer
On 11/08/17 12:04 AM, Pekka Paalanen wrote: > On Mon, 7 Aug 2017 17:36:27 +0900 > Michel Dänzer wrote: >> On 04/08/17 06:57 PM, Roman Gilg wrote: >>> On Fri, Aug 4, 2017 at 8:44 AM, Michel Dänzer >> > wrote: >>> >>> On