谈谈 code review？

• 优点：保证代码质量，保证工程代码风格一致。
• 缺点：过度 review 导致项目效率低下，遇到不专业的 review 只想喷人。

最近就遇到个烦人的，改来改去，就写个模块的接口，反反复复改，想到喊我改到哪。 比如一个函数：

int xxxx_init(CTX * ctx,A *a){ if (xxx){ return rc; } xxxx // 实际逻辑代码段 return rc; } 

一般来说，项目风格就是这样的，先检查 ctx 啥的，然后如果实际逻辑，最后返回调用状态。 这个方法能给我提 3 个 comments 。

1. it is simple memcpy... do we really need all the checks below?
2. i guess this function should return，remove A * a 。
3. is it documented on API level? (实际逻辑代码段上一顿 bb)

看到这个我都不想回也不再改了，想问你调用 CTX 是有状态的，你调用函数前不需要 check ？其二本身方法逻辑就是大页来分配 A，名字也叫 xxx_init，我还返回个毛球啊。

搞得真想跑路。