header improvements - Githubissues

extism / cpp-sdk

Extism C++ Host SDK

BSD 3-Clause "New" or "Revised" License

7 stars 3 forks source link

header improvements #14

Closed G4Vi closed 6 months ago

G4Vi commented 7 months ago

[x] make Buffer::vector,string functions const
[x] include ordering? remove extern "C" from extism.h include
[x] Review many class variables being exposed public
[x] Review object passing such as std::vector being passed around by value
[x] Bump to C++ 17, replace ManifestKey with std::optional
[x] Consider making extism.h a private dependency instead of a public one
[x] Consider making json/json.h a private dependency instead of a public one