Snapchat / djinni

A tool for generating cross-language type declarations and interface bindings. Djinni's new home is in the Snapchat org.
Apache License 2.0
166 stars 44 forks source link

New version and remove experimental from WASM #131

Closed paulocoutinhox closed 1 year ago

paulocoutinhox commented 1 year ago

Hi,

I have some suggestions:

  1. Create a new version with latest things
  2. Remove experimental texts from WASM, ex: image
  3. Remove the headers not used in DataView.hpp:
    #include <functional>
    #include <memory>

    and use only:

    #include <cstddef>
    #include <cstdint>

Thanks.

paulocoutinhox commented 1 year ago

The experimental optional still required? Since the supported version is C++17?

https://github.com/Snapchat/djinni/search?q=experimental