[asterisk-dev] [Code Review]: Preserve DTMF length in main/features.c

Olle E. Johansson oej at edvina.net
Wed Nov 2 07:54:25 CDT 2011

25 okt 2011 kl. 00:10 skrev Terry Wilson:

> I'm perfectly ok with committer==reviewer. If we can trust someone to commit code to the repo, surely we can trust them to look at someone else's code to review it before committing it. Commit access without review access is pretty worthless. I see commit access as a way to reduce bottleneck problems of getting code into the tree. A monkey could almost be trained to commit code if it didn't have any right to decide whether the code should be committed at all.
> All non-trivial patches should be reviewed no matter who the developer is (Digium employee, community committer, or random person submitting a patch). In my opinion, any developer who could commit a patch should be able to review someone else's patch and commit it. I think guilt and public shame for putting something through that is broken should be enough of a deterrent to sloppy reviewing. Nobody's perfect, and its all a learning process.

+100 !

Thanks, Terry.


More information about the asterisk-dev mailing list