diff options
| author | Cyril Brulebois <kibi@debian.org> | 2011-05-24 18:33:17 +0200 |
|---|---|---|
| committer | Cyril Brulebois <kibi@debian.org> | 2011-06-16 00:43:49 +0200 |
| commit | 8da16898d9de5da1f80b789f0b371070968285f8 (patch) | |
| tree | 28512da8691a30e51982ba282db21c3c785b464e /fb | |
| parent | 9a8bfc2c06f0a9206b64ae01a783cf5513f021a6 (diff) | |
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>
Diffstat (limited to 'fb')
| -rw-r--r-- | fb/fbarc.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/fb/fbarc.c b/fb/fbarc.c index a0c5343e0..e54c80491 100644 --- a/fb/fbarc.c +++ b/fb/fbarc.c @@ -68,15 +68,12 @@ fbPolyArc (DrawablePtr pDrawable, BoxRec box; int x2, y2; RegionPtr cclip; - int wrapped = 0; +#ifdef FB_ACCESS_WRAPPER + int wrapped = 1; +#endif cclip = fbGetCompositeClip (pGC); fbGetDrawable (pDrawable, dst, dstStride, dstBpp, dstXoff, dstYoff); -#ifdef FB_ACCESS_WRAPPER - wrapped = 1; -#else - wrapped = 0; -#endif while (narcs--) { if (miCanZeroArc (parcs)) |