search

atom / language-clojure

Clojure package for Atom
Other
49 stars 48 forks source link

False-positive, but inconsistent, parsing of :/ keyword #84

Closed peter-lyons-kehl closed 5 years ago

peter-lyons-kehl commented 5 years ago

Hi. Thank you for language-clojure,

#{:| :/ :- :L :+} generates a "Read error: Invalid use of / in symbol name". That is incorrect - the map literal works (in current CLJ 1.10). Also note: the literal is for a map of keywords, but the error mentions 'symbol'.

That bug is local/inconsistent. If you have a keyword :/ on its own, it parses well.

rsese commented 5 years ago

Thanks for the report!

Can you share a minimal full code example that we can copy and paste that shows the problem? It's helpful in general but also specifically for me since I haven't used Clojure before.

rsese commented 5 years ago

Oh also, please use the issue template when creating issues - the template details and format is super helpful for us when triaging issues.

peter-lyons-kehl commented 5 years ago

Hi Robert @rsese.

The minimal full code example: :/. That is a literal for a keyword whose name is one character long, containing a slash. However, it upsets Atom: image

That syntax works in clj 1.10.0 (Clojure REPL): image

rsese commented 5 years ago

lol gotcha, thanks :+1:

Also I think I misunderstood your report, so just to circle back to your original comments, you mentioned:

{:| :/ :- :L :+} generates a "Read error: Invalid use of / in symbol name".

This wouldn't be generated by language-clojure, this package just provides syntax highlighting.

Are you using a specific package for Clojure linting/validation? If so, that would be the package to report the error to.

no-response[bot] commented 5 years ago

This issue has been automatically closed because there has been no response to our request for more information from the original author. With only the information that is currently in the issue, we don't have enough information to take action. Please reach out if you have or find the answers we need so that we can investigate further.