Moritz Barsnick <barsnick <at> gmx.net> writes: > > + case AV_PIX_FMT_GBRP: > > + case AV_PIX_FMT_GBRP10: > > + case AV_PIX_FMT_GBRP12: > > + ctx->params->vui.matrixCoeffs = AVCOL_SPC_RGB; > > + ctx->params->vui.bEnableVideoSignalTypePresentFlag = 1; > > + ctx->params->vui.bEnableColorDescriptionPresentFlag = 1; > > case AV_PIX_FMT_YUV444P: > > case AV_PIX_FMT_YUV444P10: > > case AV_PIX_FMT_YUV444P12: > > Fallthrough intended?
Definitely. > Might be worth documenting then, because reading > such switch/cases can become confusing. I believe we don't usually add a comment here, several files distinguish in a similar way between gbrp and yuv444p. Carl Eugen _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
