lann / wasm-wave

Web Assembly Value Encoding
Apache License 2.0
38 stars 6 forks source link

Constrain char/string literal characters #31

Open lann opened 10 months ago

lann commented 10 months ago

Currently char and string escapes are only required where lexing would otherwise fail: \\ and either \' (char) or \" (string).

In order to avoid human parsing problems, I think a few other escapes should be mandatory:

lann commented 10 months ago

See: https://github.com/peterhuene/wac/blob/b294ae04cd85f12a619db08165b3117b8f977b0d/crates/wac-parser/src/lexer.rs#L46-L82