[pulseaudio-discuss] [PATCH] add log category

Arun Raghavan arun.raghavan at collabora.co.uk
Mon May 28 20:29:23 PDT 2012


On Thu, 2012-05-24 at 21:53 +0800, rong deng wrote:
> Hi all,
> 
> 
> This is more like a RFC patch, it can't even compile well... However,
> I'd like to gather some information here on the mailing list to make
> sure that I'm on the right track and my work would be liked by others.
> 
> 
> This 'category' model is mainly after the design of gstreamer. So we
> have a 'pa_log_category_t' type to show what kind of category it's,
> and in its category, we can specify a threshold and the color to
> display.  (look at the changes in these two files src/pulsecore/log.h
> and src/pulsecore/log.c for more details.)
> 
> 
> Now the main problem to me is that the usage is not so
> developer-friendly. Any place where we use pa_log related functions, I
> have to specify the macro PA_LOG_CATEGORY_DEFAULT. That's almost
> everywhere... But yes, it should only be written for only and after
> that other developers should be OK with that.

We spoke of this on IRC -- this looks reasonable to me, and inevitable
if we want to group output logically together.

I haven't checked every category you've added, but they seem to be
broadly okay and the principle is sound.

-- Arun




More information about the pulseaudio-discuss mailing list