search

bakpakin / Fennel

Lua Lisp Language
https://fennel-lang.org
MIT License
2.48k stars 126 forks source link

Issue when strings containing @ are simplified #451

Closed alexaandru closed 1 year ago

alexaandru commented 1 year ago

First of all, apologies if this is not the right place for reporting bugs related to fnlfmt. I would appreciate if you pointed me to the right place if that's the case.

Now for the actual issue:

Given an x.fnl file containing:

{"@foo.bar" 42}

fnlfmt x.fnl produces {:@foo.bar 42} which then throws x.fnl:1:2 Parse error: invalid character: @ error. I use Fennel 1.3.1-dev on PUC Lua 5.4 and fnlfmt version 0.3.1-dev.

technomancy commented 1 year ago

No problem; this is fine, thanks. In fact, this is a bug in Fennel itself; you get the same problem putting that table into the Fennel repl outside antifennel. The root cause is probably in src/fennel/view.fnl.

technomancy commented 1 year ago

This should be fixed in 8404556; thanks for reporting.

alexaandru commented 1 year ago

Awesome & you're most welcome! Cheers! :-)