[gstreamer-bugs] [Bug 317038] audioconvert advertises handling 8 channels but handles only two

[GStreamer (bugzilla.gnome.org)]

Do not reply to this via email (we are currently unable to handle email
responses and they get discarded).  You can add comments to this bug at
http://bugzilla.gnome.org/show_bug.cgi?id=317038
 GStreamer | gst-plugins-base | Ver: HEAD CVS





------- Comment #4 from Tim-Philipp Müller  2006-02-14 17:52 UTC -------
Looking at this again, there are two questions:

(a)
Is this really a bug or just inconvenience that's worth paying to uncover
potential bugs more easily? I don't think audioconvert is actually buggy here.
So far the rule has been that if we have more than 2 channels, the producer or
recipient must specify a channel layout in the caps. The warning from
audioconvert simply notifies us of the fact that there are elements involved
which don't adhere to that requirement, in this case capsfilter/fakesink. If we
apply something like that patch above, a default layout will be used if none is
provided explicitly. The question is if we really want that, because then we
don't get warnings any longer when there are decoders or audiosinks or encoders
involved that don't specify a channel layout. Users might just find that things
'sound weird', which isn't very helpful when trying to locate the problem
later.


(b) if we do want to use default layouts like in the patch - what should the
default layouts be? The layouts in the patch don't exactly match the ones used
by alsasink it seems (I can't remember the source of the above layouts though).
Do we just want to use the layouts used by alsa?


-- 
Configure bugmail: http://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.