On Mon, 2016-02-22 at 17:19 +0900, Jungsup Lee wrote: > Simplify code by using PA_IDXSET_FOREACH. > > Signed-off-by: Jungsup Lee <[email protected]> > --- > src/modules/module-role-cork.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/src/modules/module-role-cork.c b/src/modules/module-role-cork.c > index 7c0f59f..4ae2005 100644 > --- a/src/modules/module-role-cork.c > +++ b/src/modules/module-role-cork.c > @@ -70,7 +70,7 @@ static bool shall_cork(struct userdata *u, pa_sink *s, > pa_sink_input *ignore) { > pa_assert(u); > pa_sink_assert_ref(s); > > - for (j = PA_SINK_INPUT(pa_idxset_first(s->inputs, &idx)); j; j = > PA_SINK_INPUT(pa_idxset_next(s->inputs, &idx))) { > + PA_IDXSET_FOREACH(j, s->inputs, idx) { > const char *role; > > if (j == ignore) > @@ -99,7 +99,7 @@ static inline void apply_cork_to_sink(struct userdata *u, > pa_sink *s, pa_sink_in > pa_assert(u); > pa_sink_assert_ref(s); > > - for (j = PA_SINK_INPUT(pa_idxset_first(s->inputs, &idx)); j; j = > PA_SINK_INPUT(pa_idxset_next(s->inputs, &idx))) { > + PA_IDXSET_FOREACH(j, s->inputs, idx) { > bool corked, corked_here; > const char *role;
This doesn't apply on master any more, but a similar patch for src/modules/stream-interaction.c would be welcome. -- Tanu _______________________________________________ pulseaudio-discuss mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss
