[TRDR-3] Make read-line available in clojure.tools.reader and/or clojure.tools.reader.edn namespace? Created: 15/Mar/13 Updated: 15/Mar/13 Resolved: 15/Mar/13
|Reporter:||Andy Fingerhut||Assignee:||Nicola Mometto|
You mention the enhanced version of read-line that takes a reader argument in the README, but it seems like it is only available in the clojure.tools.reader.reader-types namespace. Is that intentional? Perhaps making it available in the clojure.tools.reader and/or clojure.tools.reader.edn namespace might be more convenient for the user of the library? I understand that the current implementation of read-line is Java-specific, so maybe this is the only reasonable way to expose it.
In any case, documenting the namespace in which this enhanced read-line is available in the README would be good.
|Comment by Nicola Mometto [ 15/Mar/13 10:26 AM ]|
I added more documentation on the readme, making it clear in which namespace read-line is defined.
read-line definitely belongs in the c.t.r.reader-types namespace, since it works on reader-types and returns a string, so it's not a reader function.
I hope this addresses this ticket, I'm closing it, feel free to reopen it you think the doc needs to be more clear.
|Comment by Andy Fingerhut [ 15/Mar/13 7:04 PM ]|
Looks thoroughly documented to me – above and beyond what I would have asked for. Thanks.