Skip to content

Instantly share code, notes, and snippets.

@brandonbloom
Created February 19, 2013 02:41
Show Gist options
  • Star 0 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save brandonbloom/4982655 to your computer and use it in GitHub Desktop.
Save brandonbloom/4982655 to your computer and use it in GitHub Desktop.
[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