From 80f59c8a4f51ccc27653a2de2dee2563b1e911f1 Mon Sep 17 00:00:00 2001 From: Cyril Brulebois Date: Mon, 31 Jan 2011 06:22:39 +0100 Subject: [PATCH] xvmc: Stop using uninitialized variable. MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 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 Signed-off-by: Chris Wilson --- src/xvmc/intel_xvmc.c | 4 +--- 1 file changed, 1 insertion(+), 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; } /*