X-Git-Url: http://git.tuebingen.mpg.de/?p=paraslash.git;a=blobdiff_plain;f=wmadec_filter.c;h=aab7cf94f76adc0311aa6cd925da00ad5d8323a4;hp=4bbc24137e7e64f0af2246b515417cc3e6665c5c;hb=4d425382c007b200121b4be5120db1acd30d292f;hpb=05492521d51352fa8c4f55b71b33e14656f0bd0d

diff --git a/wmadec_filter.c b/wmadec_filter.c
index 4bbc2413..aab7cf94 100644
--- a/wmadec_filter.c
+++ b/wmadec_filter.c
@@ -1199,8 +1199,6 @@ static void wmadec_close(struct filter_node *fn)
 	if (!pwd)
 		return;
 	wmadec_cleanup(pwd);
-	free(fn->buf);
-	fn->buf = NULL;
 	free(fn->private_data);
 	fn->private_data = NULL;
 }
@@ -1210,35 +1208,8 @@ static int wmadec_execute(struct btr_node *btrn, const char *cmd, char **result)
 	struct filter_node *fn = btr_context(btrn);
 	struct private_wmadec_data *pwd = fn->private_data;
 
-	if (!strcmp(cmd, "samplerate")) {
-		if (pwd->ahi.sample_rate == 0)
-			return -ERRNO_TO_PARA_ERROR(ENAVAIL);
-		*result = make_message("%u", pwd->ahi.sample_rate);
-		return 1;
-	}
-	if (!strcmp(cmd, "channels")) {
-		if (pwd->ahi.channels == 0)
-			return -ERRNO_TO_PARA_ERROR(ENAVAIL);
-		*result = make_message("%u", pwd->ahi.channels);
-		return 1;
-	}
-	return -ERRNO_TO_PARA_ERROR(ENOTSUP);
-}
-
-#define WMADEC_MAX_PENDING (640 * 1024)
-
-static void wmadec_pre_select(struct sched *s, struct task *t)
-{
-	struct filter_node *fn = container_of(t, struct filter_node, task);
-	size_t iqs = btr_get_input_queue_size(fn->btrn);
-
-	t->error = 0;
-	if (iqs == 0)
-		return;
-	if (btr_bytes_pending(fn->btrn) > WMADEC_MAX_PENDING)
-		return; /* FIXME, should use reasonable bound on timeout */
-	s->timeout.tv_sec = 0;
-	s->timeout.tv_usec = 1;
+	return decoder_execute(cmd, pwd->ahi.sample_rate, pwd->ahi.channels,
+		result);
 }
 
 #define WMA_OUTPUT_BUFFER_SIZE (128 * 1024)
@@ -1246,44 +1217,43 @@ static void wmadec_pre_select(struct sched *s, struct task *t)
 static void wmadec_post_select(__a_unused struct sched *s, struct task *t)
 {
 	struct filter_node *fn = container_of(t, struct filter_node, task);
-	int ret, converted = 0;
+	int ret, converted;
 	struct private_wmadec_data *pwd = fn->private_data;
 	struct btr_node *btrn = fn->btrn;
-	size_t iqs, len, min_len;
+	size_t len;
 	char *in;
 
-	min_len = pwd? WMA_FRAME_SKIP + pwd->ahi.block_align : 8192;
 next_buffer:
+	converted = 0;
 	t->error = 0;
-	if (btr_bytes_pending(btrn) > WMADEC_MAX_PENDING)
+	ret = btr_node_status(btrn, fn->min_iqs, BTR_NT_INTERNAL);
+	if (ret < 0)
+		goto err;
+	if (ret == 0)
 		return;
-	iqs = btr_get_input_queue_size(btrn);
-	if (iqs < min_len) {
-		if (!btr_no_parent(btrn)) /* wait for more data */
-			return;
-		ret = -E_WMADEC_EOF;
+	btr_merge(btrn, fn->min_iqs);
+	len = btr_next_buffer(btrn, (char **)&in);
+	ret = -E_WMADEC_EOF;
+	if (len < fn->min_iqs)
 		goto err;
-	}
-	/* avoid "buffer too small" errors from the decoder */
-	btr_merge_to(btrn, min_len);
-	len = btr_next_buffer(btrn, &in);
-	assert(len >= min_len);
 	if (!pwd) {
 		ret = wma_decode_init(in, len, &pwd);
 		if (ret < 0)
 			goto err;
 		if (ret == 0) {
-			min_len += 4096;
+			fn->min_iqs += 4096;
 			goto next_buffer;
 		}
+		fn->min_iqs = 2 * (WMA_FRAME_SKIP + pwd->ahi.block_align);
 		fn->private_data = pwd;
 		converted = pwd->ahi.header_len;
 		goto success;
 	}
+	fn->min_iqs = WMA_FRAME_SKIP + pwd->ahi.block_align;
 	for (;;) {
 		char *out;
 		int out_size = WMA_OUTPUT_BUFFER_SIZE;
-		if (converted + WMA_FRAME_SKIP + pwd->ahi.block_align > len)
+		if (converted + fn->min_iqs > len)
 			break;
 		out = para_malloc(WMA_OUTPUT_BUFFER_SIZE);
 		ret = wma_decode_superframe(pwd, out,
@@ -1298,55 +1268,17 @@ next_buffer:
 	}
 success:
 	btr_consume(btrn, converted);
-	goto next_buffer;
+	return;
 err:
 	assert(ret < 0);
-	wmadec_close(fn);
 	t->error = ret;
-	btr_del_node(btrn);
-}
-
-static ssize_t wmadec_convert(char *inbuffer, size_t len,
-		struct filter_node *fn)
-{
-	int ret, converted = 0;
-	struct private_wmadec_data *pwd = fn->private_data;
-
-	if (len <= WMA_FRAME_SKIP)
-		return 0;
-	if (!pwd) {
-		ret = wma_decode_init(inbuffer, len, &pwd);
-		if (ret <= 0)
-			return ret;
-		fn->private_data = pwd;
-		fn->fc->channels = pwd->ahi.channels;
-		fn->fc->samplerate = pwd->ahi.sample_rate;
-		return pwd->ahi.header_len;
-	}
-	for (;;) {
-		int out_size;
-		if (converted + WMA_FRAME_SKIP + pwd->ahi.block_align > len)
-			break;
-		out_size = fn->bufsize - fn->loaded;
-		if (out_size < 128 * 1024)
-			break;
-		ret = wma_decode_superframe(pwd, fn->buf + fn->loaded,
-			&out_size, (uint8_t *)inbuffer + converted + WMA_FRAME_SKIP,
-			len - WMA_FRAME_SKIP);
-		if (ret < 0)
-			return ret;
-		fn->loaded += out_size;
-		converted += ret + WMA_FRAME_SKIP;
-	}
-	return converted;
+	btr_remove_node(btrn);
 }
 
 static void wmadec_open(struct filter_node *fn)
 {
-	fn->bufsize = 1024 * 1024;
-	fn->buf = para_malloc(fn->bufsize);
 	fn->private_data = NULL;
-	fn->loaded = 0;
+	fn->min_iqs = 4096;
 }
 
 /**
@@ -1358,8 +1290,7 @@ void wmadec_filter_init(struct filter *f)
 {
 	f->open = wmadec_open;
 	f->close = wmadec_close;
-	f->convert = wmadec_convert;
 	f->execute = wmadec_execute;
-	f->pre_select = wmadec_pre_select;
+	f->pre_select = generic_filter_pre_select;
 	f->post_select = wmadec_post_select;
 }