[pulseaudio-discuss] [PATCH v3] pulse: Document client libraries logging behavior
Ahmed S. Darwish
darwish.07 at gmail.com
Fri Aug 14 06:39:51 PDT 2015
On Thu, Aug 13, 2015 at 02:43:02PM +0200, Tanu Kaskinen wrote:
> On Wed, 2015-08-12 at 22:33 +0200, Ahmed S. Darwish wrote:
> > Document how to modify the client libraries logging behvaior
> > using any of the PA-specific environment variables.
> > Using the `PULSE_LOG_*'environment variables makes debugging
> > and tracing PA applications quite easy, thus the need for an
> > official documentation text.
> > Signed-off-by: Ahmed S. Darwish <darwish.07 at gmail.com>
> > ---
> > src/pulse/pulseaudio.h | 34 ++++++++++++++++++++++++++++++++++
> > 1 file changed, 34 insertions(+)
> Thanks for the patch! I have a small complaint about one thing:
> > + * - `PULSE_LOG_NO_RATE_LIMIT`: If defined, do not rate limit the logging
> > + * output of any PulseAudio component or module. This is especially
> > + * true for the udev audio hardware detection module, where a potential
> > + * for excessive logging output exists.
> This documentation is aimed at client developers, so I think talking
> about module logging isn't really appropriate here.
Hmm, I agree.
> .. Also, I don't
> remember rate limiting having effect on the udev module logging in
> particular, even though it generates lots of output.
There is some rate-limiting I've seen in `module-udev-detect.c',
but indeed as you've said, this is mostly related to the daemon
rather than the client libraries - so let's strip out this part.
> I suggest the following text: "If defined, do not rate limit the
> logging output. Rate limiting skips certain log messages when their
> frequency is considered too high." How does that sound to you?
Sounds much better; will send a follow-up patch now :-)
More information about the pulseaudio-discuss