frederik
/
xserver
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fb: Simplify logic, get rid of set but unused variable. 

wrapped is only useful is FB_ACCESS_WRAPPER is set; simplify the logic
accordingly, and only set it when that's defined.

The following warning goes away accordingly:
|   CC     libfb_la-fbarc.lo
| fbarc.c: In function 'fbPolyArc':
| fbarc.c:71:11: warning: variable 'wrapped' set but not used [-Wunused-but-set-variable]

Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>
Signed-off-by: Cyril Brulebois <kibi@debian.org>
This commit is contained in:
Cyril Brulebois 2011-05-24 18:33:17 +02:00
parent 9a8bfc2c06
commit 8da16898d9
1 changed files with 3 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
fb/fbarc.c
View File

@ -68,15 +68,12 @@ fbPolyArc (DrawablePtr pDrawable,
BoxRec box; BoxRec box;
int x2, y2; int x2, y2;
RegionPtr cclip; RegionPtr cclip;
int wrapped = 0; #ifdef FB_ACCESS_WRAPPER
int wrapped = 1;
#endif
cclip = fbGetCompositeClip (pGC); cclip = fbGetCompositeClip (pGC);
fbGetDrawable (pDrawable, dst, dstStride, dstBpp, dstXoff, dstYoff); fbGetDrawable (pDrawable, dst, dstStride, dstBpp, dstXoff, dstYoff);
#ifdef FB_ACCESS_WRAPPER
wrapped = 1;
#else
wrapped = 0;
#endif
while (narcs--) while (narcs--)
{ {
if (miCanZeroArc (parcs)) if (miCanZeroArc (parcs))