Re: [FFmpeg-devel] [PATCH] lavc/hevc_cabac: fix cbf_cb and cbf_cr for transform depth 4

2019-12-17 Thread James Almer
On 12/11/2019 5:47 AM, Linjie Fu wrote: > The max transform depth is 5(from 0 to 4), so we need 5 cabac states for > cbf_cb and cbf_cr. > > See Table 9-4 for details. > > Signed-off-by: Xu Guangxin > Signed-off-by: Linjie Fu > --- > libavcodec/hevc_cabac.c | 42 +---

Re: [FFmpeg-devel] [PATCH] lavc/hevc_cabac: fix cbf_cb and cbf_cr for transform depth 4

2019-12-17 Thread Fu, Linjie
Hi, > -Original Message- > From: ffmpeg-devel On Behalf Of Fu, > Linjie > Sent: Sunday, December 15, 2019 09:56 > To: FFmpeg development discussions and patches de...@ffmpeg.org> > Subject: Re: [FFmpeg-devel] [PATCH] lavc/hevc_cabac: fix cbf_cb and cbf_cr &g

Re: [FFmpeg-devel] [PATCH] lavc/hevc_cabac: fix cbf_cb and cbf_cr for transform depth 4

2019-12-14 Thread Fu, Linjie
> -Original Message- > From: ffmpeg-devel On Behalf Of > Thilo Borgmann > Sent: Thursday, December 12, 2019 22:15 > To: ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpeg-devel] [PATCH] lavc/hevc_cabac: fix cbf_cb and cbf_cr > for transform depth 4 > > Hi, >

Re: [FFmpeg-devel] [PATCH] lavc/hevc_cabac: fix cbf_cb and cbf_cr for transform depth 4

2019-12-12 Thread Fu, Linjie
> -Original Message- > From: ffmpeg-devel On Behalf Of > Thilo Borgmann > Sent: Thursday, December 12, 2019 22:15 > To: ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpeg-devel] [PATCH] lavc/hevc_cabac: fix cbf_cb and cbf_cr > for transform depth 4 > > Hi, >

Re: [FFmpeg-devel] [PATCH] lavc/hevc_cabac: fix cbf_cb and cbf_cr for transform depth 4

2019-12-12 Thread Thilo Borgmann
; Subject: Re: [FFmpeg-devel] [PATCH] lavc/hevc_cabac: fix cbf_cb and cbf_cr >> for transform depth 4 >> >> Am Mi., 11. Dez. 2019 um 09:57 Uhr schrieb Linjie Fu : >>> >>> The max transform depth is 5(from 0 to 4), so we need 5 cabac states for >>> cbf_cb and

Re: [FFmpeg-devel] [PATCH] lavc/hevc_cabac: fix cbf_cb and cbf_cr for transform depth 4

2019-12-11 Thread Fu, Linjie
> -Original Message- > From: ffmpeg-devel On Behalf Of > Carl Eugen Hoyos > Sent: Wednesday, December 11, 2019 18:19 > To: FFmpeg development discussions and patches de...@ffmpeg.org> > Subject: Re: [FFmpeg-devel] [PATCH] lavc/hevc_cabac: fix cbf_cb and cbf_cr &g

Re: [FFmpeg-devel] [PATCH] lavc/hevc_cabac: fix cbf_cb and cbf_cr for transform depth 4

2019-12-11 Thread Carl Eugen Hoyos
Am Mi., 11. Dez. 2019 um 09:57 Uhr schrieb Linjie Fu : > > The max transform depth is 5(from 0 to 4), so we need 5 cabac states for > cbf_cb and cbf_cr. > > See Table 9-4 for details. Does this change the output of a file? If yes, please provide it / implement a fate test. Carl Eugen

[FFmpeg-devel] [PATCH] lavc/hevc_cabac: fix cbf_cb and cbf_cr for transform depth 4

2019-12-11 Thread Linjie Fu
The max transform depth is 5(from 0 to 4), so we need 5 cabac states for cbf_cb and cbf_cr. See Table 9-4 for details. Signed-off-by: Xu Guangxin Signed-off-by: Linjie Fu --- libavcodec/hevc_cabac.c | 42 +- 1 file changed, 21 insertions(+), 21 deletions