Great article on code reviews !

[Simo]

On Mon, 2014-06-02 at 10:17 -0700, Jeremy Allison wrote:
> Hurrah, FreeBSD is now joining the club :-).
> 
> http://julipedia.meroh.net/2014/05/code-review-culture-meets-freebsd.html
> 
> Some really great points in the article:
> 
> "But it's hard to step aside from your code once it is
> building and running and all that is left for the world to
> witness your creation is to "commit" it to the tree. But you
> know what? In the vast majority of cases, nobody cares if you
> commit your change at that exact moment, or tomorrow, or the
> week after. It may be hard to hear this, but that pre-commit
> "high" that rushes you to submit your code is usually
> counterproductive and dangerous. (Been there, done that, and
> ended up having to fix the commit soon after for stupid
> reasons... countless times... and that is shameful.)"
> 
> I am soooo guilty of the above :-).

Haha, it always happen w/o code review.
I am guilty of that too, what could possibly go wrong with my
magnificent 'fix', commit ... BOOM!

Simo.