<div class="gmail_quote">On Mon, Mar 28, 2011 at 10:06 AM, David Levin <span dir="ltr"><<a href="mailto:levin@chromium.org">levin@chromium.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

Here's a change that I felt worth getting someone to glance at but didn't feel worth the overhead of a bug:<div>† †<a href="http://trac.webkit.org/changeset/81305" target="_blank">http://trac.webkit.org/changeset/81305</a><br>

<br>

</div><div>Since I was gardener and this was affecting the bots, it was a timely situation. (Sometimes getting in your fix right before another break comes in is important in these cases.)</div><div><br></div><div>dave</div>



<div><br></div><div>PS Dmitry found a flaw in my original change log text -- due to my haste, I originally had put in the wrong valgrind error.</div></blockquote><div><br></div><div>This seems like the kind of thing that'd be nice to put in the bug after the fact, no? †:-)</div>

<div><br></div><div>If the issue is simply one of overhead, then we should allow committers to omit change logs when they're not necessary as well. †For example, no matter how one feels about ChangeLogs, I don't think it's possible to claim it's useful here: <a href="http://trac.webkit.org/changeset/81864">http://trac.webkit.org/changeset/81864</a></div>

<div><br></div><div>J</div><div>†</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div><div class="gmail_quote"><div><div></div><div class="h5">On Mon, Mar 28, 2011 at 9:58 AM, Jeremy Orlow <span dir="ltr"><<a href="mailto:jorlow@chromium.org" target="_blank">jorlow@chromium.org</a>></span> wrote:<br>



</div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div></div><div class="h5"><p>Can you please explain why?† Its very little overhead and is useful for tracking regressions and such.</p>


<p>J</p><div><div></div><div>
<div class="gmail_quote">On Mar 28, 2011 9:52 AM, "Darin Adler" <<a href="mailto:darin@apple.com" target="_blank">darin@apple.com</a>> wrote:<br type="attribution">> On Mar 27, 2011, at 1:31 AM, Jeremy Orlow wrote:<br>




> <br>>> I'd even go a bit further and say that if something is worth a review (even if it's over the shoulder), it's worth a bug + a bug number.<br>> <br>> This is where I do not agree. Review is a requirement, but I donít think <a href="http://bugs.webkit.org" target="_blank">bugs.webkit.org</a> should be.<br>




> <br>>     -- Darin<br>> <br></div>
</div></div><br></div></div><div class="im">_______________________________________________<br>
webkit-dev mailing list<br>
<a href="mailto:webkit-dev@lists.webkit.org" target="_blank">webkit-dev@lists.webkit.org</a><br>
<a href="http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev" target="_blank">http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev</a><br>
<br></div></blockquote></div><br></div>
</blockquote></div><br>