[FFmpeg-devel] [PATCH] lavu/cpu: Use av_cpu_ prefix

[Andreas Rheinhardt]

Thilo Borgmann:
> Am 20.07.21 um 00:27 schrieb Andreas Rheinhardt:
>> Thilo Borgmann:
>>> Am 19.07.21 um 23:51 schrieb James Almer:
>>>> On 7/19/2021 6:47 PM, Thilo Borgmann wrote:
>>>>> Am 19.07.21 um 23:20 schrieb James Almer:
>>>>>> On 7/19/2021 6:19 PM, Andreas Rheinhardt wrote:
>>>>>>> Thilo Borgmann:
>>>>>>>> Hi,
>>>>>>>>
>>>>>>>> $subject for the new force cpu count function.
>>>>>>>>
>>>>>>>> Thanks,
>>>>>>>> Thilo
>>>>>>>>
>>>>>>>
>>>>>>> I just wanted to complain that you did not modify APIchanges as well;
>>>>>>> but then I noticed that there is no entry there for this new function.
>>>>>>> But there should be.
>>>>>>
>>>>>> No version bump either, so this patch should definitely do both things.
>>>>>
>>>>> Ok, does the attached look better?
>>>>
>>>> You need to bump the library version when you add new API. Since you didn't when you first added the function, you can do it now.
>>>> And of course, the version you report in doc/APIchanges should be the one that introduced the new API.
>>>
>>> Hopefully compliant patch attached.
>>
>>
>>
>>> -void av_force_cpu_count(int count){
>>> +void av_cpu_force_count(int count){
>>>      atomic_store_explicit(&cpu_count, count, memory_order_relaxed);
>>>  }
>>
>> While just at it, you can put the '{' on a line of its own as we do for
>> opening brackets of functions.
> 
> Attached.
> 
No objection from me for this version.

- Andreas