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-852 Clojure <p>Note the following stacktrace line numbers are from the latest 1.4-snapshot, though this happens on 1.3 as well.</p> <p>Example:</p> <div class="preformatted panel" style="border-width: 1px;"><div class="preformattedContent panelContent"> <pre>user=&gt; (defn foo ^long [^long x] x) #'user/foo user=&gt; (def x (inc (foo 10))) CompilerException java.lang.IllegalArgumentException: Unable to resolve classname: long, compiling:(NO_SOURCE_PATH:2) user=&gt; (pst) CompilerException java.lang.IllegalArgumentException: Unable to resolve classname: long, compiling:(NO_SOURCE_PATH:2) clojure.lang.Compiler.analyzeSeq (Compiler.java:6444) clojure.lang.Compiler.analyze (Compiler.java:6244) clojure.lang.Compiler.analyze (Compiler.java:6205) clojure.lang.Compiler.analyzeSeq (Compiler.java:6432) clojure.lang.Compiler.analyze (Compiler.java:6244) clojure.lang.Compiler.access$100 (Compiler.java:37) clojure.lang.Compiler$DefExpr$Parser.parse (Compiler.java:492) clojure.lang.Compiler.analyzeSeq (Compiler.java:6437) clojure.lang.Compiler.analyze (Compiler.java:6244) clojure.lang.Compiler.analyze (Compiler.java:6205) clojure.lang.Compiler.eval (Compiler.java:6497) clojure.lang.Compiler.eval (Compiler.java:6459) Caused by: IllegalArgumentException Unable to resolve classname: long clojure.lang.Compiler$HostExpr.tagToClass (Compiler.java:1003) clojure.lang.Compiler$InvokeExpr.getJavaClass (Compiler.java:3474) clojure.lang.Compiler.getMatchingParams (Compiler.java:2304) clojure.lang.Compiler$StaticMethodExpr.&lt;init&gt; (Compiler.java:1510) clojure.lang.Compiler$HostExpr$Parser.parse (Compiler.java:910) clojure.lang.Compiler.analyzeSeq (Compiler.java:6437) clojure.lang.Compiler.analyze (Compiler.java:6244) </pre> </div></div> <p>Note though that the following works fine:</p> <div class="preformatted panel" style="border-width: 1px;"><div class="preformattedContent panelContent"> <pre>user=&gt; (def x (foo (inc 10))) #'user/x </pre> </div></div> CLJ-852

IllegalArgumentException thrown when defining a var whose value is calculated with a primitive fn.

Defect Critical Closed Completed Ben Smith-Mannschott Alexander Taggart Mon, 10 Oct 2011 15:24:07 -0500 Fri, 23 Mar 2012 08:40:35 -0500 Fri, 23 Mar 2012 08:40:35 -0500 Release 1.3 Release 1.4 4 5 <p><a href="http://dev.clojure.org/jira/browse/CLJ-852" title="IllegalArgumentException thrown when defining a var whose value is calculated with a primitive fn."><del>CLJ-852</del></a>-test.patch <br/> reproduces this issue on master</p> <p><a href="http://dev.clojure.org/jira/browse/CLJ-852" title="IllegalArgumentException thrown when defining a var whose value is calculated with a primitive fn."><del>CLJ-852</del></a>-test.patch initially failed to use (is ...) leading to ugly output on failure.</p> <p><a href="http://dev.clojure.org/jira/browse/CLJ-852" title="IllegalArgumentException thrown when defining a var whose value is calculated with a primitive fn."><del>CLJ-852</del></a>-rfc-fix.patch </p> <p>add special cases for "int"…"boolean" to tagToClass()</p> <p>This causes the test to pass, however, there are still some open questions:</p> <p>Why does tagToClass first call maybeClass and then possibly overwrite the result it gets back without even looking at it? This strikes me as surprising. I would have expected a check for null, and then only running the if/else chain of special cases if maybeClass failed to return a non-null result.</p> <p>The suppressed exception in maybeClass commented with "//aargh" makes me wonder if maybeClass and tagToClass are factored correctly.</p> <p>This looks reasonable to me, but given the number of places tagToClass gets called I would definitely want Rich to take a look.</p> <p>See also<br/> <a href="http://groups.google.com/group/clojure/browse_thread/thread/f1672a7e09c34721">http://groups.google.com/group/clojure/browse_thread/thread/f1672a7e09c34721</a></p> <p><a href="http://dev.clojure.org/jira/browse/CLJ-852" title="IllegalArgumentException thrown when defining a var whose value is calculated with a primitive fn."><del>CLJ-852</del></a>-fix.patch removes the RFC comments</p> <p>This fixes the issue. It appears to me like the cases "int" .. "boolean" (i.e. the non-array primitive types) were simply forgotten in tagToClass(). </p> <p>It's not impossible that tagToClass is being misused somehow ant that's the cause of the problem, but I see no evidence for that. (I'd need a better understanding of the code base to make that determination &#8211; mind reading skills would help too.)</p> <p>It's also conceivable that maybeClass() should be doing the resolution from primitive type to Class object, but that doesn't seem likely to me.</p> <p>clj-852-patch2.txt is exactly the same as Ben's <a href="http://dev.clojure.org/jira/browse/CLJ-852" title="IllegalArgumentException thrown when defining a var whose value is calculated with a primitive fn."><del>CLJ-852</del></a>-fix.patch (including keeping his name and dates on the commits), except:</p> <p>(1) I moved his new unit tests to an already-existing file metadata.clj, which seemed to have related tests with def and metadata, whereas Ben's patch adds a brand new test file. Not sure whether keeping the number of files down is important or not.</p> <p>(2) I removed some non-ASCII characters from his commit notes.</p> <p>Both his patch and this newer one apply cleanly to latest master as of Feb 22, 2012. Neither adds any new errors or warnings to compilation or tests (unless you apply the new test part of the patch without his proposed compiler fix, where the new test does fail as expected). No docstrings need changing that I can think of. Ben and I have both signed CAs. Patch status is "Code and Test".</p> <p>Screened. The addition seems harmless, merely recognizing more tags in tagToClass.</p> Approval Ok Global Rank Patch Code and Test Waiting On richhickey