使用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 已添加到补丁提案

Answer 10

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