<div dir="ltr"><div>Thank you for your feedback.<br><br></div>I will modify all patches to have short description below email title. I will add "glx:" to the GLX patches.<br><br></div><div class="gmail_extra"><br>
<br><div class="gmail_quote">2013/7/16 Kenneth Graunke <span dir="ltr"><<a href="mailto:kenneth@whitecape.org" target="_blank">kenneth@whitecape.org</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On 07/15/2013 01:41 PM, Ian Romanick wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 07/15/2013 07:28 AM, Tomasz Lis wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
The change is to correctly handle the value of renderType in both<br>
fbconfig and context. Uses of renderType are commented, where needed,<br>
to avoid confusion between the two types. The fact that float configs<br>
can be only used for pbuffers is correctly addressed.<br>
</blockquote>
<br>
These commit messages are not so good.  The first line should be a short<br>
description of the change.  The remainder should be additional<br>
information (details or supporting evidence) of the change.  For this<br>
patch, I would suggest:<br>
<br>
Correctly handle the value of renderType in both fbconfig and context.<br>
<br>
Uses of renderType are commented, where needed, to avoid confusion<br>
between the two types. The fact that float configs can be only used<br>
for pbuffers is correctly addressed.<br>
</blockquote>
<br></div>
Also, commit messages for the glx code should start with the "glx:" prefix, i.e.<br>
<br>
glx: Correctly handle renderType in both fbconfig and context.<br>
<br>
In general, to figure out the prefix for a particular directory, run git log on a file you edited and see what other people have picked.<br>
<br>
Patches 1-3 look reasonable to me; I didn't get a chance to look at the rest of the series, but I trust Ian's review.<br>
<br>
--Ken<br>
</blockquote></div><br></div>