<br><br><div class="gmail_quote">On Fri, Sep 10, 2010 at 11:50 AM, Denys Dmytriyenko <span dir="ltr"><<a href="mailto:denis@denix.org">denis@denix.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im">>> I'd agree with this quite strongly. I'm generally pretty careful to let<br>
>> others review the changes from my coworkers, to avoid any bias.<br>
<br>
</div>Not to point any fingers, but mishaps happen and numerous reverts would be a<br>
living proof of that in the repository... :) Nobody's perfect.</blockquote></div><br>Mishaps have nothing to do with this, nor does being perfect, or reverts. You're missing the point entirely here. The point is, if the only people who acked a patch are from the same company as the person who wrote it, they can't be considered to have been entirely unbiased in their review of it.<br>
-- <br>Christopher Larson<br>clarson at kergoth dot com<br>Founder - BitBake, OpenEmbedded, OpenZaurus<br>Maintainer - Tslib<br>Senior Software Engineer, Mentor Graphics<br>