From 028773611949186865ea6977eca9493b2fc57853 Mon Sep 17 00:00:00 2001
From: wm4 <wm4@nowhere>
Date: Fri, 12 Feb 2016 15:58:48 +0100
Subject: sub: remove always-true check

Confuses Coverity with FORWARD_NULL on the mp_err() at the end of the
function. These pointers are never NULL.

Fixes CID 1350059.
---
 sub/dec_sub.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'sub/dec_sub.c')

diff --git a/sub/dec_sub.c b/sub/dec_sub.c
index e8b63986ab..641a5ab1da 100644
--- a/sub/dec_sub.c
+++ b/sub/dec_sub.c
@@ -104,7 +104,7 @@ struct dec_sub *sub_create(struct mpv_global *global, struct demuxer *demuxer,
             .codec = sh->codec,
         };
 
-        if (sh->codec && sub->sd->driver->init(sub->sd) >= 0)
+        if (sub->sd->driver->init(sub->sd) >= 0)
             return sub;
 
         ta_set_parent(log, NULL);
-- 
cgit v1.2.3