在使用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] 报告）