summaryrefslogtreecommitdiffstats
path: root/player/command.c
diff options
context:
space:
mode:
authorwm4 <wm4@nowhere>2014-11-21 05:12:55 +0100
committerwm4 <wm4@nowhere>2014-11-21 05:18:09 +0100
commit85fb2af369e9ea8638beaf3a34865e3553329a02 (patch)
tree5d73b2ea57a66fc8e9ef33358a4aa89071a2e60f /player/command.c
parent4704fab82c084dbcca52b5e75f7a36877921dbd9 (diff)
downloadmpv-85fb2af369e9ea8638beaf3a34865e3553329a02.tar.bz2
mpv-85fb2af369e9ea8638beaf3a34865e3553329a02.tar.xz
Remove some superfluous NULL checks
In all of these situations, NULL is logically not allowed, making the checks redundant. Coverity complained about accessing the pointers before checking them for NULL later.
Diffstat (limited to 'player/command.c')
-rw-r--r--player/command.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/player/command.c b/player/command.c
index 10a779f638..30768e9b71 100644
--- a/player/command.c
+++ b/player/command.c
@@ -1969,8 +1969,7 @@ static int probe_deint_filters(struct MPContext *mpctx)
if (check_output_format(mpctx, IMGFMT_VDPAU)) {
char filter[80] = "vdpaupp:deint=yes";
int pref = 0;
- if (mpctx->video_out)
- vo_control(mpctx->video_out, VOCTRL_GET_PREF_DEINT, &pref);
+ vo_control(mpctx->video_out, VOCTRL_GET_PREF_DEINT, &pref);
pref = pref < 0 ? -pref : pref;
if (pref > 0 && pref <= 4) {
const char *types[] =