[vlc-devel] [PATCH 1/3] vlc_common: add vlc_align helper

[Marvin Scholz]

On 12 Feb 2020, at 17:04, zhilizhao wrote:

>> On Feb 12, 2020, at 9:50 PM, Marvin Scholz <epirat07 at gmail.com> 
>> wrote:
>>
>> ---
>> include/vlc_common.h | 11 +++++++++++
>> 1 file changed, 11 insertions(+)
>>
>> diff --git a/include/vlc_common.h b/include/vlc_common.h
>> index e4c5cd85c9..9d9d0375c9 100644
>> --- a/include/vlc_common.h
>> +++ b/include/vlc_common.h
>> @@ -548,6 +548,17 @@ typedef int ( * vlc_list_callback_t ) ( 
>> vlc_object_t *,      /* variable's objec
>> /* clip v in [min, max] */
>> #define VLC_CLIP(v, min, max)    __MIN(__MAX((v), (min)), (max))
>>
>> +/**
>> + * Make integer v a multiple of align
>> + *
>> + * \note align must be a power of 2
>> + */
>> +VLC_USED
>> +static inline size_t vlc_align(size_t v, size_t align)
>> +{
>> +    return ((v) + ((align) - 1)) & ~((align) - 1);
>> +}
>
> Why not int or fixed width integer? Is there known usecase for size_t?
>

I chose it as it is mostly used for sizes so size_t seemed like a good 
fit
but I am fine with other suggestions.

>> +
>> /** Greatest common divisor */
>> VLC_USED
>> static inline int64_t GCD ( int64_t a, int64_t b )
>> -- 
>> 2.24.1 (Apple Git-126)
>>
>> _______________________________________________
>> vlc-devel mailing list
>> To unsubscribe or modify your subscription options:
>> https://mailman.videolan.org/listinfo/vlc-devel
>
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel