janestreet / ppx_sexp_conv

Generation of S-expression conversion functions from type definitions
MIT License
88 stars 17 forks source link

sexp grammar? #29

Open chetmurthy opened 4 years ago

chetmurthy commented 4 years ago

I feel like this isn't a real issue, but OTOH, it's not worth disturbing all the readers of the ocaml discussion forum.

Are s-expression grammars no longer supported in ppx_sexp_conv ? There's documentation, but I don't see anything in the code ..... and the tests don't appear to work either.

Just asking so I can know one way or the other ....

aalekseyev commented 3 years ago

It's not "no longer supported", but it may be the case of "not yet supported". I believe there's work underway to make sexp grammars work / become more useful.

The tests do pass inside Jane Street, so if there's something that looks like it should probably work, but doesn't, feel free to open a more specific issue.