[MPlayer-dev-eng] [PATCH] Stop using deprecatet AVPacket.destruct

Roberto Togni rxt at rtogni.it
Mon Aug 17 23:18:41 CEST 2015


On Sun, 16 Aug 2015 01:17:24 +0200
Roberto Togni <rxt at rtogni.it> wrote:

> Hi,
>  this patch drops the usage of the destruct field in AVPacket from
> demuxer_lavf.
> 
> Thanks to  Andreas Cadhalpun for reporting it.
> 
> 
> I haven't found an easy way to replace it; since now AVPacket.data is a
> pointer into an AVBuffer (and not a plain malloc'd buffer), we can't
> just seal it like we did before and free it later.
> 
> The removed path was just an optimization to avoid copying the demuxed
> packet data.
> 
> Also note that because of the way the condition is written, the removed
> code was already inactive (we compare the destruct field with
> av_destruct_packet to enable it, but lavc now sets it to
> dummy_destruct_packer).
> 
> 
> Will apply shortly if no complain.
> 
Applied

Ciao,
 Roberto


More information about the MPlayer-dev-eng mailing list