On 4 October 2012 03:43, Oliver McFadden <span dir="ltr"><<a href="mailto:oliver.mcfadden@linux.intel.com" target="_blank">oliver.mcfadden@linux.intel.com</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On Wed, Oct 03, 2012 at 07:54:50PM -0700, Matt Turner wrote:<br>
> > mesa: setup_glsl_generate_mipmap(): meta program compile failed<br>
><br>
> This is not a commit title. After reading the title, I should have at<br>
> least some idea of what the commit does. Pasting the error tells me<br>
> nothing.<br>
<br>
</div>I'd include Mesa's error message describing _why_ the failure occurred,<br>
if it wasn't a bunch of corrupt bytes (another bug...)<br></blockquote><div><br></div><div>I'm not sure if this is what Matt was getting at, but the difficulty I had understanding your commit title was that I couldn't tell without reading the commit whether (a) you were fixing a problem where an error failed to be generated when it should, or (b) you were fixing a problem where an error was generated when it shouldn't.  Also, I couldn't tell at a glance which area of functionality you were affecting (mipmap generation) or what you were doing to affect the error condition (changing the choice of which shader to use).  Finally, the prefix "mesa:" misled me into thinking that this patch affected core mesa (meta ops are not part of core mesa--they're an optional component that is included by some but not all drivers).</div>
<div><br></div><div>FWIW, I usually try to make the commit title a verb phrase in present tense that describes what the code now does that it didn't previously do, and the circumstances under which the new behaviour manifests.  So for this patch (as you've revised it, with the code motion removed), I'd recommend titling it something like "meta: use proper shaders to generate mipmaps for GLES2".</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Oliver McFadden.<br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
mesa-dev mailing list<br>
<a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/mesa-dev" target="_blank">http://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
</div></div></blockquote></div><br>