On Thu, Mar 12, 2015 at 10:52 PM, wm4 <[email protected]> wrote: > On Thu, 12 Mar 2015 18:36:51 -0400 > Bradley Sepos <[email protected]> wrote: > >> Updated version of patch by Carl Eugen Hoyos: >> https://patches.libav.org/patch/42533/ >> https://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=79209f5d6ca31c73260f1c5f5dbaa8395102d9f1 >> >> Works in limited testing. >> >> --- >> libavcodec/libx264.c | 6 ++++++ >> 1 file changed, 6 insertions(+) >> >> diff --git a/libavcodec/libx264.c b/libavcodec/libx264.c >> index >> 6e707a867a850190677c9c778404c91a1d4e9390..8e19be47828a701db7dc46d56a40e2935c3e79d2 >> 100644 >> --- a/libavcodec/libx264.c >> +++ b/libavcodec/libx264.c >> @@ -290,8 +290,10 @@ static int convert_pix_fmt(enum AVPixelFormat pix_fmt) >> case AV_PIX_FMT_YUV420P9: >> case AV_PIX_FMT_YUV420P10: return X264_CSP_I420; >> case AV_PIX_FMT_YUV422P: >> + case AV_PIX_FMT_YUVJ422P: >> case AV_PIX_FMT_YUV422P10: return X264_CSP_I422; >> case AV_PIX_FMT_YUV444P: >> + case AV_PIX_FMT_YUVJ444P: >> case AV_PIX_FMT_YUV444P9: >> case AV_PIX_FMT_YUV444P10: return X264_CSP_I444; >> case AV_PIX_FMT_NV12: return X264_CSP_NV12; >> @@ -487,6 +489,8 @@ static av_cold int X264_init(AVCodecContext *avctx) >> x4->params.i_slice_count = avctx->slices; >> >> x4->params.vui.b_fullrange = avctx->pix_fmt == AV_PIX_FMT_YUVJ420P || >> + avctx->pix_fmt == AV_PIX_FMT_YUVJ422P || >> + avctx->pix_fmt == AV_PIX_FMT_YUVJ444P || >> avctx->color_range == AVCOL_RANGE_JPEG; > > Wait, doesn't FFmpeg now set the range automatically based on the pixel > format in utils.c? At least the decoding path does this.
Yes it does, but it doesn't change the pixel format, so this is actually needed to avoid format conversion. Also watch out the project name, people are on their toes today ;) -- Vittorio _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
