After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 763985 - audio: add some debug output about channels mapping
audio: add some debug output about channels mapping
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-base
git master
Other Linux
: Normal enhancement
: 1.9.1
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks: 763799
 
 
Reported: 2016-03-21 11:50 UTC by Guillaume Desmottes
Modified: 2016-04-12 18:50 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
alsa: coding style fix (912 bytes, patch)
2016-03-21 11:52 UTC, Guillaume Desmottes
committed Details | Review
alsa: factor out alsa_detect_channels_mapping() (3.83 KB, patch)
2016-03-21 11:52 UTC, Guillaume Desmottes
committed Details | Review
gst-audio: add gst_audio_channel_positions_to_string() (5.24 KB, patch)
2016-03-21 11:52 UTC, Guillaume Desmottes
committed Details | Review
alsa: add some debugging output to alsa_detect_channels_mapping() (2.09 KB, patch)
2016-03-21 11:52 UTC, Guillaume Desmottes
committed Details | Review
audio: add debug output if channels mapping does not match (1.24 KB, patch)
2016-03-21 11:52 UTC, Guillaume Desmottes
committed Details | Review

Description Guillaume Desmottes 2016-03-21 11:50:05 UTC
I'm part of my investigations on bug#763799 I'm adding some channels mapping related debug output as tracking down those issues isn't very convenient atm.
Comment 1 Guillaume Desmottes 2016-03-21 11:52:01 UTC
Created attachment 324421 [details] [review]
alsa: coding style fix

Was using tabs instead of spaces.
Comment 2 Guillaume Desmottes 2016-03-21 11:52:15 UTC
Created attachment 324422 [details] [review]
alsa: factor out alsa_detect_channels_mapping()

This code was duplicated in alsasrc and alsasink.
Comment 3 Guillaume Desmottes 2016-03-21 11:52:19 UTC
Created attachment 324423 [details] [review]
gst-audio: add gst_audio_channel_positions_to_string()

We currently don't log much about channel positions making debugging
harder as it should be. This is the first step in my attempt to improve
this.
Comment 4 Guillaume Desmottes 2016-03-21 11:52:26 UTC
Created attachment 324424 [details] [review]
alsa: add some debugging output to alsa_detect_channels_mapping()
Comment 5 Guillaume Desmottes 2016-03-21 11:52:31 UTC
Created attachment 324425 [details] [review]
audio: add debug output if channels mapping does not match
Comment 6 Nicolas Dufresne (ndufresne) 2016-04-12 18:49:08 UTC
Attachment 324421 [details] pushed as 592b87a - alsa: coding style fix
Attachment 324422 [details] pushed as d9268a5 - alsa: factor out alsa_detect_channels_mapping()
Attachment 324423 [details] pushed as 3cb0830 - gst-audio: add gst_audio_channel_positions_to_string()
Attachment 324424 [details] pushed as eef7312 - alsa: add some debugging output to alsa_detect_channels_mapping()
Attachment 324425 [details] pushed as 1c56cfa - audio: add debug output if channels mapping does not match
Comment 7 Nicolas Dufresne (ndufresne) 2016-04-12 18:50:39 UTC
Btw, I added missing Since 1.10 and win32 def. Thanks.