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

Reply via email to