gfredericks / seventy-one

71 in Clojure
Eclipse Public License 1.0
39 stars 9 forks source link

Correct base-10 provincialism #8

Closed senorflor closed 6 years ago

gfredericks commented 9 years ago

This would be a breaking change.

aengelberg commented 9 years ago

I'd propose making seventy-one a ^:dynamic variable (71 by default), while keeping the seventy-ones list for alternative bases to bind it to.

Another idea would be to introduce a (with-radix r ...) macro which re-binds seventy-one to 71 parsed in the new base.

To me that seems like a sound addition that preserves the functionality you're aiming for while keeping the base-10 defaults for those already using seventy-one. What do you think of this, Gary?

senorflor commented 6 years ago

Well, this was fun. ∃ base b ∈ ℝ for which it has been 71 days since I submitted this much needed but ultimately rejected correction improvement to seventy-one, which I now retract. (Nice work on the spec!)