[FFmpeg-devel] [PATCH 1/7] parse_packet: reset pkt->pos after it has been used

Stefano Sabatini stefasab at gmail.com
Wed Jul 25 00:36:15 CEST 2012


On date Tuesday 2012-07-24 18:15:33 +0200, Michael Niedermayer encoded:
> This should have no effect as it was not used in this case
> 
> Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
> ---
>  libavformat/utils.c |    1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/libavformat/utils.c b/libavformat/utils.c
> index 5bc79f4..e348fbd 100644
> --- a/libavformat/utils.c
> +++ b/libavformat/utils.c
> @@ -1184,6 +1184,7 @@ static int parse_packet(AVFormatContext *s, AVPacket *pkt, int stream_index)
>                                 pkt->pts, pkt->dts, pkt->pos);
>  
>          pkt->pts = pkt->dts = AV_NOPTS_VALUE;
> +        pkt->pos = -1;
>          /* increment read pointer */
>          data += len;
>          size -= len;

LGTM, assuming it has no side effects and it is useful.
-- 
FFmpeg = Fierce and Formidable Merciless Programmable Earthshaking Guru


More information about the ffmpeg-devel mailing list