Re: [FFmpeg-devel] [PATCH] libavformat/yuv4mpeg: Add color range support for Y4M Add color_range support in Y4M. Also set pixel format and color_range for YUVJ pixel formats.

2018-06-28 Thread Michael Niedermayer
On Thu, Jun 28, 2018 at 04:28:24PM +0800, Wang Cao wrote: > > > > If this is a type we are choosing then it would be probably better > > to use something else than mpeg/jpeg as names > > these are 2 standard comittees and their standards support more than > > one color range. > > maybe "full" and "

Re: [FFmpeg-devel] [PATCH] libavformat/yuv4mpeg: Add color range support for Y4M Add color_range support in Y4M. Also set pixel format and color_range for YUVJ pixel formats.

2018-06-28 Thread Wang Cao
> > If this is a type we are choosing then it would be probably better > to use something else than mpeg/jpeg as names > these are 2 standard comittees and their standards support more than > one color range. > maybe "full" and "limited" or some other terms may be better Thanks for pointing out th

Re: [FFmpeg-devel] [PATCH] libavformat/yuv4mpeg: Add color range support for Y4M Add color_range support in Y4M. Also set pixel format and color_range for YUVJ pixel formats.

2018-06-25 Thread Michael Niedermayer
On Thu, Jun 14, 2018 at 12:03:13AM +0800, Wang Cao wrote: > --- > libavformat/yuv4mpegdec.c | 8 > libavformat/yuv4mpegenc.c | 37 +++-- > 2 files changed, 43 insertions(+), 2 deletions(-) > > diff --git a/libavformat/yuv4mpegdec.c b/libavformat/yuv4mpegd

Re: [FFmpeg-devel] [PATCH] libavformat/yuv4mpeg: Add color range support for Y4M Add color_range support in Y4M. Also set pixel format and color_range for YUVJ pixel formats.

2018-06-25 Thread Sasi Inguva
Friendly ping! Thx. On Fri, Jun 22, 2018 at 6:25 AM Carl Eugen Hoyos wrote: > 2018-06-22 9:29 GMT+02:00, Wang Cao : > >> > >> My question was which authority defined this metadata for > >> y4m? Or which (non-FFmpeg-based) other software > >> understands the metadata your patch adds to the files.

Re: [FFmpeg-devel] [PATCH] libavformat/yuv4mpeg: Add color range support for Y4M Add color_range support in Y4M. Also set pixel format and color_range for YUVJ pixel formats.

2018-06-22 Thread Carl Eugen Hoyos
2018-06-22 9:29 GMT+02:00, Wang Cao : >> >> My question was which authority defined this metadata for >> y4m? Or which (non-FFmpeg-based) other software >> understands the metadata your patch adds to the files. >> > AFAIK, no other software and authority has defined this metadata > for color range.

Re: [FFmpeg-devel] [PATCH] libavformat/yuv4mpeg: Add color range support for Y4M Add color_range support in Y4M. Also set pixel format and color_range for YUVJ pixel formats.

2018-06-22 Thread Wang Cao
> > My question was which authority defined this metadata for > y4m? Or which (non-FFmpeg-based) other software > understands the metadata your patch adds to the files. > AFAIK, no other software and authority has defined this metadata for color range. According to https://linux.die.net/man/5/yuv4m

Re: [FFmpeg-devel] [PATCH] libavformat/yuv4mpeg: Add color range support for Y4M Add color_range support in Y4M. Also set pixel format and color_range for YUVJ pixel formats.

2018-06-21 Thread Carl Eugen Hoyos
2018-06-20 11:42 GMT+02:00, Wang Cao : > It's in the yuv4mpegenc.c. I added a support for color range > by specifying metadata in the Y4M header. Yes, I understand. My question was which authority defined this metadata for y4m? Or which (non-FFmpeg-based) other software understands the metadata y

Re: [FFmpeg-devel] [PATCH] libavformat/yuv4mpeg: Add color range support for Y4M Add color_range support in Y4M. Also set pixel format and color_range for YUVJ pixel formats.

2018-06-20 Thread Wang Cao
It's in the yuv4mpegenc.c. I added a support for color range by specifying metadata in the Y4M header. On Thu, Jun 14, 2018 at 12:19 AM Carl Eugen Hoyos wrote: > 2018-06-13 18:03 GMT+02:00, Wang Cao : > > > @@ -220,6 +221,12 @@ static int yuv4_read_header(AVFormatContext *s) > >

Re: [FFmpeg-devel] [PATCH] libavformat/yuv4mpeg: Add color range support for Y4M Add color_range support in Y4M. Also set pixel format and color_range for YUVJ pixel formats.

2018-06-13 Thread Carl Eugen Hoyos
2018-06-13 18:03 GMT+02:00, Wang Cao : > @@ -220,6 +221,12 @@ static int yuv4_read_header(AVFormatContext *s) > alt_pix_fmt = AV_PIX_FMT_YUV422P; > else if (strncmp("444", tokstart, 3) == 0) > alt_pix_fmt = AV_PIX_FMT_YUV444P; > +

[FFmpeg-devel] [PATCH] libavformat/yuv4mpeg: Add color range support for Y4M Add color_range support in Y4M. Also set pixel format and color_range for YUVJ pixel formats.

2018-06-13 Thread Wang Cao
--- libavformat/yuv4mpegdec.c | 8 libavformat/yuv4mpegenc.c | 37 +++-- 2 files changed, 43 insertions(+), 2 deletions(-) diff --git a/libavformat/yuv4mpegdec.c b/libavformat/yuv4mpegdec.c index eff7fc518e..86e8673b2f 100644 --- a/libavformat/yuv4mpegdec