>>>>>> "Jenny" == Jennifer Pioch <piochjennifer at googlemail.com> writes: > >>> http://cr.opensolaris.org/~chin/webrev-nonast.dec4-vs-dec6-diffs/ > >Jenny> Why do we have to review this? The tree won't build without the >Jenny> one-line patch. > >In this case the fix is obvious. But sometimes a one-line "obvious" fix >might introduce other problems. So to be safe, everything gets >reviewed.
I've a friend who is a professor in software engineering; he claims that 30% or so of single line changes have a bug. But this drops to 2% when there's one reviewer. Casper
