diff options
author | Cyril Brulebois <kibi@debian.org> | 2011-05-24 18:33:11 +0200 |
---|---|---|
committer | Jeremy Huddleston <jeremyhu@apple.com> | 2011-06-29 21:38:33 -0700 |
commit | 1834e82b42a876195d12dda28c80004d37c4fa0c (patch) | |
tree | c044dcede42c0e34bb11501301ac2a8678cc3b07 /hw | |
parent | 9fab8b475de99848866c07bde962a5f6ed01b987 (diff) |
Xext: Fix buggy checks.
XvMCScreenKey is defined as XvMCScreenKeyRec's address, so will never be
NULL. Use the check proposed by Alan Coopersmith and Jeremy Huddleston
instead.
Those warnings go away accordingly:
| CC xvmc.lo
| xvmc.c: In function 'ProcXvMCListSubpictureTypes':
| xvmc.c:490:22: warning: the comparison will always evaluate as 'false' for the address of 'XvMCScreenKeyRec' will never be NULL [-Waddress]
| xvmc.c: In function 'XvMCExtensionInit':
| xvmc.c:671:21: warning: the comparison will always evaluate as 'false' for the address of 'XvMCScreenKeyRec' will never be NULL [-Waddress]
| xvmc.c: In function 'XvMCFindXvImage':
| xvmc.c:749:22: warning: the comparison will always evaluate as 'false' for the address of 'XvMCScreenKeyRec' will never be NULL [-Waddress]
Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Signed-off-by: Cyril Brulebois <kibi@debian.org>
(cherry picked from commit 9da5aa09d70e26dd4d2ef878a21317dae7bf24de)
Diffstat (limited to 'hw')
0 files changed, 0 insertions, 0 deletions