Clojure

GC Issue 22: Make sure all integer ops return smallest representation

Details

  • Type: Defect Defect
  • Status: Closed Closed
  • Resolution: Completed
  • Affects Version/s: None
  • Fix Version/s: None
  • Component/s: None
  • Labels:
    None

Description

Reported by richhickey, Dec 28, 2008
Right now not all integers that can be represented as longs are, leading to
cases of different representations of integers of same value, and the
resulting problems for hashCode.

Comment 1 by christophe.grand, Dec 29, 2008
added LongOps 
uncommented the two lines in reduce(BigInteger)
 LongOps.patch
4.9 KB Download
Comment 2 by richhickey, Dec 29, 2008
Thanks! I'm having some problems applying the patch. I get a LongOps class with
several opsWith(LongOps) entries and other errors.

Would it be possible to produce a patch/diff with more context?
Comment 3 by christophe.grand, Dec 29, 2008
diff -u5 this time, hoping it works better for you than plain svn diff.
 LongOps.diff
5.6 KB Download
Comment 4 by richhickey, Dec 29, 2008
Patch applied - svn 1190 - thanks!
Status: Fixed

People

  • Assignee:
    Unassigned
    Reporter:
    Anonymous
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: