Hi Oleg,
Thanks!
I made another PR:
https://github.com/grame-cncm/faustlibraries/pull/57
How stupid of me, I only tested with n=2.
As for the naming, reduce is already taken:
https://github.com/grame-cncm/faustlibraries/blob/2efc2d01cb6be4e7eb2ae54fe3f3591782786ed0/reducemaps.lib#L74
Not sure what a better name would be.
Cheers,
Bart.
Oleg Nesterov <o...@redhat.com> writes:
Hi Bart,
The recent commit 7ee9bcf4459ac833bc adds parallelOp() which
looks
obviously wrong:
parallelOp(op,1) = _;
parallelOp(op,2) = op;
parallelOp(op,n) = (parallelOp(n-1),_):op;
the last line should be
parallelOp(op,n) = (parallelOp(op,n-1),_):op;
although I'd suggest
parallelOp(op,n) = op(parallelOp(op,n-1));
and... perhaps it should be named reduce() ?
Oleg.
_______________________________________________
Faudiostream-users mailing list
Faudiostream-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/faudiostream-users
