[FFmpeg-devel] [PATCH 3/3] avformat/aiffenc: Remove wrong and redundant check

Matthieu Bouron matthieu.bouron at gmail.com
Wed Oct 2 10:09:28 EEST 2019


On Wed, Oct 02, 2019 at 06:04:12AM +0200, Andreas Rheinhardt wrote:
> 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

LGTM.

-- 
Matthieu B.


More information about the ffmpeg-devel mailing list