将行尾和列尾添加到分析器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 报告）