[FFmpeg-devel] [PATCH] avformat/avienc: Remove unneeded seekable tests

Reimar Döffinger Reimar.Doeffinger at gmx.de
Sun Mar 6 11:53:27 CET 2016


On Sat, Mar 05, 2016 at 06:38:40PM +0100, Michael Niedermayer wrote:
> diff --git a/libavformat/avienc.c b/libavformat/avienc.c
> index 0cfffb7..357dd34 100644
> --- a/libavformat/avienc.c
> +++ b/libavformat/avienc.c
> @@ -306,8 +306,7 @@ static int avi_write_header(AVFormatContext *s)
>              avio_wl32(pb, enc->codec_tag);
>          else
>              avio_wl32(pb, 1);
> -        if (enc->codec_type == AVMEDIA_TYPE_VIDEO && pb->seekable)
> -            avist->strh_flags_offset = avio_tell(pb);
> +        avist->strh_flags_offset = avio_tell(pb);
>          avio_wl32(pb, 0); /* flags */
>          avio_wl16(pb, 0); /* priority */
>          avio_wl16(pb, 0); /* language */
> @@ -369,8 +368,7 @@ static int avi_write_header(AVFormatContext *s)
>                      && enc->pix_fmt == AV_PIX_FMT_RGB555LE
>                      && enc->bits_per_coded_sample == 15)
>                      enc->bits_per_coded_sample = 16;
> -                if (pb->seekable)
> -                    avist->pal_offset = avio_tell(pb) + 40;
> +                avist->pal_offset = avio_tell(pb) + 40;
>                  ff_put_bmp_header(pb, enc, ff_codec_bmp_tags, 0, 0);
>                  pix_fmt = avpriv_find_pix_fmt(avpriv_pix_fmt_bps_avi,
>                                                enc->bits_per_coded_sample);

Looks fine to me.


More information about the ffmpeg-devel mailing list