tim-janik / beast

Beast - Music Synthesizer and Composer
GNU Lesser General Public License v2.1
84 stars 12 forks source link

BSE: randomhash.hh: fix compilation on g++ 7.4.0 #120

Closed swesterfeld closed 5 years ago

swesterfeld commented 5 years ago

std::string in C++17 cannot be used at compile time, so constexpr is not possible

tim-janik commented 5 years ago

fnv1a_consthash64() is named consthash because it proivdes a compile-time hash function, removing the constexpr means it cannot be used at compile time. It's a pitty that g++ doesn't handle this as well as clang, I'm removing the entire function for now.