[Bug 777376] matrixmix: New element that mixes audio channels

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=777376

--- Comment #8 from Sebastian Dröge (slomo) <slomo at coaxion.net> ---
(In reply to Vivia Nikolaidou from comment #7)
> IMHO, when constructing the matrix, the user could make a mistake. Having
> input and output channels as an explicit property enforce that it isn't at
> least too small. Also, as you said, forgetting to set the matrix shouldn't
> make the element auto-negotiate one for you. I think the current way might
> be too verbose, but it's also the most intuitive one.

Ack. So for the auto mode, what's the use case of that? It seems like a very
specific thing, not necessarily what one would expect.

And what happens if mode=auto and a matrix is set? IMHO this auto mode is a bit
weird and better to be done at the application level.

> On the other hand, maybe the default should be 0 input channels, 0 output
> channels and a NULL matrix. If the element tries to go to READY like this,
> it prints out a big fat warning "PLEASE SET YOUR VARIABLES" and refuses to
> transition. With the current defaults, it just does stereo passthrough
> instead, but it also sets the channel-mask to 0, essentially acting as a
> capssetter.

Yes

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.