[pulseaudio-discuss] [PATCH] switch-on-connect: Don't move a stream already moving

[Dalleau, Frederic]

Hi Tanu,

> Can you explain how o->source can be NULL? It can't be just because it's
> "already in the process of being moved". The only places that I found
> where pa_source_output.source is set to NULL are
> pa_source_output_start_move() and pa_source_output_unlink(). Of those,
> pa_source_output_start_move() is the interesting one, but prior to
> setting o->source to NULL, the source output is removed from the
> source's list of outputs, so PA_IDXSET_FOREACH(o, def->outputs, idx)
> shouldn't touch any outputs that are being moved.

This comment makes sense. I'll reproduce the issue to clarify what's missing,
I come back to you ASAP.

Frédéric