Switch to tools.reader for cljs.analyzer/forms-seq

Description

Switches cljs.analyzer to use tools.reader, so we can get more accurate column location for symbols, prepping for more fleshed-out source maps.

Environment

CLJS master

Activity

Show:
David Nolen
July 27, 2013, 8:09 PM

I tried applying this patch and rerunning the bootstrap script. This works but when I try to run script/test I get an error about the tools.reader not being on the classpath. Even trying to require tools.reader at via the repl doesn't work for me.

Sean Grove
July 22, 2013, 8:20 PM

CA's been processed and I'm listed on the contributing page, so shouldn't be blocked by that any longer.

Sean Grove
July 15, 2013, 5:00 PM

Updated with POM information and bootstrap update

David Nolen
July 15, 2013, 4:43 PM

Excellent, we need two more things in this patch, can you update bootstrap.sh and the POM file? Thanks.

Completed
Your pinned fields
Click on the next to a field label to start pinning.

Assignee

Unassigned

Reporter

Sean Grove

Patch

Code