search

WebAssembly / wasm-c-api

Wasm C API prototype
Apache License 2.0
537 stars 77 forks source link

Use enum classes #121

Closed rossberg closed 4 years ago

rossberg commented 4 years ago

Addresses #118. But it is a breaking change and introduces extra verbosity. Is it worth it?

MattPD commented 4 years ago

FWIW, the verbosity can be somewhat reduced in C++20, given the Using Enum feature: https://wg21.link/P1099 (admittedly, given the current goal of "no post-C++11 features in C++ API", this may be a little bit more of a longer-term aspect).

rossberg commented 4 years ago

Two thumbs up and no detractor after months -- merging. :)