[pulseaudio-discuss] [PATCH RFCv3 43/51] core: Add volume-util.h
Arun Raghavan
arun at accosted.net
Thu Nov 6 07:48:18 PST 2014
On 6 November 2014 21:07, Peter Meerwald <pmeerw at pmeerw.net> wrote:
>
>> > implement inlineable functions PA_CVOLUME_VALID(),
>> > PA_CVOLUME_CHANNELS_EQUALS_TO(), PA_CVOLUME_IS_MUTED(),
>> > PA_CVOLUME_IS_NORM() that assume data is valid
>>
>> Why are these uppercase? AFAIK, we don't usually uppercase inline functions.
>
> I've modelled after e.g. PA_ALIGN_PTR() in pulsecore/macro.h or
> PA_SOURCE_IS_LINKED() in source.h
>
> I think there is no clear rule; I'd prefer PA_CVOLUME_IS_NORM() to
> pa_cvolume_is_norm_internal() or pa_cvolume_is_norm_unchecked()
>
> thanks, p.
My preference is lower case for functions (I suspect the exceptions
began as macros).
-- Arun
More information about the pulseaudio-discuss
mailing list