Reminder: 2016 X.Org Board of Directors Elections Nomination period is NOW

2016-03-12 Thread Peter Hutterer
We are seeking nominations for candidates for election to the X.Org Foundation Board of Directors. All X.Org Foundation members are eligible for election to the board. Nominations for the 2016 election are now open and will remain open until 23:59 UTC on 15 March 2016. The Board consists of

Re: [PATCH] modesetting: allow switching from software to hardware cursors.

2016-03-12 Thread Michael Thayer
Hello Jasper, My case is virtual machine cursor pass-through. VirtualBox has a setting which lets the user enable or disable guest-host cursor integration. When it is enabled, the guest pointer position tracks the host one, and the guest cursor is rendered by the host. When it is

Re: [PATCH:makedepend] Use do { ... } while(0) idiom to make debug() macro safer

2016-03-12 Thread Jasper St. Pierre
"Safe multi-line macros" have been suggested before, and turned down. The C extension that gcc supports is statement expressions: https://gcc.gnu.org/onlinedocs/gcc/Statement-Exprs.html On Sat, Mar 12, 2016 at 11:15 AM, walter harms wrote: > > > Am 12.03.2016 19:43, schrieb Matt

Re: [PATCH] modesetting: allow switching from software to hardware cursors.

2016-03-12 Thread Jasper St. Pierre
Can you give an example of when drmSetCursor2 would fail sometimes but not always, enough to switch back and forth actively? I'm not a huge fan of the patch because now errors could cause really bizarre double-cursor, or no-cursor flickering, since there is no guarantee that the swapover happens

Re: [PATCH:makedepend] Use do { ... } while(0) idiom to make debug() macro safer

2016-03-12 Thread walter harms
Am 12.03.2016 19:43, schrieb Matt Turner: > On Sat, Mar 12, 2016 at 10:24 AM, walter harms wrote: >> >> >> Am 12.03.2016 19:15, schrieb Alan Coopersmith: >>> Cleans up several -Wempty-body warnings from gcc 5.3 >>> >>> Signed-off-by: Alan Coopersmith

[PATCH:xf86-video-s3] When checking malloc for success, it helps to use == NULL, instead of =

2016-03-12 Thread Alan Coopersmith
Fixes gcc 5.3 compiler warning: s3_bios.c: In function 'find_bios_string': s3_bios.c:49:2: warning: suggest parentheses around assignment used as truth value [-Wparentheses] if (bios = NULL) ^ [Tested by compiling only, as I have no S3 hardware, but clearly no one else has tested this

Re: [PATCH:makedepend] Use do { ... } while(0) idiom to make debug() macro safer

2016-03-12 Thread Matt Turner
On Sat, Mar 12, 2016 at 10:24 AM, walter harms wrote: > > > Am 12.03.2016 19:15, schrieb Alan Coopersmith: >> Cleans up several -Wempty-body warnings from gcc 5.3 >> >> Signed-off-by: Alan Coopersmith >> --- >> def.h | 4 ++-- >> 1 file changed, 2

Re: [PATCH:makedepend] Use do { ... } while(0) idiom to make debug() macro safer

2016-03-12 Thread walter harms
Am 12.03.2016 19:15, schrieb Alan Coopersmith: > Cleans up several -Wempty-body warnings from gcc 5.3 > > Signed-off-by: Alan Coopersmith > --- > def.h | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/def.h b/def.h > index

[PATCH:makedepend] Use do { ... } while(0) idiom to make debug() macro safer

2016-03-12 Thread Alan Coopersmith
Cleans up several -Wempty-body warnings from gcc 5.3 Signed-off-by: Alan Coopersmith --- def.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/def.h b/def.h index 1930cde..59670a9 100644 --- a/def.h +++ b/def.h @@ -82,9 +82,9 @@ extern int

Re: [PATCH] modesetting: allow switching from software to hardware cursors.

2016-03-12 Thread Michael Thayer
Hello, I would just like to politely draw attention to this again. I would assume from the silence that the people who might review it don't think it is a completely wrong thing (that would not take long to say) but don't have the spare cycles to take a closer look. Any idea when someone