calf icon indicating copy to clipboard operation
calf copied to clipboard

0.90.2 fails to build: no matching member function for call to 'lerp_table_lookup_float_mask'

Open yurivict opened this issue 5 years ago • 8 comments

The FreeBSD port fails to build:

In file included from plugin.cpp:33:
./calf/organ.h:160:19: error: no matching member function for call to 'lerp_table_lookup_float_mask'
        return ph.lerp_table_lookup_float_mask(data, ORGAN_BIG_WAVE_SIZE - 1);
               ~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~

yurivict avatar Apr 17 '19 07:04 yurivict

@kfoltman Could you please help with this? I can't update the FreeBSD port because of this regression.

yurivict avatar Jul 09 '19 20:07 yurivict

0.90.3 still has this problem. The problem exists when built with clang-8, gcc-8 builds it ok.

Please fix the clang build.

yurivict avatar Aug 02 '19 06:08 yurivict

Which version of clang-8 is it? I can't reproduce this on Linux.

kfoltman avatar Aug 02 '19 14:08 kfoltman

$ c++ -v FreeBSD clang version 8.0.0 (tags/RELEASE_800/final 356365) (based on LLVM 8.0.0)

yurivict avatar Aug 02 '19 15:08 yurivict

It compiles with version 8.0.1-+rc4-1 and 7.0.1-8 - perhaps 8.0.0 has some bugs/regressions that result in spurious errors.

Meanwhile, I've started trying to fix all the clang-8 warnings. Most are harmless, but I found a few potential bugs or at least things done not exactly right.

kfoltman avatar Aug 02 '19 18:08 kfoltman

Actually, no, I had an outdated version, without recent changes by Markus.

I can reproduce it now - seems to be a template substitution problem, so might be easy enough to fix.

kfoltman avatar Aug 02 '19 19:08 kfoltman

I just pushed a possible fix for the clang compilation error, plus some easy warning fixes.

kfoltman avatar Aug 02 '19 20:08 kfoltman

@kfoltman I just pushed a possible fix for the clang compilation error, plus some easy warning fixes.

Thanks, the latest git head as of today (0f45a436b729e71cc975d82beb4f369026e5db9f) does built with no errors with clang-3.9 on Debian Stretch, while the latest release (0.90.3) fails. A new bugfix release including whatever magic change you made above to fix the build error would be welcome!

giuliomoro avatar Aug 18 '20 17:08 giuliomoro