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

Hendrik Leppkes h.leppkes at gmail.com
Thu Apr 19 15:08:58 EEST 2018


On Thu, Apr 19, 2018 at 1:57 PM, Matthieu Bouron
<matthieu.bouron at gmail.com> wrote:
> 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.
>

If you want this in the release, you should also backport it to the
release/4.0 branch before the release ie made.

- Hendrik


More information about the ffmpeg-devel mailing list