Jamis

Results 3 comments of Jamis

Just in case anybody comes across the same problem, I fix it by removing the if clause in `tdump()` and `sqltdump()` at file `undark.c`. ```C - if (isprint(*p)) fprintf(stdout,"%c", *p);...

这不就相当于是把以前checkout的时间均摊到每次订阅脚本了吗?

> 如果你每次更新仓库都把全部脚本订阅一遍,是的 我觉得从用户体验的角度来讲这个改动是合理的。 我刚刚把代码拿到本地编译过后对比了一下。如果在clone仓库的时候checkout,在我的环境里大约需要7~8秒,用户需要眼睁睁的等这8秒。相比之下如果在订阅的时候checkout(往往用户只会订阅一个subdirectory而非整个repo),一个典型的checkout js脚本或者路径追踪子文件夹的时间可能就是1秒甚至更少,这1秒时间在用户用鼠标点点点的过程中变得不引人注意,用户几乎注意不到它的存在。我想这是给用户带来的最明显的改善。 如果从工程学的角度考虑这个问题,这个体验上的改善,引入了更复杂的代码,与之前相比(可能)稍微降低了可维护性;以及打破了一个原本成立的假设即本地存在一个完整的可访问的仓库,这会不会成为其他功能设计和实现的一个障碍。这本质上是一个trade-off,那么这个trade-off是不是划算的,就要结合实际情况仔细考量了。