<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Apr 24, 2015 at 11:16 AM, Julien Danjou <span dir="ltr"><<a href="mailto:julien@danjou.info" target="_blank">julien@danjou.info</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Fri, Apr 24 2015, Joe Gordon wrote:<br>
<br>
> When I get a -1 on one of my patches with a question, I personally treat it<br>
> as a short coming of the commit message. To often in the past I have looked<br>
> at a file, and in trying to figure out why that line is there I do a git<br>
> blame only to see a useless commit message with me as the author.<br>
<br>
</span>That's a thing that I've been stated over and over again in this thread<br>
and actually paraphrased from the first paragraph on my original email.<br>
It'd be cool if we could stop restating the obvious over and over again.<br></blockquote><div><br></div><div><br></div><div>So you did, sorry.</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
Could someone give me an example of how we are supposed to improve the<br>
patch or commit message when one get a -1 with e.g. the question:<br>
  "Why do you use getattr(foo, "bar", None)?"<br>
<br></blockquote><div><br></div><div>By calling them out in the review or on irc, and explain to them when its appropriate to use a -1.   I don't think its safe to assume that a significant number of people who do these -1s are read every thread on the ML.</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
when the answer is "Well, otherwise it will raise an error and the code<br>
will fail" because the reviewer do not know how getattr() works.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Julien Danjou<br>
// Free Software hacker<br>
// <a href="http://julien.danjou.info" target="_blank">http://julien.danjou.info</a><br>
</font></span></blockquote></div><br></div></div>