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 159433 - [osssink] doesn't check for supported channels
[osssink] doesn't check for supported channels
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins
0.8.5
Other Linux
: Normal major
: 0.8.7
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2004-11-25 12:11 UTC by Toni Willberg
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
gstosselement.c - adds functionality to check if device supports mono, stereo or both (2.43 KB, patch)
2004-12-13 14:36 UTC, Toni Willberg
none Details | Review
oss_probe.c - checks if device supports mono, stereo or both (2.20 KB, patch)
2004-12-13 14:38 UTC, Toni Willberg
none Details | Review
gstosselement.c - adds functionality to check if device supports mono, stereo or both (2.54 KB, patch)
2004-12-13 16:09 UTC, Toni Willberg
none Details | Review

Description Toni Willberg 2004-11-25 12:11:56 UTC
$gst-launch-0.8 sinesrc ! gsmenc ! udpsink

$gst-launch-0.8 udpsrc ! gsmdec ! audioconvert ! osssink
WARNING **: couldn't set the right number of channels (wanted 1, got 2), enjoy
the tone difference

Audio is played too slowly, half the frequency.

#gsteamer:
14:03 | <BBB> hm, does oss actually check the supported chans on startup?
Comment 1 Toni Willberg 2004-12-13 14:36:58 UTC
Created attachment 34797 [details] [review]
gstosselement.c - adds functionality to check if device supports mono, stereo or both

ChangeLog entry:

2004-12-13  Toni Willberg <toniw@iki.fi>

	* sys/oss/gstosselement.c: Added functionality to test if
	  the device supports mono, stereo or both. Fixes bug #159433
	* sys/oss/oss_probe.c: Added same mono/stereo test. Added missing
	  copyright and license header.
Comment 2 Toni Willberg 2004-12-13 14:38:00 UTC
Created attachment 34798 [details] [review]
oss_probe.c - checks if device supports mono, stereo or both
Comment 3 Ronald Bultje 2004-12-13 15:16:10 UTC
if both mono_supported and stereo_supported are FALSE, please be sure to error
out (or set channels to the 1,2 range). Don't set it to 0, we'll fail at random
places elsewhere and it'll be harder to track... Otherwise, good work.
Comment 4 Toni Willberg 2004-12-13 16:09:39 UTC
Created attachment 34799 [details] [review]
gstosselement.c - adds functionality to check if device supports mono, stereo or both

Added error better error handling; now sets channels [1,2] if the device
doesn't seem to support channels at all.
Comment 5 Ronald Bultje 2004-12-16 11:34:14 UTC
Both applied, thanks for the work!