Wrong initialization of free-field/diffuse-field EQ

AustrianAudioGmbH / PolarDesigner

Plug-in allowing you to control the polar pattern of your OC818 microphone in up to five frequency bands. Developed by Simon, Thomas, IEM, and AA.

GNU General Public License v3.0

34 stars 5 forks source link

Wrong initialization of free-field/diffuse-field EQ #16

Open oholiba opened 1 year ago

oholiba commented 1 year ago

In PluginProcessor.cpp, in lines 289, 293, and 297 different impulse responses are loaded into the same convolution unit "dfEqOmniConv". To fix this bug, the units "dfEqEightConv" (line 289), "ffEqOmniConv" (line 293) and "ffEqEightConv" (line 297) should be loaded with their appropriate impulse responses.

The bug was confirmed by testing the plugin in Reaper: the free field EQ has no effect on the processed audio.

This bug was introduced in the update to JUCE 6 (commit 48ca6cd).

seclorum commented 1 year ago

Good catch, it would definitely be a necessary fix for next release ..