[FFmpeg-devel] [PATCH] avcodec/mediacodecdec_common: make stride and slice-height non-mandatory fields

[Matthieu Bouron]

On Wed, Apr 18, 2018 at 6:13 PM, Matthieu Bouron <matthieu.bouron at gmail.com>
wrote:

> On Mon, Apr 16, 2018 at 03:53:23PM +0200, Matthieu Bouron wrote:
> > On Wed, Apr 11, 2018 at 3:14 PM, Matthieu Bouron <
> matthieu.bouron at gmail.com>
> > wrote:
> >
> > > Fixes decoding on the Samsung Chromebook Pro which do not set the codec
> > > output format stride and slice-height fields.
> > > ---
> > >  libavcodec/mediacodecdec_common.c | 4 ++--
> > >  1 file changed, 2 insertions(+), 2 deletions(-)
> > >
> > > diff --git a/libavcodec/mediacodecdec_common.c
> b/libavcodec/mediacodecdec_
> > > common.c
> > > index e31adb487c..e59cf19aad 100644
> > > --- a/libavcodec/mediacodecdec_common.c
> > > +++ b/libavcodec/mediacodecdec_common.c
> > > @@ -385,10 +385,10 @@ static int mediacodec_dec_parse_format(
> AVCodecContext
> > > *avctx, MediaCodecDecConte
> > >      AMEDIAFORMAT_GET_INT32(s->width,  "width", 1);
> > >      AMEDIAFORMAT_GET_INT32(s->height, "height", 1);
> > >
> > > -    AMEDIAFORMAT_GET_INT32(s->stride, "stride", 1);
> > > +    AMEDIAFORMAT_GET_INT32(s->stride, "stride", 0);
> > >      s->stride = s->stride > 0 ? s->stride : s->width;
> > >
> > > -    AMEDIAFORMAT_GET_INT32(s->slice_height, "slice-height", 1);
> > > +    AMEDIAFORMAT_GET_INT32(s->slice_height, "slice-height", 0);
> > >      s->slice_height = s->slice_height > 0 ? s->slice_height :
> s->height;
> > >
> > >      if (strstr(s->codec_name, "OMX.Nvidia.")) {
> > > --
> > > 2.17.0
> > >
> > >
> > Ping.
>
> If there is no objection I will push the patch in a few hours (I would
> like to have it in the next release).
>
>
Patch pushed.

[...]