Clojure

Converted from http://www.assembla.com/spaces/clojure/tickets/210
Attachments:
sorted-set-by.diff - https://www.assembla.com/spaces/clojure/documents/cCsdcIZ_Wr3QwreJe5aVNr/download/cCsdcIZ_Wr3QwreJe5aVNr
ssb2.diff - https://www.assembla.com/spaces/clojure/documents/bHx3hk2Her3QkseJe5aVNr/download/bHx3hk2Her3QkseJe5aVNr

timothypratley said: [file:cCsdcIZ_Wr3QwreJe5aVNr]: removed bad initializers, added tests

timothypratley said: public static PersistentTreeSet create(Comparator comp, Object... init)
was shadowing
public static PersistentTreeSet create(Comparator comp, List init)

I can't see any reason why Object... init should be there (it is only for sorted-set-by but not needed) so I've simply removed it and added test coverage. Please advise if this is valid or if there is any reason Object... init is useful.

richhickey said: FYI - please don't assign things to me - thanks

richhickey said: 'ant test' fails for clojure, with:
[java] ERROR in (test-sorted-set-by) (test_clojure.clj:77)
[java] expected: (set? (sorted-set-by <))
[java] actual: java.lang.NullPointerException: null
[java] at clojure.lang.PersistentTreeSet.create (PersistentTreeSet.java:33)

timothypratley said: Hi Rich,

Ok thanks.

I tried from a fresh branch, for some reason 'ant test' does not recompile the java code (which has changed very slightly to fix the bug). However if you first run 'ant' then run 'ant test' it works fine and all tests pass. This seems to show that the build.xml isn't finding the dependency, but that is unrelated to this patch I believe.

Can you please try again with a clean build before ant test?

Regards,
Tim.

richhickey said: I did ant + ant test. Just tried again, same problem. Perhaps some important change didn't make it into your patch? Please pull down master and apply your patch to it in order to reproduce/diff. Thanks.

timothypratley said: Short answer - I am pretty sure the next attachment will fix it!

Long answer -

Hmmmm very odd! I can't reproduce any error here so it must be related to my environment somehow.

git clone git://github.com/richhickey/clojure.git
cd clojure
ant
ant test
works fine.... of course
git am -s < sorted-set-by.diff
ant
[java] ERROR in (test-sorted-set-by) (test_clojure.clj:77)
[java] expected: (set? (sorted-set-by <))
[java] actual: java.lang.NullPointerException: null
[java] at clojure.lang.PersistentTreeSet.create (PersistentTreeSet.java:50)
expected, line number 50 is different from 33, again that's expected
ant
ant test
works fine for me but errors on line 33 PersistentTreeSet for you
PersistentTreeSet line 33:
for(Object key : init)
init is clearly null
I added some printouts - it never gets called for me - but does for you
Summary - must be shadowing again!!! Argghhhh...

Uploading patch now which should resolve.

timothypratley said: [file:bHx3hk2Her3QkseJe5aVNr]: new patch