]> code.delx.au - pulseaudio/commitdiff
Revert "ladspa-sink: Handle empty chunks in sink_input_pop_cb"
authorTanu Kaskinen <tanu.kaskinen@linux.intel.com>
Wed, 15 Jan 2014 12:24:25 +0000 (14:24 +0200)
committerTanu Kaskinen <tanu.kaskinen@linux.intel.com>
Wed, 15 Jan 2014 12:24:25 +0000 (14:24 +0200)
This reverts commit dbe66b0b5e6b6e24df45bda8bf656836f18e7f4b.

I accidentally pushed this patch that was superseded by
e96785c1c34706e902f28c25844f31d8201db2b2.

src/modules/module-ladspa-sink.c

index 46ce9e440e29e562e9649dfe54ab1adfd983f688..cbb1aea69c42f3a297207178a147cf128511bcc8 100644 (file)
@@ -474,29 +474,22 @@ static int sink_input_pop_cb(pa_sink_input *i, size_t nbytes, pa_memchunk *chunk
 
     pa_memblockq_drop(u->memblockq, chunk->length);
 
-    src = (tchunk.memblock ? pa_memblock_acquire_chunk(&tchunk) : NULL);
+    src = pa_memblock_acquire_chunk(&tchunk);
     dst = pa_memblock_acquire(chunk->memblock);
 
     for (h = 0; h < (u->channels / u->max_ladspaport_count); h++) {
-        if (src) {
-            for (c = 0; c < u->input_count; c++)
-                pa_sample_clamp(PA_SAMPLE_FLOAT32NE, u->input[c], sizeof(float), src+ h*u->max_ladspaport_count + c, u->channels*sizeof(float), n);
-        } else {
-            for (c = 0; c < u->input_count; c++)
-                memset(u->input[c], 0, (n * sizeof(float)));
-        }
+        for (c = 0; c < u->input_count; c++)
+            pa_sample_clamp(PA_SAMPLE_FLOAT32NE, u->input[c], sizeof(float), src+ h*u->max_ladspaport_count + c, u->channels*sizeof(float), n);
         u->descriptor->run(u->handle[h], n);
         for (c = 0; c < u->output_count; c++)
             pa_sample_clamp(PA_SAMPLE_FLOAT32NE, dst + h*u->max_ladspaport_count + c, u->channels*sizeof(float), u->output[c], sizeof(float), n);
     }
 
-    if (tchunk.memblock) {
-        pa_memblock_release(tchunk.memblock);
-        pa_memblock_unref(tchunk.memblock);
-    }
-
+    pa_memblock_release(tchunk.memblock);
     pa_memblock_release(chunk->memblock);
 
+    pa_memblock_unref(tchunk.memblock);
+
     return 0;
 }