FFTW: No matching constructor for initialization of FFTWImpl

Shouldn’t this line be something like this:
return new FFTWImpl (order, std::move (lib), symbols);

The constructor of FFTWImpl is
FFTWImpl (size_t orderToUse, DynamicLibrary&& libraryToUse, **const** Symbols& symbols)

Otherwise it won’t compile if you want to use the FFTW

How did this get into the master branch? :smiley:

Happened here: https://github.com/WeAreROLI/JUCE/commit/eaf0f990d48ff448b2e40b8a76cf02630108523f#diff-b7ab5abe537279438f81657dffdeeb43


Whoops. I’ll get that sorted out, thanks.

Thanks! Any chances to cherrypick this to the master branch? Our CI will most definitely fail once we’ve moved from 5.4.1 to 5.4.2 using the master branch.

We’ll be doing a minor bugfix release in the next few days so it’ll make it’s way onto the master branch then.

1 Like