欢迎!请参阅关于页面以获取有关如何使用本网站的更多信息。
首先,core.reducers需要clojure.walk:as walk,但没有使用别名。其次,函数reducer中的coll-reduce在两个系统实现中使用了'this',而该文件中的类似实现使用了'coll'。据我所知,使用'coll'没有区别(所有测试通过,性能没有变化)而且与代码的其他部分更一致。
这两件事足够小,可以合并成一个清理案例。
由:wagjo发表评论
您的补丁是错误的。如果您想用coll替换它,则必须同时调用xf on f1。
(clojure.core.protocols/coll-reduce this f1 (f1))
变为
(clojure.core.protocols/coll-reduce coll (xf f1) (f1))
由:ska2342发表评论
令人惊讶的是,测试套件没有检测到这一点。
由:jafingerhut发表评论
添加一个能够检测到这一点的测试是否简单直接?
我很乐意调查这个问题。不过,在找到时间之前,可能需要几天时间。