[FFmpeg-devel] [PATCH] avpacket: do not copy data when buf ref is available.

Clément Bœsch ubitux at gmail.com
Wed Mar 13 01:44:46 CET 2013


On Wed, Mar 13, 2013 at 12:09:54AM +0100, Michael Niedermayer wrote:
[...]
> > Correct wording: av_dup_packet should not be affected, it's a noop in case
> > pkt->buf is *NOT* NULL. The behaviour of copy_packet_data() is unchanged
> > in case pkt->buf is NULL, which is the only case where copy_packet_data
> > can be called from av_dup_packet().
> > 
> > On a side note, while the behaviour of av_dup_packet() is unchanged, I'm
> > not sure it is really correct. But this is not what the patch is meant to
> > fix anyway.
> 
> patch LGTM
> 

Description amended and pushed. Thanks.

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130313/d508a67f/attachment.asc>


More information about the ffmpeg-devel mailing list