<div dir="ltr">On 14 August 2013 21:07, Kenneth Graunke <span dir="ltr"><<a href="mailto:kenneth@whitecape.org" target="_blank">kenneth@whitecape.org</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">SURF_INDEX_DRAW is the identity function, and it's unlikely that it will<br>
change, but we may as well use it for documentation's sake.<br>
<br>
Signed-off-by: Kenneth Graunke <<a href="mailto:kenneth@whitecape.org" target="_blank">kenneth@whitecape.org</a>><br></blockquote><div><br></div><div>The comment above the declaration of SURF_INDEX_DRAW (brw_context.h) says that it's never used:<br>

<br> * Note that nothing actually uses the SURF_INDEX_DRAW macro, so it has to be<br> * the identity function or things will break.  We do want to keep draw buffers<br> * first so we can use headerless render target writes for RT 0.<br>

<br></div><div>Would you mind updating the comment in this commit?<br><br></div><div>With that fixed, this patch is:<br><br>Reviewed-by: Paul Berry <<a href="mailto:stereotype441@gmail.com">stereotype441@gmail.com</a>><br>
</div></div></div></div>