[FFmpeg-devel] [PATCH 05/11] avformat/mpjpegdec: Remove redundant initializations
Marton Balint
cus at passwd.hu
Tue Feb 11 01:18:03 EET 2020
On Tue, 7 Jan 2020, Andreas Rheinhardt wrote:
> The AVPacket destined for a demuxer's output has already been
> initialized before it reaches the demuxer.
>
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at gmail.com>
> ---
> libavformat/mpjpegdec.c | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/libavformat/mpjpegdec.c b/libavformat/mpjpegdec.c
> index 1e2ab0db1a..df2880412d 100644
> --- a/libavformat/mpjpegdec.c
> +++ b/libavformat/mpjpegdec.c
> @@ -331,9 +331,7 @@ static int mpjpeg_read_packet(AVFormatContext *s, AVPacket *pkt)
> int remaining = 0, len;
>
> const int read_chunk = 2048;
> - av_init_packet(pkt);
> - pkt->data = NULL;
> - pkt->size = 0;
> +
> pkt->pos = avio_tell(s->pb);
>
> while ((ret = ffio_ensure_seekback(s->pb, read_chunk - remaining)) >= 0 && /* we may need to return as much as all we've read back to the buffer */
Thanks, applied.
Regards,
Marton
More information about the ffmpeg-devel
mailing list