diff options
author | Cyril Brulebois <kibi@debian.org> | 2011-01-31 06:22:39 +0100 |
---|---|---|
committer | Chris Wilson <chris@chris-wilson.co.uk> | 2011-01-31 10:00:31 +0000 |
commit | 80f59c8a4f51ccc27653a2de2dee2563b1e911f1 (patch) | |
tree | 3d0eade804ff02c0c1b5c8c3e09fa17999539099 | |
parent | 86284abafbe1364f82509d2656c82ba11cdce6cc (diff) |
xvmc: Stop using uninitialized variable.
The actual code was removed in 5f64122551, where the get_surface_status
driver callback was dropped. Just return Success if there were no issues
with the parameter.
Get rid of the following with CFLAGS="-Wall -Werror":
| CC intel_xvmc.lo
| cc1: warnings being treated as errors
| intel_xvmc.c: In function ‘XvMCSyncSurface’:
| intel_xvmc.c:677: error: ‘ret’ may be used uninitialized in this function
| intel_xvmc.c:672: note: ‘ret’ was declared here
Signed-off-by: Cyril Brulebois <kibi@debian.org>
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
-rw-r--r-- | src/xvmc/intel_xvmc.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/src/xvmc/intel_xvmc.c b/src/xvmc/intel_xvmc.c index d89cda22..1b969457 100644 --- a/src/xvmc/intel_xvmc.c +++ b/src/xvmc/intel_xvmc.c @@ -669,12 +669,10 @@ _X_EXPORT Status XvMCPutSurface(Display * display, XvMCSurface * surface, */ _X_EXPORT Status XvMCSyncSurface(Display * display, XvMCSurface * surface) { - Status ret; - if (!display || !surface) return XvMCBadSurface; - return ret; + return Success; } /* |