Fix a few glitches in the ocaml codegen - Githubissues

zefchain / serde-reflection

Rust libraries and tools to help with interoperability and testing of serialization formats based on Serde.

Apache License 2.0

124 stars 22 forks source link

Fix a few glitches in the ocaml codegen #9

Closed ma2bd closed 2 years ago

ma2bd commented 2 years ago

Summary

Add missing to_caml_case()
Also test if field names are Ocaml keywords
Prefix enum variants with the name of the enum (this is a bit sad in Ocaml but hard to avoid because Rust works like this)

Test Plan

CI

curl https://raw.githubusercontent.com/aptos-labs/aptos-core/master/testsuite/generate-format/tests/staged/aptos.yaml > aptos.yaml

rm -rf test

cargo run --bin serdegen -- --language ocaml --with-runtimes serde bcs --target-source-dir test -- aptos.yaml

(cd test && dune build)