Fork me on GitHub
#cljs-dev
<
2015-10-30
>
kamn00:10:20

For a potential solution to cljs-868, I created a macro in the analyzer but I am having issues with the script/self-host-test

kamn00:10:49

If I just copy one macro (no-warn) and rename it (no-warn2) and use it (in analyze-fn-methods-pass2) it will fail.

kamn00:10:10

Even though I have not changed the underlying functionality in any way

kamn00:10:34

I feel like I am missing something basic but I have tried clean/rebuild/bootstrap with no luck

kamn00:10:12

I error out on the first defn test in self-host test with an "Could not eval" -> "cannot read property of undefiend" error

kamn00:10:16

Any thoughts?

dnolen00:10:09

cljs-868 does not really need a macro

dnolen00:10:35

all you need to do is disable all warnings except for a recur-mistach or whatever

dnolen00:10:07

I would get that working first

kamn01:10:07

I might have over engineered and I will look back at what I have

kamn01:10:21

The macro is not really that important. The important part is having a bind some dyn things to make sure that the :fn-arity warning does not send error until the second pass

kamn01:10:06

and then making sure that all warnings but :fn-arity warning are turned off when it starts the second pass

kamn01:10:18

so instead of changing no-warn I just created a new macro to activate second-pass warnings only in cljs-warnings