matroskadec: Fix a double negation typo
authorMichael Niedermayer <michaelni@gmx.at>
Sat, 20 Oct 2012 14:58:33 +0000 (16:58 +0200)
committerMartin Storsjö <martin@martin.st>
Wed, 2 Jul 2014 17:15:42 +0000 (20:15 +0300)
This typo has existed since this code was added in c16582579.
Newer versions of clang pointed out that this comparison always
was true (since the result of the negation is either 0 or 1, while
AVDISCARD_ALL has the value 48).

Signed-off-by: Martin Storsjö <martin@martin.st>
libavformat/matroskadec.c

index d5bc923..22108ad 100644 (file)
@@ -2560,7 +2560,7 @@ static int matroska_read_seek(AVFormatContext *s, int stream_index,
         tracks[i].audio.buf_timecode   = AV_NOPTS_VALUE;
         tracks[i].end_timecode         = 0;
         if (tracks[i].type == MATROSKA_TRACK_TYPE_SUBTITLE &&
-            !tracks[i].stream->discard != AVDISCARD_ALL) {
+            tracks[i].stream->discard != AVDISCARD_ALL) {
             index_sub = av_index_search_timestamp(
                 tracks[i].stream, st->index_entries[index].timestamp,
                 AVSEEK_FLAG_BACKWARD);