Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Does edn consider foo// to be a valid symbol? #51

Closed
bpsm opened this issue May 27, 2013 · 2 comments
Closed

Does edn consider foo// to be a valid symbol? #51

bpsm opened this issue May 27, 2013 · 2 comments

Comments

@bpsm
Copy link

bpsm commented May 27, 2013

This is a follow-up to the recent commit clojure/clojure@88cad25.

Currently, bpsm/edn-java rejects the symbol foo//. I suspect now that this is not correct behavior.

commit 88cad2510243289d2bbe28cf8bd14007d9df5fc3
Author: Chris Gray <chrismgray@gmail.com>
Date:   2011-11-09 22:23:15 -0700

    Allow foo// meaning the function / in namespace foo

    Added regression tests

    Signed-off-by: Stuart Halloway <stu@thinkrelevance.com>
@richhickey
Copy link
Contributor

Does the spec imply it doesn't?

@wagjo
Copy link

wagjo commented May 31, 2014

I would say that specs permits / to be used either alone or once only in the middle of the symbol. I don't see where the specs allows for / to be part of the name. Current specs reads

/ has special meaning in symbols. It can be used once only in the middle of a symbol to separate the prefix (often a namespace) from the name, e.g. my-namespace/foo. / by itself is a legal symbol, but otherwise neither the prefix nor the name part can be empty when the symbol contains /.

tomjakubowski added a commit to tomjakubowski/edn-rs that referenced this issue Dec 12, 2014
tomjakubowski added a commit to tomjakubowski/edn-rs that referenced this issue Dec 12, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants