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

[BugMaster]

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?