Fix a bug in the output of para_audioc.
[paraslash.git] / filter_common.c
index d83629b..1b02e1c 100644 (file)
@@ -23,17 +23,14 @@ struct filter filters[NUM_SUPPORTED_FILTERS] = {FILTER_ARRAY};
 
 /**
  * Call the init function of each supported filter.
- *
- * \param all_filters the array of all supported filters.
- *
  * \sa filter::init
  */
-void filter_init(struct filter *all_filters)
+void filter_init(void)
 {
        int i;
 
        FOR_EACH_SUPPORTED_FILTER(i)
-               all_filters[i].init(all_filters + i);
+               filters[i].init(filters + i);
 }
 
 /**
@@ -102,7 +99,7 @@ static void call_callbacks(struct filter_node *fn, char *inbuf, size_t inlen,
  * there is input available for the filter node in question. If the convert
  * function consumed some or all of its input data, all registered input
  * callbacks are called.  Similarly, if a convert function produced output, all
- * registerd output callbacks get called.
+ * registered output callbacks get called.
  *
  * On errors a (negative) error code is stored in t->error.
  *
@@ -121,15 +118,13 @@ void filter_pre_select(__a_unused struct sched *s, struct task *t)
                return;
        }
 again:
-       ib = fc->inbuf;
+       ib = *fc->inbufp;
        loaded = fc->in_loaded;
        conv = 0;
        FOR_EACH_FILTER_NODE(fn, fc, i) {
                struct filter *f = filters + fn->filter_num;
                if (fn->loaded < fn->bufsize) {
                        size_t size, old_fn_loaded = fn->loaded;
-//                     PARA_DEBUG_LOG("fc %p loaded: %zd, calling %s convert\n",
-//                             fc, *loaded, fn->filter->name);
                        t->error = f->convert(ib, *loaded, fn);
                        if (t->error < 0)
                                return;
@@ -137,24 +132,29 @@ again:
                        call_callbacks(fn, ib, size, fn->buf + old_fn_loaded,
                                fn->loaded - old_fn_loaded);
                        *loaded -= size;
-                       conv += size;
-                       if (*loaded && size) {
-//                             PARA_DEBUG_LOG("moving %zd bytes in input "
-//                                     "buffer for %s filter\n",
-//                                     *loaded,  fn->filter->name);
+                       conv += size + fn->loaded - old_fn_loaded;
+                       if (*loaded && size)
                                memmove(ib, ib + size, *loaded);
-                       }
                }
                ib = fn->buf;
                loaded = &fn->loaded;
        }
        conv_total += conv;
-//     PARA_DEBUG_LOG("eof (in/out/fc): %d/%d/%d out_loaded: %zd, "
-//             "conv: %d, conv_total: %d\n", *fc->input_eof,
-//             fc->output_eof? *fc->output_eof : -42,
-//             fc->eof, *fc->out_loaded, conv, conv_total);
        if (conv)
                goto again;
+       if (conv_total) {
+               /*
+                * Other pre_select functions might have already been called by
+                * now and decided to do nothing, e.g. because their output
+                * buffer was full or the input buffer was empty. We just
+                * converted something which caused these buffers to change but
+                * we can't make the other tasks reconsider their decision at
+                * this point.  So force a minimal timeout for the next select
+                * call to avoid unnecessary delays.
+                */
+               s->timeout.tv_sec = 0;
+               s->timeout.tv_usec = 1;
+       }
        if (*fc->input_error >= 0)
                return;
        if (*fc->out_loaded)