search

bpsm / edn-java

a reader for extensible data notation
Eclipse Public License 1.0
100 stars 24 forks source link

edn-java doesn't recognize 'foo//' as a symbol #30

Closed bpsm closed 10 years ago

bpsm commented 11 years ago

I suspect this is a defect. Awaiting feedback on edn-format/edn#51 to know how to proceed.

bpsm commented 11 years ago

I'm not going to merge this change right now:

The commit referenced by edn-format/edn#51 changes LispReader to accept foo// as meaning the symbol / in the namespace foo, but does not update EdnReader accordingly. In the face of ambiguity in the spec, it makes sense to stay compatible with EdnReader.

bpsm commented 10 years ago

EdnReader now supports reading 'foo//', so we will do so also. https://github.com/clojure/clojure/commit/a989c01eed7ce29cc7672b7a402e4f335d792882

bpsm commented 10 years ago

Implemented https://github.com/bpsm/edn-java/commit/8024c5655576d9704824a50910262a8fbd25042a