[FFmpeg-devel] [PATCH v9 03/13] vvcdec: add cabac decoder

[Lynne]

Jan 2, 2024, 14:22 by nuomi2021 at gmail.com:

> On Mon, Jan 1, 2024 at 11:13 PM Lynne <dev at lynne.ee> wrote:
>
>>
>>
>>
>> Jan 1, 2024, 15:14 by nuomi2021 at gmail.com:
>>
>> > +
>> > +//fixme
>> > +static void vvc_refill2(CABACContext* c) {
>> >
>>
>> Fix what?
>> Also, wrong coding style.
>>
> The function is a copy of refill2 in cabac_functions.h, including the style.
> The refill2 is protected by get_cabac_inline.
> On x86, get_cabac_inline is only defined when HAVE_7REGS &&
> !BROKEN_COMPILER is true.
> It can be challenging to figure out how to get refill2 defined in this
> context.
> Let us use https://github.com/ffvvc/FFmpeg/issues/178 to track this
>

Are you thinking about sharing the code later on?
If not, then there's no reason to have the duplicated function
and style issues.