On 6 November 2014 21:07, Peter Meerwald <[email protected]> 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 _______________________________________________ pulseaudio-discuss mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss
