FFTW: No matching constructor for initialization of FFTWImpl

#1

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:

0 Likes

#2

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

@ed95

0 Likes

#3

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

0 Likes

#4

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.

0 Likes

#5

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