将行号和列号添加到分析器AST

Question

一些现有的Clojure工具（链接：1）目前建立在tools.analyzer.jvm之上，需要AST节点上有行号和列号。

此数据目前在ClojureScript分析器中缺失，这阻止了这些工具移植到ClojureScript（链接：2）

(链接：1) https://github.com/clojure-emacs/refactor-nrepl
(链接：2) https://github.com/clojure-emacs/refactor-nrepl/issues/195#issuecomment-303910871

Answer 1

julienfantin发表的评论：

这是一个补丁，它添加了行号和列号，并试图规范如何从环境变量中获取这些数据。

Answer 2

anmonteiro发表的评论：

我认为这已经在CLJS-1461中得到了解决，其目标是实现与tools.analyzer AST的完全兼容。

Answer 3

dnolen发表的评论：

CLJS-1461是一个大型项目，我们不确定需要多长时间。在此期间，如果我们能得到补丁，那么朝着这个方向的逐步推进没有问题。

Answer 4

dnolen发表的评论：

这个补丁看起来很好，但最好能得到一些反馈，实际上没有影响到源映射。

Answer 5

julienfantin发表的评论：

不幸的是，我们的主要项目依赖于较老的ClojureScript版本，因此我无法在我们的主代码库上测试这个补丁。你需要注意哪些具体问题呢？

Answer 6

dnolen发表的评论：

朱利恩，你不需要测试这个补丁，我在尝试寻求外部帮助：）

Answer 7

评论者：benedek.fazekas

对此有何进展？只是在查看https://github.com/clojure-emacs/refactor-nrepl/issues/195时发现了这个问题尚未解决。
谢谢

Answer 8

dnolen发表的评论：

我们只需要更多的反馈。试用这个补丁，并告诉我们源映射是否仍然有效。

Answer 9

参考： https://clojure.atlassian.net/browse/CLJS-2051（julienfantin 报告）