Re: [FFmpeg-devel] [PATCH 23/29] avcodec/nvenc: remove YUVJ pixel formats usage
On Sun, 10 Dec 2017 14:14:30 +0100 Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > libavcodec/nvenc.c | 6 ++ > 1 file changed, 2 insertions(+), 4 deletions(-) > > diff --git a/libavcodec/nvenc.c b/libavcodec/nvenc.c > index 4a91d99720..6ef51adaf0 100644 > --- a/libavcodec/nvenc.c > +++ b/libavcodec/nvenc.c > @@ -911,8 +911,7 @@ static av_cold int > nvenc_setup_h264_config(AVCodecContext *avctx) vui->colourMatrix = > avctx->colorspace; vui->colourPrimaries = avctx->color_primaries; > vui->transferCharacteristics = avctx->color_trc; > -vui->videoFullRangeFlag = (avctx->color_range == AVCOL_RANGE_JPEG > -|| ctx->data_pix_fmt == AV_PIX_FMT_YUVJ420P || > ctx->data_pix_fmt == AV_PIX_FMT_YUVJ422P || ctx->data_pix_fmt == > AV_PIX_FMT_YUVJ444P); > +vui->videoFullRangeFlag = avctx->color_range == AVCOL_RANGE_JPEG; > > vui->colourDescriptionPresentFlag = > (avctx->colorspace != 2 || avctx->color_primaries != 2 || > avctx->color_trc != 2); @@ -999,8 +998,7 @@ static av_cold int > nvenc_setup_hevc_config(AVCodecContext *avctx) vui->colourMatrix = > avctx->colorspace; vui->colourPrimaries = avctx->color_primaries; > vui->transferCharacteristics = avctx->color_trc; > -vui->videoFullRangeFlag = (avctx->color_range == AVCOL_RANGE_JPEG > -|| ctx->data_pix_fmt == AV_PIX_FMT_YUVJ420P || > ctx->data_pix_fmt == AV_PIX_FMT_YUVJ422P || ctx->data_pix_fmt == > AV_PIX_FMT_YUVJ444P); > +vui->videoFullRangeFlag = avctx->color_range == AVCOL_RANGE_JPEG; > > vui->colourDescriptionPresentFlag = > (avctx->colorspace != 2 || avctx->color_primaries != 2 || > avctx->color_trc != 2); Looks fine. --phil ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
[FFmpeg-devel] [PATCH 23/29] avcodec/nvenc: remove YUVJ pixel formats usage
Signed-off-by: Paul B Mahol --- libavcodec/nvenc.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/libavcodec/nvenc.c b/libavcodec/nvenc.c index 4a91d99720..6ef51adaf0 100644 --- a/libavcodec/nvenc.c +++ b/libavcodec/nvenc.c @@ -911,8 +911,7 @@ static av_cold int nvenc_setup_h264_config(AVCodecContext *avctx) vui->colourMatrix = avctx->colorspace; vui->colourPrimaries = avctx->color_primaries; vui->transferCharacteristics = avctx->color_trc; -vui->videoFullRangeFlag = (avctx->color_range == AVCOL_RANGE_JPEG -|| ctx->data_pix_fmt == AV_PIX_FMT_YUVJ420P || ctx->data_pix_fmt == AV_PIX_FMT_YUVJ422P || ctx->data_pix_fmt == AV_PIX_FMT_YUVJ444P); +vui->videoFullRangeFlag = avctx->color_range == AVCOL_RANGE_JPEG; vui->colourDescriptionPresentFlag = (avctx->colorspace != 2 || avctx->color_primaries != 2 || avctx->color_trc != 2); @@ -999,8 +998,7 @@ static av_cold int nvenc_setup_hevc_config(AVCodecContext *avctx) vui->colourMatrix = avctx->colorspace; vui->colourPrimaries = avctx->color_primaries; vui->transferCharacteristics = avctx->color_trc; -vui->videoFullRangeFlag = (avctx->color_range == AVCOL_RANGE_JPEG -|| ctx->data_pix_fmt == AV_PIX_FMT_YUVJ420P || ctx->data_pix_fmt == AV_PIX_FMT_YUVJ422P || ctx->data_pix_fmt == AV_PIX_FMT_YUVJ444P); +vui->videoFullRangeFlag = avctx->color_range == AVCOL_RANGE_JPEG; vui->colourDescriptionPresentFlag = (avctx->colorspace != 2 || avctx->color_primaries != 2 || avctx->color_trc != 2); -- 2.11.0 ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel