[FFmpeg-devel] [PATCH 5/5] lavc: add h264 mediacodec decoder

[Matthieu Bouron]

On Wed, Feb 17, 2016 at 06:39:46PM +0100, Matthieu Bouron wrote:
> On Mon, Feb 15, 2016 at 10:17:25PM +0100, Matthieu Bouron wrote:
> > On Mon, Feb 15, 2016 at 08:42:48PM +0100, Michael Niedermayer wrote:
> > > On Mon, Feb 15, 2016 at 06:52:30PM +0100, Matthieu Bouron wrote:
> > > > From: Matthieu Bouron <matthieu.bouron at stupeflix.com>
> > > > 
> > > > ---
> > > >  configure                       |    5 +
> > > >  libavcodec/Makefile             |    3 +
> > > >  libavcodec/allcodecs.c          |    1 +
> > > >  libavcodec/mediacodec_wrapper.c | 1522 +++++++++++++++++++++++++++++++++++++++
> > > >  libavcodec/mediacodec_wrapper.h |  123 ++++
> > > >  libavcodec/mediacodecdec.c      |  753 +++++++++++++++++++
> > > >  libavcodec/mediacodecdec.h      |   98 +++
> > > >  libavcodec/mediacodecdec_h264.c |  365 ++++++++++
> > > >  8 files changed, 2870 insertions(+)
> > > >  create mode 100644 libavcodec/mediacodec_wrapper.c
> > > >  create mode 100644 libavcodec/mediacodec_wrapper.h
> > > >  create mode 100644 libavcodec/mediacodecdec.c
> > > >  create mode 100644 libavcodec/mediacodecdec.h
> > > >  create mode 100644 libavcodec/mediacodecdec_h264.c
> > > 
> > > breaks fate:
> > > 
> > >     vsynth1-dnxhd-1080i-10bit
> > > TEST    vsynth1-dnxhd-1080i-colr
> > > TEST    vsynth1-dv
> > > --- ./tests/ref/fate/source     2016-02-15 13:45:07.088605740 +0100
> > > +++ tests/data/fate/source      2016-02-15 20:24:16.205110282 +0100
> > > @@ -27,3 +27,4 @@
> > >  compat/avisynth/windowsPorts/windows2linux.h
> > >  compat/float/float.h
> > >  compat/float/limits.h
> > > +libavcodec/mediacodec_wrapper.h
> > > Test source failed. Look at tests/data/fate/source.err for details.
> > > make: *** [fate-source] Error 1
> > > make: *** Waiting for unfinished jobs....
> > 
> > Thanks. Fixed locally by changing #define FF_MEDIACODEC_H to
> > AVCODEC_MEDIACODEC_WRAPPER_H in the relevant header.
> > 
> > Development branch updated.
> 
> Patch updated locally and development branch updated:
>   * add missing copyright notices
>   * correctly flush remaining frames when reaching end of stream.
> 
> I will send soon an updated version of the patch to the ml.
> 

Updated patch attached with the following differences from the first
patch:
  * add missing copyright notices
  * correctly flush remaining frames when reaching end of stream
  * properly set frame->pkt_pts

Matthieu
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-lavc-add-h264-mediacodec-decoder.patch
Type: text/x-diff
Size: 104469 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160219/1a64214c/attachment.patch>