Re: [CM] tap fixed - patch to tankrev.scm

2016-10-27 Thread bil
I think the macro version is slower -- s7 does not currently expand macros when first encountered; you can use define-expansion for that, but the speedup is not great. Since all the smpls->samples calls involve constants, I'd precompute those offsets: (define* (tank-reverb (predelay 0.0) (decay

[CM] tap fixed - patch to tankrev.scm

2016-10-27 Thread anders . vinjar
Here's a patch to tankrev.scm. The negative delay-times in the previous version were temporary workarounds to not hit the (now fixed) bug in tap. Redefining smpls->samples as a macro here to avoid some unnecessary calculations in the loop. Btw, do we want this? Does s7 do checking and optimizin