Created
February 19, 2013 02:41
-
-
Save brandonbloom/4982655 to your computer and use it in GitHub Desktop.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
[21:35:40] <dnolen> definitely - we sould have a compiler.test namespace | |
[21:35:45] <dnolen> should | |
[21:36:12] <bbloom> dnolen: ok, the reason :ret was removed was because it relied on non-do forms understanding statements | |
[21:37:15] <bbloom> dnolen: basically, type inference can happen more generally, so we really shouldn't check :ret, we should check :tag or whatever generally... only analyzing 'do should look at :ret b/c it knows about that and :statements | |
[21:37:50] <bbloom> just an oversight on my part, should be pretty easy to get :let, :letfn, :try, etc to look at their body node for :ret and infer the type | |
[21:37:56] <yedi> i'm getting a dependency error when running lein ring server for libs that i don't even include in my project.clj, is this expected? | |
[21:38:15] <yedi> specifically it says hiccup can't be found and i'm not even using hiccup | |
[21:38:32] <yedi> this only started happening when i added sandbar to the project | |
[21:38:47] dyad (~dan@5e052914.bb.sky.com) left IRC. (Ping timeout: 260 seconds) | |
[21:39:12] <bbloom> dnolen: RE: compiler.test namespace. i think the easiest thing is just to just check in some pretty printed code. maybe core.test | |
[21:39:25] atyz (~atyz@196-215-129-41.dynamic.isadsl.co.za) left IRC. (Ping timeout: 260 seconds) | |
[21:40:07] dek5 (~eugene@mail.quickpay.ru) joined the channel. | |
[21:40:12] <bbloom> dnolen: gold file tests are great for things that may change and need a human to inspect to confirm/accept |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment