2018-08-23 11:11 GMT+02:00, Paul B Mahol <one...@gmail.com>: > On 8/23/18, Carl Eugen Hoyos <ceffm...@gmail.com> wrote: >> 2018-08-22 21:24 GMT+02:00, Paul B Mahol <one...@gmail.com>: >>> On 8/22/18, Carl Eugen Hoyos <ceffm...@gmail.com> wrote: >>>> 2018-08-22 18:00 GMT+02:00, Paul B Mahol <one...@gmail.com>: >>>> >>>>> + switch (avctx->bits_per_coded_sample) { >>>>> + case 12: >>>>> + s->stride = 3LL * FFALIGN(avctx->width, 8) >> 1; >>>>> + break; >>>>> + default: >>>>> + return AVERROR_INVALIDDATA; >>>>> + } >>>> >>>> Why are the condition and the error needed? >>> >>> Because only that is supported. >> >> Do valid samples with other values exist? > > No.
Then I suggest to make the whole block above just: s->stride = 3LL * FFALIGN(avctx->width, 8) >> 1; Carl Eugen _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel