fasterthanlime / shin

:warning: (def shin (dissoc clojurescript :jvm :google_closure)) (deprecated)
MIT License
35 stars 1 forks source link

Relay sourcemap info to escodegen #12

Open fasterthanlime opened 9 years ago

fasterthanlime commented 9 years ago

Right now it's lost in translation (literally)

fasterthanlime commented 9 years ago

This is a long piece of work btw, translator is 1.7K lines long so far :( a DSL could help, but at perf. costs maybe? — but it's not a complicated piece of work. There needs to be a way to convert Token to SourceLocation & put that in the JST whenever translating the AST.

Thanks to #98 we're keeping a lot of the token info even when passing through macros, so this issue is even more interesting than I thought.

fasterthanlime commented 9 years ago

Also, maybe there's no need to relay info for every token (don't care about numbers?), mostly function/method calls & symbols, I think. Even that would be significant progress.