diff options
| author | Cyril Brulebois <kibi@debian.org> | 2011-01-31 06:22:39 +0100 |
|---|---|---|
| committer | Owain G. Ainsworth <oga@openbsd.org> | 2011-03-16 20:16:34 +0000 |
| commit | a64629ae324bc0af22a01da2ba18ad8445b85890 (patch) | |
| tree | 07f72965439a01432a85527b4563f4f7e74618d6 | |
| parent | 5609f8e235fd30ae3e8c71a31727ce49bce57c24 (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>
(cherry picked from commit 80f59c8a4f51ccc27653a2de2dee2563b1e911f1)
Signed-off-by: Owain G. Ainsworth <oga@openbsd.org>
| -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; } /* |