gst/filter/: Improve debugging a bit.
authorSebastian Dröge <slomo@circular-chaos.org>
Sun, 12 Aug 2007 15:41:57 +0000 (15:41 +0000)
committerSebastian Dröge <slomo@circular-chaos.org>
Sun, 12 Aug 2007 15:41:57 +0000 (15:41 +0000)
Original commit message from CVS:
* gst/filter/gstbpwsinc.c: (bpwsinc_build_kernel):
* gst/filter/gstlpwsinc.c: (lpwsinc_build_kernel):
Improve debugging a bit.

gst/audiofx/audiowsincband.c
gst/audiofx/audiowsinclimit.c

index f2bfda7..d9690b6 100644 (file)
@@ -338,8 +338,6 @@ bpwsinc_build_kernel (GstBPWSinc * self)
     return;
   }
 
-  self->have_kernel = TRUE;
-
   /* Clamp frequencies */
   self->lower_frequency =
       CLAMP (self->lower_frequency, 0.0,
@@ -354,10 +352,13 @@ bpwsinc_build_kernel (GstBPWSinc * self)
     self->upper_frequency = tmp;
   }
 
-  /* fill the lp kernel */
-  GST_DEBUG ("bpwsinc: initializing LP kernel of length %d with cut-off %f",
-      len, self->lower_frequency);
+  GST_DEBUG ("bpwsinc: initializing filter kernel of length %d "
+      "with lower frequency %.2lf Hz "
+      ", upper frequency %.2lf Hz for mode %s",
+      len, self->lower_frequency, self->upper_frequency,
+      (self->mode == MODE_BAND_PASS) ? "band-pass" : "band-reject");
 
+  /* fill the lp kernel */
   w = 2 * M_PI * (self->lower_frequency / GST_AUDIO_FILTER (self)->format.rate);
   kernel_lp = g_new (gdouble, len);
   for (i = 0; i < len; ++i) {
@@ -383,9 +384,6 @@ bpwsinc_build_kernel (GstBPWSinc * self)
     kernel_lp[i] /= sum;
 
   /* fill the hp kernel */
-  GST_DEBUG ("bpwsinc: initializing HP kernel of length %d with cut-off %f",
-      len, self->upper_frequency);
-
   w = 2 * M_PI * (self->upper_frequency / GST_AUDIO_FILTER (self)->format.rate);
   kernel_hp = g_new (gdouble, len);
   for (i = 0; i < len; ++i) {
@@ -438,9 +436,10 @@ bpwsinc_build_kernel (GstBPWSinc * self)
   /* set up the residue memory space */
   if (self->residue)
     g_free (self->residue);
-
   self->residue =
       g_new0 (gdouble, len * GST_AUDIO_FILTER (self)->format.channels);
+
+  self->have_kernel = TRUE;
 }
 
 /* GstAudioFilter vmethod implementations */
index 55daaf7..50d0c0f 100644 (file)
@@ -319,9 +319,7 @@ lpwsinc_build_kernel (GstLPWSinc * self)
   gint len = 0;
   gdouble w;
 
-  /* fill the kernel */
   len = self->kernel_length;
-  GST_DEBUG ("lpwsinc: initializing filter kernel of length %d", len);
 
   if (GST_AUDIO_FILTER (self)->format.rate == 0) {
     GST_DEBUG ("rate not set yet");
@@ -337,6 +335,13 @@ lpwsinc_build_kernel (GstLPWSinc * self)
   self->frequency =
       CLAMP (self->frequency, 0.0, GST_AUDIO_FILTER (self)->format.rate / 2);
 
+  GST_DEBUG ("lpwsinc: initializing filter kernel of length %d "
+      "with cutoff %.2lf Hz "
+      "for mode %s",
+      len, self->frequency,
+      (self->mode == MODE_LOW_PASS) ? "low-pass" : "high-pass");
+
+  /* fill the kernel */
   w = 2 * M_PI * (self->frequency / GST_AUDIO_FILTER (self)->format.rate);
 
   if (self->kernel)