On Jan 14, 10:53 am, Robert Pfeiffer <pfeiffer.rob...@googlemail.com>
wrote:
> On 14 Jan., 13:59, Rich Hickey <richhic...@gmail.com> wrote:
>
> > [...] the toString in the patch is quite
> > inefficient.
>
> Becausesymbolsare immutable, they could cache the printable
> representation in an extra field, but this would double the space
> occupied by them.
>
> > Second, it would be better not to replicate the logic for
> > macro characters etc that already exists in the reader. That's not to
> > say the reader stuff is ready for reuse as is, but making it so is the
> > right thing.
>
> Issue 13 is related to this. Syntactic validation ofsymbolsin
> clojure.core/symbol depends on the reader syntax of clojure also.
>
> Would this patch eliminate the need to validatesymbols? Are there
> other constraints than that the printed representation read again
> should result in the same symbol?
>
Again, I haven't thought this through, but the general idea is to put
this logic in a single place.
Rich
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Clojure" group.
To post to this group, send email to clojure@googlegroups.com
To unsubscribe from this group, send email to
clojure+unsubscr...@googlegroups.com
For more options, visit this group at
http://groups.google.com/group/clojure?hl=en
-~----------~----~----~----~------~----~------~--~---
