评论由:razum2um_ 发布
嗨 Jérémie,Alex 和 Luc,
我是aprint的作者,很高兴你们喜欢我的库 :)
以下是我对此的看法
1) 我真的很喜欢把它推入tools.trace
2) 当前所有clojure contrib-libs都与lang具有相同的补丁接受策略,我已经对此表示过疑问
https://github.com/clojure/clojure/pull/17#issuecomment-53628365 尝试理解,如果一个库依赖于另一个库,则最后一个库必须与原始库具有相同的策略,但这并不是这种情况,因为我都很难停止接受pull请求。
3) 如果你们无法改变这项政策,我认为内联是一个最佳选择。
关于“什么会被内联?”这个问题的一些观点
4) 我有点惊讶,人们没有像我和这些一样想要与 repl 集成的意图
https://github.com/greglook/whidbey, 不喜欢颜色,他们只是喜欢布局。所以着色可以被扔掉,我们消除对 clansi 的依赖。或者我可以内联 clansi(MIT 许可证),记录一个 :^dynamic 来开启颜色(默认关闭)
5)精简布局的代码依赖于 jline,它肯定不会进到里面 :) 因此,它将保持 pprint/*print-right-margin* 的默认值,默认为 72,但在我个人看来不是最佳选择,但让它保持原样。(或者你还在非常老的终端上运行 Clojure?)
6)我认为引入一个新的 :^dynamic 来表示它是没有意义的,我建议依赖于 pprint/*print-pretty*,如果是 true - 则使用精简布局而不是 pprint 的默认布局
因此
- 我可以填写并签署一份 CA
- 为你准备一个依赖的免费补丁,包括只有精简布局(如果你喜欢,还可以包括颜色),由 pprint/*print-pretty* 和 pprint/*print-right-margin* 控制