summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorCyril Brulebois <kibi@debian.org>2011-01-31 06:22:39 +0100
committerChris Wilson <chris@chris-wilson.co.uk>2011-01-31 10:00:31 +0000
commit80f59c8a4f51ccc27653a2de2dee2563b1e911f1 (patch)
tree3d0eade804ff02c0c1b5c8c3e09fa17999539099
parent86284abafbe1364f82509d2656c82ba11cdce6cc (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.c4
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;
}
/*