Please attach a patch to the ticket so it can be reviewed there. Thanks! On Tue, May 8, 2012 at 6:20 PM, Dave Sann <daves...@gmail.com> wrote:
> All is quite - so I raised http://dev.clojure.org/jira/browse/CLJS-243 > > > On Wednesday, 2 May 2012 22:28:01 UTC+10, Dave Sann wrote: >> >> I needed unicode support for the clojurescript reader so I have made an >> attempt at adding it. >> >> This works for me - but it is my first foray into changes to this source. >> >> my commit is here https://github.com/**davesann/clojurescript/commit/** >> 1f142d283959eead06ba0d00f9cba7**fe3053d4a8<https://github.com/davesann/clojurescript/commit/1f142d283959eead06ba0d00f9cba7fe3053d4a8> >> >> I have done the initial change in a relatively simple fashion. >> >> I would appreciate if someone could review and make constructive comments >> as to changes that might need to be made. >> >> if/when it passes scrutiny - I can submit a pull request or patch as >> required. >> >> Cheers >> >> Dave >> >> -- > 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 > Note that posts from new members are moderated - please be patient with > your first post. > 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 > -- 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 Note that posts from new members are moderated - please be patient with your first post. 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