http://dev.clojure.org/jira This file is an XML representation of an issue en-us 4.4 649 25-07-2011 http://dev.clojure.org/jira/browse/CLJ-441 Clojure <p>Checking in coercions has been added, making them safe, but removing the bit-twiddling power. We need some unchecked (truncating) coercion ops</p> CLJ-441

Add unchecked coercions

Enhancement Blocker Closed Completed Rich Hickey Assembla Importer Tue, 28 Sep 2010 10:36:00 -0500 Wed, 8 Dec 2010 15:52:28 -0600 Fri, 26 Nov 2010 13:32:33 -0600 Release 1.3 0 0 <p>Converted from <a href="http://www.assembla.com/spaces/clojure/tickets/441">http://www.assembla.com/spaces/clojure/tickets/441</a><br/> Attachments:<br/> unchecked-casts.diff - <a href="https://www.assembla.com/spaces/clojure/documents/dVK0q-Znmr37hgeJe5cbLA/download/dVK0q-Znmr37hgeJe5cbLA">https://www.assembla.com/spaces/clojure/documents/dVK0q-Znmr37hgeJe5cbLA/download/dVK0q-Znmr37hgeJe5cbLA</a><br/> unchecked-casts-without-typo.diff - <a href="https://www.assembla.com/spaces/clojure/documents/alGr58Znur34SreJe5cbCb/download/alGr58Znur34SreJe5cbCb">https://www.assembla.com/spaces/clojure/documents/alGr58Znur34SreJe5cbCb/download/alGr58Znur34SreJe5cbCb</a><br/> 0441-unchecked-coercions-plus-table.patch - <a href="https://www.assembla.com/spaces/clojure/documents/c1gyXG1xKr37lweJe5cbLr/download/c1gyXG1xKr37lweJe5cbLr">https://www.assembla.com/spaces/clojure/documents/c1gyXG1xKr37lweJe5cbLr/download/c1gyXG1xKr37lweJe5cbLr</a></p> <p>richhickey said: See also #284 which will be addressed by this</p> <p>ataggart said: [<a href="file:dVK0q-Znmr37hgeJe5cbLA">file:dVK0q-Znmr37hgeJe5cbLA</a>]: implementation and tests for unchecked casts</p> <p>ataggart said: Added unchecked casts from every numeric primitive and Object to every numeric primitive. Character and char can only get cast to int, as per the convention with the checked cast.</p> <p>Added tests for the above in clojure.test-clojure.numbers.</p> <p>ataggart said: [<a href="file:alGr58Znur34SreJe5cbCb">file:alGr58Znur34SreJe5cbCb</a>]: Same as above but without the typo in the docs</p> <p>stu said: [<a href="file:c1gyXG1xKr37lweJe5cbLr">file:c1gyXG1xKr37lweJe5cbLr</a>]</p> <p>stu said: Third patch subsumes previous two and adds a test table for various conversions.</p> <p>richhickey said: I'd be interested in better names than unchecked-*</p> <p>any ideas? Think also about unchecked-add et al</p> <p>richhickey said: let's create a clojure.unchecked ns and put these and the other unchecked arithmetic ops in there, using same names as normal, e.g. <ins>, inc etc. Only put the long versions in there, we'll have to decide separately about the truncating int-add etc. Then people can either use unchecked or alias and do uc/</ins> etc</p> <p>Let's create a clojure.unchecked ns and put these and the other unchecked arithmetic ops in there, using same names as normal, e.g. &#43;, inc etc. Only put the &#45;long versions in there, we'll have to decide separately about the truncating int-add etc. Then people can either use unchecked or alias and do uc/+ etc</p> <p>error: patch failed: test/clojure/test_clojure/numbers.clj:31</p> <p>Fixed up the patch to merge properly as of 8225407032ea643cbe3db7f35ef97b1230fc65b8</p> Approval Ok Global Rank Patch Code and Test