Skip to content
This repository has been archived by the owner on Jan 28, 2019. It is now read-only.

Rollback ^:deprecated on ns-name #47

Closed
arrdem opened this issue Feb 1, 2016 · 0 comments
Closed

Rollback ^:deprecated on ns-name #47

arrdem opened this issue Feb 1, 2016 · 0 comments

Comments

@arrdem
Copy link
Collaborator

arrdem commented Feb 1, 2016

ns-name has a distinct purpose, [Namespace] -> Symbol which is distinct from name being [Named] -> String. While most uses of ns-name were of the form (name (ns-name (the-ns ...))), that doesn't mean that dropping ns-name to ^:deprecated status was a great idea.

A better change would have been to move the old Namespace.getName():Symbol to Namespace.getNamingSymbol():Symbol or something and then to preserve ns-name using the explicit Symbol getter.

arrdem added a commit that referenced this issue Feb 1, 2016
Deprecating ns-name was a mistake, since it serves a unique purpose with
respect to name, being to fetch the naming _symbol_ rather than the
naming _string_.

Fixes #47
@arrdem arrdem closed this as completed in #48 Feb 1, 2016
arrdem added a commit that referenced this issue Feb 2, 2016
Deprecating ns-name was a mistake, since it serves a unique purpose with
respect to name, being to fetch the naming _symbol_ rather than the
naming _string_.

Fixes #47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant