添加行号和列号到分析器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

以下是一个补丁，添加行号和列号并尝试从env中获取这些数据的标准方式。

Answer 2

评论者：anmonteiro

我认为该问题已在CLJS-1461中解决，该问题的目标是实现与tools.analyzer AST的完全兼容。

Answer 3

评论者：dnolen

CLJS-1461是一个大的项目，我们不确定它需要多长时间。在此之前，如果我们能够获得补丁，我们不介意在该方向上采取逐步措施。

Answer 4

评论者：dnolen

这个补丁看起来不错，但如果能够得到一些反馈，确实说明源映射没有受到影响会更好。

Answer 5

评论者：julienfantin

不幸的是，我们的主要项目依赖于较旧的ClojureScript版本，所以我无法在我的主要代码库上测试这个补丁。有没有特别需要注意的事情？

Answer 6

评论者：dnolen

Julien，不需要你测试这个补丁，这里正在尝试获取一些外部帮助 :)

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 提出）