[FFmpeg-devel] [PATCH 3/3] avformat/aiffenc: Remove wrong and redundant check
Andreas Rheinhardt
andreas.rheinhardt at gmail.com
Wed Oct 2 07:04:12 EEST 2019
The check "if (!pb->seekable & AVIO_SEEKABLE_NORMAL)" is wrong, because
! has higher precendence than &. But it is also redundant, because this
part of the code is only ever reached when the AVIO_SEEKABLE_NORMAL flag
is set for pb. So simply remove the check.
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at gmail.com>
---
libavformat/aiffenc.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/libavformat/aiffenc.c b/libavformat/aiffenc.c
index 0b837cd264..d09c9afb95 100644
--- a/libavformat/aiffenc.c
+++ b/libavformat/aiffenc.c
@@ -49,9 +49,6 @@ static int put_id3v2_tags(AVFormatContext *s, AIFFOutputContext *aiff)
AVIOContext *pb = s->pb;
AVPacketList *pict_list = aiff->pict_list;
- if (!pb->seekable & AVIO_SEEKABLE_NORMAL)
- return 0;
-
if (!s->metadata && !aiff->pict_list)
return 0;
--
2.21.0
More information about the ffmpeg-devel
mailing list