2018-08-23 11:11 GMT+02:00, Paul B Mahol <[email protected]>: > On 8/23/18, Carl Eugen Hoyos <[email protected]> wrote: >> 2018-08-22 21:24 GMT+02:00, Paul B Mahol <[email protected]>: >>> On 8/22/18, Carl Eugen Hoyos <[email protected]> wrote: >>>> 2018-08-22 18:00 GMT+02:00, Paul B Mahol <[email protected]>: >>>> >>>>> + 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 [email protected] http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
