morganstanley / hobbes

A language and an embedded JIT compiler
http://hobbes.readthedocs.io/
Apache License 2.0
1.16k stars 105 forks source link

Disallow string record labels with leading .'s #356

Closed adam-antonik closed 4 years ago

adam-antonik commented 4 years ago

Sorry, but my previous PR introduced the possibility of chaos, i.e. {".f1" = 1}, or worse {a=1, ".pa"=2}. Given the wide-spread use of '.' internally, it is probably better to just directly disallow such labels or accessors directly.

smunix commented 4 years ago

Good catch indeed!