[MPlayer-dev-eng] [PATCH] Update vo_png to use updated API

Alexander Strasser eclipse7 at gmx.net
Wed Aug 14 12:29:16 CEST 2013


On 2013-08-08 22:18 +0200, Alexander Strasser wrote:
> On 2013-07-31 02:11 +0200, Alexander Strasser wrote:
> > On 2013-07-14 21:56 +0200, Jiang Jiang wrote:
> > > avcodec_encode_video is deprecated.
[diff snipped]
> > 
> >   LGTM but it is pretty late here. I will re-review next week and apply
> > if I hear no objections and find no flaws.
> 
>   I am going to commit this soon. Together with a simplified error
> path and the addition of a call to av_free_packet. I think it is more
> correct that way, even if today it won't do much. In future it makes
> sure the packet and potentially attached side data will be freed
> accordingly.
> 
>   Looks like this when viewed as an incremental patch:
> 
[diff snipped]
> 
> >   BTW using the newer API seems also to fix the less common cases with
> > small video resolution where we currently would let avcodec_encode_video
> > fail demanding a minimum size.

  Applied now including error clean up in a follow-up commit.

Thanks for the patch and sorry for the slightly long delay,
  Alexander
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20130814/446f8379/attachment.asc>


More information about the MPlayer-dev-eng mailing list