使用 pr-str 而不是 str 打印对象时改进错误信息

Question

ClojureScript 的许多错误信息都包含类似这样的无效参数

(throw (js/Error. (str "不支持的名称: " x)))

如果 {{x}} 是 nil，那么产生的错误信息为 {{"不支持的名称: "}}，这在调试时有点令人困惑。如果 {{x}} 被包装在 {{pr-str}} 里面，那么错误信息就会更加易于理解：{{"不支持的名称: nil"}}。

如果对此感兴趣，我可以准备一个补丁，用来使用 pr-str 包装这些类型的错误。

Answer 1

评论者：dnolen

去做吧

Answer 2

评论者：mfikes

我也对 CLJS-2089 进行了些思考。看来这个做法在一般情况下是正确的。

Answer 3

评论者：mfikes

补丁不再适用。

Answer 4

由 slipset 发布的评论：

希望现在已经适用了

Answer 5

评论者：mfikes

0001-CLJS-1908-Wrap-vars-in-throw-message-with-pr-str.patch 已不适用

Answer 6

由 slipset 发布的评论：

0001-CLJS-1908-Wrap-vars-in-throw-message-with-pr-str-2.patch 已更新，应该适用

Answer 7

评论者：mfikes

0001-CLJS-1908-Wrap-vars-in-throw-message-with-pr-str-2.patch LGTM

Answer 8

评论者：mfikes

0001-CLJS-1908-Wrap-vars-in-throw-message-with-pr-str-2.patch 通过 CI 和 Canary 测试 (/)

Answer 9

评论者：mfikes

0001-CLJS-1908-Wrap-vars-in-throw-message-with-pr-str-2.patch 已添加到 Patch Tender (i)

Answer 10

参考：[https://clojure.atlassian.net/browse/CLJS-1908](https://clojure.atlassian.net/browse/CLJS-1908) (由 [email protected] 报告)