What's the C++ version used for DaisySP? about daisysp HOT 5 CLOSED

I'm a fairly new migrant to C++ from C -- So with that, I don't have a strong basis for an opinion.

As of now, I don't think we're using any features beyond C++11.

Unless there are any reasons not to use c++17 over 14 or 11, then I think that's fine.

from daisysp.

Worth mentioning that I just checked on the STM32 Arduino Core (I made a wrapper of this library for Arduino) and it uses gnu++14 for its builds.

from daisysp.

Hmm, that's a shame. I'm not sure if this can be changed on the Arduino side, but it seems not trivial. C++17 has really neat features for compile time optimization. But I guess C++14 is better than nothing :-)

from daisysp.

Just commenting so this doesn't seem like its stagnating. I think, at least for now, we're going to go with c++14 so that compatibility with Arduino doesn't become an immediate issue.

I set this on my WSL build yesterday, and had a few warnings/errors that I didn't have the time to dig into. Should be able to get to this before the weekend.

from daisysp.

Set to 14 for now, but open to moving to 17 at some point.

I'm still not totally familiar with the arduino-related build process. In my very brief attempt to build libdaisy with c++17 I realized the HAL was throwing a lot of register related warnings that did not happen in c++14. Definitely worth investigating at some point.

from daisysp.