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/CLJS-480 ClojureScript <p>It looks like <tt>&#42;cljs-data-readers&#42;</tt> is not being bound to <tt>&#42;data-readers&#42;</tt> where it should be.</p> <p>At a minimum, <tt>cljs.repl/load-stream</tt> does not. The practical effect of this instance is that e.g. <tt>(load-file "some/cljs/file.cljs")</tt> will fail if that file or any other contains a user-defined literal tag. Similar behaviour exists elsewhere, e.g. <tt>(load-namespace 'some.cljs.ns)</tt> will not load a namespace that contains user-defined tags (I <em>think</em> that one is due to <tt>&#42;cljs-data-readers&#42;</tt> not being bound in the analyzer).</p> <p>This is related to <a href="http://dev.clojure.org/jira/browse/CLJS-479" title="load-file in REPL improperly qualifies current-namespace ::keywords">CLJS-479</a> insofar as it's another symptom of ClojureScript loading code in subtly different ways in multiple places.</p> <p>I'm happy to do either of:</p> <ol> <li>submit issues+patches fixing issues as I encounter them, or</li> <li>attempt a more significant refactoring that sets up (hopefully) one and only one place where ClojureScript reads code (i.e. uses <tt>LispReader</tt>) so that behaviour can be consistent</li> </ol> <p>I'm somewhat hesitant re: (2) insofar as the specific requirements of the compiler and analyzer and REPL may be divergent in ways that I'm not yet aware of, but it seems like it's worth a shot.</p> CLJS-480

*cljs-data-readers* is bound to *data-readers* inconsistently

Defect Major Open Unresolved Unassigned Chas Emerick Fri, 1 Mar 2013 15:44:22 -0600 Tue, 21 May 2013 10:06:43 -0500 0 0 <p>Attached patch 0001 binds <b>data-readers</b> in cljs.compiler/parse-ns.</p> <p>This allows the compiler to run, even if files with reader tags are already compiled.</p> <p>This in turn allows multiple test compilations to be brought back, which I'll submit as part of a patch for an additional debugging flag, if nobody beats me to it.</p> <p>I think this won't fix load-file <img class="emoticon" src="http://dev.clojure.org/jira/images/icons/emoticons/help_16.gif" height="16" width="16" align="absmiddle" alt="" border="0"/>, but after the hotfix we should probably look for the proper place in the compiler to bind <b>data-readers</b>.</p> <p>Patch 0002 is my overhaul for the test script.<br/> Since it's a test case for patch 0001, I think it's justifiable on this ticket.</p> <p>Note: On my machine, with SpiderMonkey 185, the test case (not (integer? 1e+308)) fails. It probably doesn't have to do anything with the removed -n flag (there is no js -n omm), I just wanted to mention it.</p> <p>Please move the test script patch to a different ticket. Thanks.</p> Global Rank