On 14.02.2015 19:36, Chris Wilson wrote: > We pass the pPixmap->drawable.id to the ShmDetachSegment function after > the pPixmap is freed. Fortunately, we don't use the value inside > ShmDetachSegment and can simply pass zero instead. > > Signed-off-by: Chris Wilson <[email protected]> > --- > Xext/shm.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/Xext/shm.c b/Xext/shm.c > index db9d474..52d9974 100644 > --- a/Xext/shm.c > +++ b/Xext/shm.c > @@ -260,7 +260,7 @@ ShmDestroyPixmap(PixmapPtr pPixmap) > pScreen->DestroyPixmap = ShmDestroyPixmap; > > if (shmdesc) > - ShmDetachSegment(shmdesc, pPixmap->drawable.id); > + ShmDetachSegment(shmdesc, 0); > > return ret; > } > @@ -427,7 +427,7 @@ ProcShmAttach(ClientPtr client) > > /*ARGSUSED*/ static int > ShmDetachSegment(void *value, /* must conform to DeleteType */ > - XID shmseg) > + XID unused) > { > ShmDescPtr shmdesc = (ShmDescPtr) value; > ShmDescPtr *prev; >
It's a static function, so you can just remove the unused parameter. -- Earthling Michel Dänzer | http://www.amd.com Libre software enthusiast | Mesa and X developer _______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
