请在2024年Clojure状态调查!中分享您的想法。

欢迎!请参阅关于页面以获取更多有关如何使用此信息的信息。

0
Clojure

首先,core.reducers需要使用clojure.walk:as walk,但没有使用该别名。
其次,在reducer函数中,coll-reduce具有两个arity的实现使用'this',而该文件中类似实现使用'coll'。根据我的了解,使用'coll'没有区别(所有测试通过,性能没有变化)并且更符合其他代码。

这两项似乎足够小,可以归入一个清理案例中。

5 个答案

0

评论者:wagjo

你的补丁是错误的。如果你想用coll替换它,你必须也调用xf f1。

`(clojure.core.protocols/coll-reduce this f1 (f1))`

变为

`(clojure.core.protocols/coll-reduce coll (xf f1) (f1))`

0

评论者:ska2342

惊异,测试套件并没有检测到这一点。

0

评论者:jafingerhut

添加一个能检测到这个问题的测试是否简单直接?

0

评论者:ska2342

我很乐意调查这个问题。虽然可能需要几天时间,但我会找到时间的。

0
参考:[https://clojure.atlassian.net/browse/CLJ-1151](https://clojure.atlassian.net/browse/CLJ-1151)(由ska2342报告)
...