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

[Matthieu Bouron]

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).

-- 
Matthieu B.