You've already forked ivan-torvalds-GuitarPedal
forked from AllSpiceMirrors/torvalds-GuitarPedal
3be9e6a482
Back when mixing was each effect's own business, they all did a plain linear blend except for the vibrato, which did an equal-power one with a sin/cos pair. When the mix moved into the effect chain that distinction got flattened, and the vibrato quietly lost it. It shouldn't have. The two laws differ in what they assume about whether the wet and dry are correlated, and each is wrong by 3dB in the other's case. When the wet is a filtered version of the dry - which is most of what's here - the amplitudes add, half of each gives you back the original, and an equal-power mix puts a 3dB hump right in the middle of the sweep where everyone leaves the knob. When they're uncorrelated the powers add instead, and it's the linear mix that leaves a 3dB hole there. So it isn't one law for everything, it is a property of the effect, and the effect header is where it should say so. Add a 'MIX:' annotation alongside PRIORITY and DEFAULT_MIX, defaulting to LINEAR, and mark the four whose wet really is decorrelated from the dry: vibrato, echo, reverb and the pitch shifter. Phaser and flanger stay linear on purpose, even though it's tempting. An allpass or comb has the same magnitude spectrum as the input and only differs in phase, and the cancellation is the whole point - equal power would fill the notches back in by 3dB and make them sound weak. Nothing uses the field yet; do_effect_step() still does what it did. Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>