Message ID | 20170819033151.149562-1-jzern@google.com |
---|---|
State | Accepted |
Commit | d8269519e4a338fa9521612db9b68c04a0fb2deb |
Headers | show |
On Fri, Aug 18, 2017 at 8:31 PM, James Zern <jzern@google.com> wrote: > this is undocumented in the vp9 bitstream and decoding specification > doc, but matches libvpx > this came up in comparing decode results of ffvp9 and libvpx with source from a hardware encoder. > Signed-off-by: James Zern <jzern@google.com> > --- > libavcodec/vp9.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/vp9.c b/libavcodec/vp9.c > index 7d8aced8c8..94430db9a3 100644 > --- a/libavcodec/vp9.c > +++ b/libavcodec/vp9.c > @@ -459,7 +459,7 @@ static int decode_frame_header(AVCodecContext *avctx, > s->bytesperpixel = 1; > s->pix_fmt = AV_PIX_FMT_YUV420P; > avctx->colorspace = AVCOL_SPC_BT470BG; > - avctx->color_range = AVCOL_RANGE_JPEG; > + avctx->color_range = AVCOL_RANGE_MPEG; > } > s->s.h.refreshrefmask = get_bits(&s->gb, 8); > w = get_bits(&s->gb, 16) + 1; > -- > 2.14.1.480.gb18f417b89-goog >
Hi, On Fri, Aug 18, 2017 at 11:31 PM, James Zern <jzern@google.com> wrote: > this is undocumented in the vp9 bitstream and decoding specification > doc, but matches libvpx > > Signed-off-by: James Zern <jzern@google.com> > --- > libavcodec/vp9.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/vp9.c b/libavcodec/vp9.c > index 7d8aced8c8..94430db9a3 100644 > --- a/libavcodec/vp9.c > +++ b/libavcodec/vp9.c > @@ -459,7 +459,7 @@ static int decode_frame_header(AVCodecContext *avctx, > s->bytesperpixel = 1; > s->pix_fmt = AV_PIX_FMT_YUV420P; > avctx->colorspace = AVCOL_SPC_BT470BG; > - avctx->color_range = AVCOL_RANGE_JPEG; > + avctx->color_range = AVCOL_RANGE_MPEG; > OK. Ronald
On Sat, Aug 19, 2017 at 8:04 AM, Ronald S. Bultje <rsbultje@gmail.com> wrote: > Hi, > > On Fri, Aug 18, 2017 at 11:31 PM, James Zern <jzern@google.com> wrote: >> >> this is undocumented in the vp9 bitstream and decoding specification >> doc, but matches libvpx >> >> Signed-off-by: James Zern <jzern@google.com> >> --- >> libavcodec/vp9.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/libavcodec/vp9.c b/libavcodec/vp9.c >> index 7d8aced8c8..94430db9a3 100644 >> --- a/libavcodec/vp9.c >> +++ b/libavcodec/vp9.c >> @@ -459,7 +459,7 @@ static int decode_frame_header(AVCodecContext *avctx, >> s->bytesperpixel = 1; >> s->pix_fmt = AV_PIX_FMT_YUV420P; >> avctx->colorspace = AVCOL_SPC_BT470BG; >> - avctx->color_range = AVCOL_RANGE_JPEG; >> + avctx->color_range = AVCOL_RANGE_MPEG; > > > OK. > applied. thanks.
diff --git a/libavcodec/vp9.c b/libavcodec/vp9.c index 7d8aced8c8..94430db9a3 100644 --- a/libavcodec/vp9.c +++ b/libavcodec/vp9.c @@ -459,7 +459,7 @@ static int decode_frame_header(AVCodecContext *avctx, s->bytesperpixel = 1; s->pix_fmt = AV_PIX_FMT_YUV420P; avctx->colorspace = AVCOL_SPC_BT470BG; - avctx->color_range = AVCOL_RANGE_JPEG; + avctx->color_range = AVCOL_RANGE_MPEG; } s->s.h.refreshrefmask = get_bits(&s->gb, 8); w = get_bits(&s->gb, 16) + 1;
this is undocumented in the vp9 bitstream and decoding specification doc, but matches libvpx Signed-off-by: James Zern <jzern@google.com> --- libavcodec/vp9.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)