周四下午，收到同事的一封mail，他告诉我他的业务代码中使用的一个库接口的行为与预期不同，并在mail中给出了测试代码和测试结果。而这个接口是之前由我封装实现的。 

这个库仅仅是对libevent做了一层薄薄的封装，目的是使其接口的使用方式符合部门的一贯风格。虽说封装简单，但单元测试也是一应俱全，不敢马虎，必要的地方mock也一并上阵，总体来说我个人还是比较满意的。

不过还是出现了问题，问题出在libevent提供的timer的行为上。这是我们第一次使用libevent，我基于以前对timer的事件处理机制的理解作出了错误的假定：libevent提供的timer是PERSIST的。同事的测试结果却表明这个timer设定是一次性的：只运行一次便失效了，除非在timer的callback handler中重新添加timer事件。

《程序员修炼之道》中在介绍如何深思熟虑而不是靠巧合编程时说过：要测试你的假定。我恰恰是在这块儿犯了错误。尚没有明确libevent的行为就做出了假定而且没有对这个假定做细致测试，最终导致了这个问题的发生。

这个问题最终是这么解决的：了解了一下，libevent 1.x.x版本确实不支持persist timer事件处理，但是在libevent 2.x.x版本中，作者增加了对persist timer的支持。我们原本使用的是1.4.10的libevent，为了支持persist timer，我将库升级到2.0.10(已经是stable版了)。不过仅仅是升级库版本还是不行的。我们不能直接使用libevent提供的evtimer_set接口，因为使用该接口后，我们得到的依旧是一次性的timer。这里需要用event_set(e, -1, EV_PERSIST, eh, arg)替代evtimer_set(e, eh, arg)，这样设置后得到的timer才是persist的。 

很多公司的过程中都有阶段性统计新增或修改的有效代码行数这一环节，这里先不论统计出的结果用于做什么，就统计本身而言，常常存在诸多问题，比如统计过程耗时且繁琐、统计结果中估算成分较大，不精确等。这些问题以前也一直困扰着我们，并且长时间没有想出很好的解决办法。

今天脑子里突然冒出一个想法：能否根据svn diff得到的结果分析出来有效代码量呢？ svn diff的结果一般是这样的，分为几类：

纯新增代码，如：
+void foo() {
+    … …
+}

纯删除代码，如：
-void foo() {
-    … …
-}

修改的代码，如：
-void foo(void);
+void foo(int);

我们所要统计的所谓有效代码更多是指纯新增的代码和修改的代码，纯删除的代码可忽略不计。这样一来实际有效代码行数 = 纯新增代码行数 + 修改代码行数；而修改的代码在svn diff结果中体现为一减一加，实际修改行数是等于其+的行数的。也就是说有效代码行数就是svn diff结果中所有前缀为+的行的行数。svn diff输出格式相对规整，通过解析得到这个行数并非难事。最简单的方法就是使用Shell脚本了。

脚本全部内容这里就不列出来了，这里可以下载。其核心代码只有以下两行：

svn diff -r$start_revision:$end_revision $target $USERNAME $PASSWD > $TEMPFILE
add_lines_count=`grep "^+" $TEMPFILE|grep -v "^+++"|sed 's/^.//'|sed '/^$/d'|wc -l`

首先我们使用svn diff命令将两个修订号之间的差异重定向到一个临时文件中，然后使用grep、sed和wc的组合完成行数的计算：其中首先过滤出以+开头的行，但去除其中+++开头的行，得到的是所有只以一个+开头的行。再利用set 's/^.//'删除每行行首的那个+，用set '/^$/d'删除所有空行，最后利用wc -l计算总行数。

也就是说通过上面脚本运行后得到的有效代码行数是不包括空行的，但是包含注释代码。

有了这个脚本，以后的版本有效代码量统计就相当精确了，而且也无需每个人都参与统计，大大减少了工作量，甚至可以将这个工作做成自动化完成。

现在的我痛恨一切效率低下的个人行为和过程活动！遇到问题坚决改善，绝不姑息^_^。