On 6/17/2023 7:07 PM, Leo Izen wrote:
> - if (strcmp(col, pri) || strcmp(col, trc)) {
> + if (enc->colorspace == AVCOL_SPC_RGB &&
> + enc->color_primaries == AVCOL_PRI_BT709 &&
> + enc->color_trc == AVCOL_TRC_IEC61966_2_1) {
> + av_bprintf(&bprint, "sRGB, ");
> + } else if (strcmp(col, pri) || strcmp(col, trc)) {
I real dont think special casing a single colorspace like this is a good idea,
nor do I think hiding the explicit values is.
Same reason we do not special case e.g. HLG.
- Derek
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
