sys/oss/gstosssrc.c: Set bytes_per_sample correctly (is not always 4, but depends...

Original commit message from CVS:
* sys/oss/gstosssrc.c: (gst_oss_src_prepare):
Set bytes_per_sample correctly (is not always 4, but
depends on width and number of channels).

diff --git a/ChangeLog b/ChangeLog
index 5f35c01..0d60da6 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,11 @@
 2005-10-24  Tim-Philipp Müller  <tim at centricular dot net>
 
+       * sys/oss/gstosssrc.c: (gst_oss_src_prepare):
+         Set bytes_per_sample correctly (is not always 4, but 
+         depends on width and number of channels).
+
+2005-10-24  Tim-Philipp Müller  <tim at centricular dot net>
+
        * ext/flac/gstflacenc.c: (gst_flacenc_base_init),
        (gst_flacenc_init), (gst_flacenc_sink_setcaps),
        (gst_flacenc_seek_callback), (gst_flacenc_write_callback),

diff --git a/sys/oss/gstosssrc.c b/sys/oss/gstosssrc.c
index 6c2f700..bb5d7ca 100644 (file)
--- a/sys/oss/gstosssrc.c
+++ b/sys/oss/gstosssrc.c
@@ -389,8 +389,12 @@ gst_oss_src_prepare (GstAudioSrc * asrc, GstRingBufferSpec * spec)
 
   spec->segsize = info.fragsize;
   spec->segtotal = info.fragstotal;
-  spec->bytes_per_sample = 4;
-  oss->bytes_per_sample = 4;
+
+  if (spec->width != 16 && spec->width != 8)
+    goto dodgy_width;
+
+  spec->bytes_per_sample = (spec->width / 8) * spec->channels;
+  oss->bytes_per_sample = (spec->width / 8) * spec->channels;
   memset (spec->silence_sample, 0, spec->bytes_per_sample);
 
   GST_DEBUG ("got segsize: %d, segtotal: %d, value: %08x", spec->segsize,
@@ -411,6 +415,12 @@ wrong_format:
         ("Unable to get format %d", spec->format), (NULL));
     return FALSE;
   }
+dodgy_width:
+  {
+    GST_ELEMENT_ERROR (oss, RESOURCE, OPEN_READ,
+        ("Unexpected width %d", spec->width), (NULL));
+    return FALSE;
+  }
 }
 
 static gboolean