<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Nov 6, 2013 at 7:21 PM, Day, Phil <span dir="ltr"><<a href="mailto:philip.day@hp.com" target="_blank">philip.day@hp.com</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">><br>
> Leaving a mark.<br>
> ===============<br>
><br>
> You review a change and see that it is mostly fine, but you feel that since you<br>
> did so much work reviewing it, you should at least find<br>
> *something* wrong. So you find some nitpick and -1 the change just so that<br>
> they know you reviewed it.<br>
><br>
</div><div class="im">> This is quite obvious. Just don't do it. It's OK to spend an hour reviewing<br>
> something, and then leaving no comments on it, because it's simply fine, or<br>
> because we had to means to test someting (see the first pattern).<br>
><br>
><br>
<br>
</div>Another one that comes into this category is adding a -1 which just says "I agree with<br>
the other -1's in here".   If you have some additional perspective and can expand on<br>
it then that's fine - otherwise it adds very little and is just review count chasing.<br>
<br>
It's an unfortunate consequence of counting and publishing review stats that having<br>
such a measure will inevitable also drive behavour.</blockquote><div><br></div><div>I agree that having no comment with a +1 is OK.  If I think the code looks good I'll +1 it.  If I think the code could be better I'll -1 it and leave comments.</div>
<div><br></div><div>I don't think that leaving a -1 with a 'what they said' comment is a bad thing.  As the developer writing the patch it's helpful to know there is a consensus and it's not just one person requesting a change.</div>
</div><div><br></div>-- <br>David<br>blog: <a href="http://www.traceback.org" target="_blank">http://www.traceback.org</a><br>twitter: <a href="http://twitter.com/dstanek" target="_blank">http://twitter.com/dstanek</a><div>
www: <a href="http://dstanek.com" target="_blank">http://dstanek.com</a></div>
</div></div>