[x264-devel] [PATCH] fix mb_type statistic bug in X264

[royzzduan]

On Tue, 9 Feb 2021 01:05:10 +0300
BugMaster <BugMaster at narod.ru> wrote:

> On Tue, 9 Feb 2021 00:56:56 +0300, BugMaster wrote:
> > On Sun, 7 Feb 2021 09:35:26 +0800, Lingjiang Fang wrote:  
> >> On Thu,  4 Feb 2021 12:26:17 +0800
> >> zhengzhi Duan <royzzduan at foxmail.com> wrote:  
> 
> >>> From: royduan <royduan at tencent.com>
> >>> 
> >>> ---
> >>>  encoder/ratecontrol.c | 2 +-
> >>>  1 file changed, 1 insertion(+), 1 deletion(-)
> >>> 
> >>> diff --git a/encoder/ratecontrol.c b/encoder/ratecontrol.c
> >>> index e52270ba..e667a24e 100644
> >>> --- a/encoder/ratecontrol.c
> >>> +++ b/encoder/ratecontrol.c
> >>> @@ -1831,7 +1831,7 @@ int x264_ratecontrol_end( x264_t *h, int
> >>> bits, int *filler ) h->stat.frame.i_mb_count_skip = mbs[P_SKIP] +
> >>> mbs[B_SKIP]; h->stat.frame.i_mb_count_i = mbs[I_16x16] +
> >>> mbs[I_8x8] + mbs[I_4x4]; h->stat.frame.i_mb_count_p = mbs[P_L0] +
> >>> mbs[P_8x8];
> >>> -    for( int i = B_DIRECT; i < B_8x8; i++ )
> >>> +    for( int i = B_DIRECT; i <= B_8x8; i++ )  
> 
> >> I helped him to send this patch for review, sorry if there is
> >> something wrong.
> >> Hope someone can give us a response :)  
> 
> > Hi.  
> 
> > I added more MB stats fixes and created MR:
> > https://code.videolan.org/videolan/x264/-/merge_requests/55
> > Commit:
> > https://code.videolan.org/BugMaster/x264/-/commit/50a8f3b34d6a33ce8fe553c36e5d5bf568df00f6
> >  
> 
> Btw. Check that I have correctly written the name in the commit
> message. Or do I need to write the first word with a lowercase?
It's OK. thanks

> 
> _______________________________________________
> x264-devel mailing list
> x264-devel at videolan.org
> https://mailman.videolan.org/listinfo/x264-devel