Thanks for pointing this out. Fixed in master thanks to your patch. Also landed a patch which should cover more problematic warning cases around arithmetic.
David On Wed, Dec 24, 2014 at 8:04 AM, Thomas Heller <[email protected]> wrote: > Hey, > > I just looked at the warnings code and it seems like a mistake that disabling > one warning actually disables 3 by accident. > > Anyways, the warning definitions [1] are all pretty self explanatory, more so > if you look at the actual text the produce in the error functions below [1]. > > You could wait for [2] if you want to get the other warnings back or use > :warnings with > > {:invalid-arithmetic false > :undeclared-var true > :undeclared-ns true} > > > > HTH, > /thomas > > > [1] > https://github.com/clojure/clojurescript/blob/d3da2349c175a9066f4d9e4476302ff497f51937/src/clj/cljs/analyzer.clj#L40-L62 > [2] http://dev.clojure.org/jira/browse/CLJS-923 > > > > On Wednesday, December 24, 2014 7:12:30 AM UTC+1, [email protected] wrote: >> A recent upgrade has introduced a bunch of new warnings of these types: >> >> 1. cljs.core/+, all arguments must be numbers, got [#{nil clj-nil} #{nil any >> clj-nil}] instead.... >> 2. WARNING: Use of undeclared Var... >> >> I tried adding some changes to my project.clj with out much luck or >> understanding for what is or should be possible: >> >> :compiler {... >> :warning-level :quiet}} >> >> Which does nothing. And then: >> >> :compiler {... >> :warnings {:invalid-arithmetic false}} >> >> Which suppresses both the arithmetic and the undeclared vars warnings. >> >> Is there any place that provides good documentation on these clojurescript >> compiler options? I've checked both of these: >> https://github.com/clojure/clojurescript/wiki, >> https://github.com/emezeske/lein-cljsbuild, however there's not much depth >> or they are out of date. >> >> Thanks, >> Tim > > -- > Note that posts from new members are moderated - please be patient with your > first post. > --- > You received this message because you are subscribed to the Google Groups > "ClojureScript" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to [email protected]. > To post to this group, send email to [email protected]. > Visit this group at http://groups.google.com/group/clojurescript. -- Note that posts from new members are moderated - please be patient with your first post. --- You received this message because you are subscribed to the Google Groups "ClojureScript" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To post to this group, send email to [email protected]. Visit this group at http://groups.google.com/group/clojurescript.
