2024年Clojure调查!中分享您的想法。

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

0
Clojure

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

这两件事足够小,可以合并成一个清理案例。

5个答案

0
by

由:wagjo发表评论

您的补丁是错误的。如果您想用coll替换它,则必须同时调用xf on f1。

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

变为

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

0
by

由:ska2342发表评论

令人惊讶的是,测试套件没有检测到这一点。

0
by

由:jafingerhut发表评论

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

0

由:ska2342发表评论

我很乐意调查这个问题。不过,在找到时间之前,可能需要几天时间。

0
参考资料:https://clojure.atlassian.net/browse/CLJ-1151 (由 ska2342 汇报)
...