On Thu, 2015-03-26 at 21:54 +0100, Georg Chini wrote:
> +static const char *is_trigger_stream(struct userdata *u, pa_sink_input *i) {
I'd expect a function named "is_foo" to return a boolean value. It
looks confusing to me to have assignments like "role =
is_trigger_stream()". If the function returns some additional
information, I think it would be better to do that with a return
parameter. I can make a separate patch for this, though, so I'll push
this patch as is.
--
Tanu
