[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