Without commenting on the usefulness of this (it doesn't really
interest me), there are a couple style things I would change.
First, :added ought to be 1.3 (maybe 1.2?), not 1.0. Second, the
newsgroup seems to think that if you're changing clojure.core you
might as well update from (. staticMember class args) to (class/
staticMember args) while you're in there. You could also fix the other
shift operators if you wanted.

On Aug 23, 1:15 pm, Cameron <cpuls...@gmail.com> wrote:
> Hey all, I had an itch this weekend... so I scratched it.
>
> I was doing some bit twiddling stuff and needed the Java >>> operator.
> I had written some workarounds but they were overly-complicated and
> slower than I would have hoped, so I dug directly into core and added
> it as a first class citizen next to bit-shift-left and bit-shift-
> right. Being a very creative person, I named it ">>>". The performance
> is on par with the normal signed shift operators.
>
> To anyone who is interested, the patch can be found 
> at:http://github.com/cpulsford/bitops/blob/logical-assumption/unsigned-r...
>
> I'd love feedback, especially since this is my first time digging into
> the Java side of Clojure.
>
> Could this maybe be something useful for core?

-- 
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