Re: [PATCH] sdksyms.sh: Make sdksyms.sh work with gcc5.

2015-03-03 Thread Egbert Eich
Hans asked for other suggestions to fix the gcc5 breakage of sdksyms.sh. So here is my take on it. Egbert Eich (1): symbols: Fix sdksyms.sh to cope with gcc5 hw/xfree86/sdksyms.sh | 14 +- 1 file changed, 13 insertions(+), 1 deletion(-) -- 1.8.4.5 __

[PATCH] symbols: Fix sdksyms.sh to cope with gcc5

2015-03-03 Thread Egbert Eich
Gcc5 adds additional lines stating line numbers before and after __attribute__() which need to be skipped. Signed-off-by: Egbert Eich --- hw/xfree86/sdksyms.sh | 14 +- 1 file changed, 13 insertions(+), 1 deletion(-) diff --git a/hw/xfree86/sdksyms.sh b/hw/xfree86/sdksyms.sh index 2

Re: [PATCH] symbols: Fix sdksyms.sh to cope with gcc5

2015-03-03 Thread Daniel Stone
Hi, On 3 March 2015 at 15:27, Egbert Eich wrote: > Gcc5 adds additional lines stating line numbers before and > after __attribute__() which need to be skipped. > > Signed-off-by: Egbert Eich I barely know my awk from my sed, but this fixes the build for me on Fedora 22. Tested-by: Daniel Stone

Re: [PATCH 2/2] exa: Verify non-null pSrc in mgaDownloadFromScreen()

2015-03-03 Thread Tormod Volden
On Tue, Mar 3, 2015 at 3:18 AM, Michel Dänzer wrote: > On 03.03.2015 05:01, Tormod Volden wrote: >> mgaDownloadFromScreen(PixmapPtr pSrc, int x, int y, int w, int h, >>char *dst, int dst_pitch) >> { >> +if (!pSrc) >> + return FALSE; >> + >> PMGA(pSrc); >> >>

Re: [PATCH] damage: Only track extents where possible

2015-03-03 Thread Adam Jackson
On Thu, 2015-02-26 at 22:01 +, Chris Wilson wrote: > On Thu, Feb 26, 2015 at 02:58:45PM -0500, Adam Jackson wrote: > > So I guess the question is whether automatic windows should take the > > bounding box snap too, and I guess whichever method makes x11perf look > > better when 'xcompmgr -a' is

Re: [PATCH] symbols: Fix sdksyms.sh to cope with gcc5

2015-03-03 Thread Keith Packard
Egbert Eich writes: > Gcc5 adds additional lines stating line numbers before and > after __attribute__() which need to be skipped. > > Signed-off-by: Egbert Eich If someone wants to offer to review this, that'd be awesome. If I don't hear anything fairly soon (today-ish), I'll go ahead and just

Re: [PATCH 2/2] exa: Verify non-null pSrc in mgaDownloadFromScreen()

2015-03-03 Thread Alex Deucher
On Tue, Mar 3, 2015 at 2:21 PM, Tormod Volden wrote: > On Tue, Mar 3, 2015 at 3:18 AM, Michel Dänzer wrote: >> On 03.03.2015 05:01, Tormod Volden wrote: >>> mgaDownloadFromScreen(PixmapPtr pSrc, int x, int y, int w, int h, >>>char *dst, int dst_pitch) >>> { >>> +if (!

Re: [PATCH 2/2] exa: Verify non-null pSrc in mgaDownloadFromScreen()

2015-03-03 Thread Tormod Volden
On Tue, Mar 3, 2015 at 9:21 PM, Alex Deucher wrote: > On Tue, Mar 3, 2015 at 2:21 PM, Tormod Volden wrote: >> On Tue, Mar 3, 2015 at 3:18 AM, Michel Dänzer wrote: >>> About patch 1, since mgaDownloadFromScreen doesn't do anything fancier >>> than memcpy, you can just remove it altogether and leave

Re: [PATCH 2/2] exa: Verify non-null pSrc in mgaDownloadFromScreen()

2015-03-03 Thread Alex Deucher
On Tue, Mar 3, 2015 at 3:50 PM, Tormod Volden wrote: > On Tue, Mar 3, 2015 at 9:21 PM, Alex Deucher wrote: >> On Tue, Mar 3, 2015 at 2:21 PM, Tormod Volden wrote: >>> On Tue, Mar 3, 2015 at 3:18 AM, Michel Dänzer wrote: About patch 1, since mgaDownloadFromScreen doesn't do anything fancier >>

[PATCH:xkbcomp] Stop including in parseutils.c

2015-03-03 Thread Alan Coopersmith
I can not find any point in our git history at which alloca() or the Xalloca.h macros ALLOCATE_LOCAL() & DEALLOCATE_LOCAL() were actually used in this source file. Signed-off-by: Alan Coopersmith --- parseutils.c |1 - 1 file changed, 1 deletion(-) diff --git a/parseutils.c b/parseutils.c i

[PATCH xf86-input-libinput] Up the scroll dist value for touchpads

2015-03-03 Thread Peter Hutterer
For source FINGER and CONTINUOUS, the axis value is the same as relative motion - but scrolling in X usually doesn't have the same speed as finger movement, it's a lot coarser. We don't know ahead of time where we'll get the scroll events from. Set a default scroll distance of 15 and multiply any

Re: [PATCH 2/2] exa: Verify non-null pSrc in mgaDownloadFromScreen()

2015-03-03 Thread Michel Dänzer
On 04.03.2015 07:07, Alex Deucher wrote: > On Tue, Mar 3, 2015 at 3:50 PM, Tormod Volden wrote: >> On Tue, Mar 3, 2015 at 9:21 PM, Alex Deucher wrote: >>> On Tue, Mar 3, 2015 at 2:21 PM, Tormod Volden wrote: On Tue, Mar 3, 2015 at 3:18 AM, Michel Dänzer wrote: > About patch 1, since mgaDo