Re: [FFmpeg-devel] [PATCH 1/3] avcodec: use av_frame_free() for coded_frame

2015-03-02 Thread Clément Bœsch
On Mon, Mar 02, 2015 at 10:01:57AM +0100, Clément Bœsch wrote: > On Mon, Mar 02, 2015 at 08:31:16AM +, Paul B Mahol wrote: > > On 2/28/15, Clement Boesch wrote: > > > On Sat, Feb 28, 2015 at 10:38:32AM +0100, Clement Boesch wrote: > > >> --- > > >> libavcodec/avuienc.c| 2 +- > > >

Re: [FFmpeg-devel] [PATCH 1/3] avcodec: use av_frame_free() for coded_frame

2015-03-02 Thread Nicolas George
Le duodi 12 ventôse, an CCXXIII, Clement Boesch a écrit : > I was confused by the output of FATE, it's not in r210, it's in prores: > *avctx->coded_frame = *pic; This code is invalid: if a future version of lavu adds fields at the end of AVFrame, they would not be copied. (I hoped w

Re: [FFmpeg-devel] [PATCH 1/3] avcodec: use av_frame_free() for coded_frame

2015-03-02 Thread Paul B Mahol
On 3/2/15, Clement Boesch wrote: > On Mon, Mar 02, 2015 at 08:31:16AM +, Paul B Mahol wrote: >> On 2/28/15, Clement Boesch wrote: >> > On Sat, Feb 28, 2015 at 10:38:32AM +0100, Clement Boesch wrote: >> >> --- >> >> libavcodec/avuienc.c| 2 +- >> >> libavcodec/libopenjpegenc.c

Re: [FFmpeg-devel] [PATCH 1/3] avcodec: use av_frame_free() for coded_frame

2015-03-02 Thread Clément Bœsch
On Mon, Mar 02, 2015 at 08:31:16AM +, Paul B Mahol wrote: > On 2/28/15, Clement Boesch wrote: > > On Sat, Feb 28, 2015 at 10:38:32AM +0100, Clement Boesch wrote: > >> --- > >> libavcodec/avuienc.c| 2 +- > >> libavcodec/libopenjpegenc.c | 4 ++-- > >> libavcodec/libtheoraenc.c

Re: [FFmpeg-devel] [PATCH 1/3] avcodec: use av_frame_free() for coded_frame

2015-03-02 Thread Paul B Mahol
On 2/28/15, Clement Boesch wrote: > On Sat, Feb 28, 2015 at 10:38:32AM +0100, Clement Boesch wrote: >> --- >> libavcodec/avuienc.c| 2 +- >> libavcodec/libopenjpegenc.c | 4 ++-- >> libavcodec/libtheoraenc.c | 2 +- >> libavcodec/libutvideoenc.cpp| 2 +- >> libavcodec/li

Re: [FFmpeg-devel] [PATCH 1/3] avcodec: use av_frame_free() for coded_frame

2015-02-28 Thread Clément Bœsch
On Sat, Feb 28, 2015 at 10:38:32AM +0100, Clément Bœsch wrote: > --- > libavcodec/avuienc.c| 2 +- > libavcodec/libopenjpegenc.c | 4 ++-- > libavcodec/libtheoraenc.c | 2 +- > libavcodec/libutvideoenc.cpp| 2 +- > libavcodec/libvpxenc.c | 2 +- > libavcodec/pror

[FFmpeg-devel] [PATCH 1/3] avcodec: use av_frame_free() for coded_frame

2015-02-28 Thread Clément Bœsch
--- libavcodec/avuienc.c| 2 +- libavcodec/libopenjpegenc.c | 4 ++-- libavcodec/libtheoraenc.c | 2 +- libavcodec/libutvideoenc.cpp| 2 +- libavcodec/libvpxenc.c | 2 +- libavcodec/proresenc_anatoliy.c | 2 +- libavcodec/proresenc_kostya.c | 2 +- libavcodec/r