diff options
author | wm4 <wm4@nowhere> | 2014-11-21 05:12:55 +0100 |
---|---|---|
committer | wm4 <wm4@nowhere> | 2014-11-21 05:18:09 +0100 |
commit | 85fb2af369e9ea8638beaf3a34865e3553329a02 (patch) | |
tree | 5d73b2ea57a66fc8e9ef33358a4aa89071a2e60f /player/command.c | |
parent | 4704fab82c084dbcca52b5e75f7a36877921dbd9 (diff) | |
download | mpv-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.c | 3 |
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[] = |