]> git.sesse.net Git - ffmpeg/commitdiff
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 d5bc9230bbf70ac45b264fa9cc4a12cbf5a1d2d5..22108adc99994b268840ac4dbc1441e442da98a3 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);