<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Nov 11, 2015 at 2:07 PM, Chad Versace <span dir="ltr"><<a href="mailto:chad.versace@intel.com" target="_blank">chad.versace@intel.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="">On Fri 30 Oct 2015, Nanley Chery wrote:<br>
> From: Nanley Chery <<a href="mailto:nanley.g.chery@intel.com">nanley.g.chery@intel.com</a>><br>
><br>
> Simplify future updates to the extension struct array by removing<br>
> the sentinel.<br>
><br>
> Signed-off-by: Nanley Chery <<a href="mailto:nanley.g.chery@intel.com">nanley.g.chery@intel.com</a>><br>
<br>
</span>Patch 1 is<br>
Reviewed-by: Chad Versace <<a href="mailto:chad.versace@intel.com">chad.versace@intel.com</a>><br>
<br></blockquote><div><br></div><div>Thanks.<br> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Since your series is long and risks suffering from rebase conflicts,<br>
I suggest you push the little cleanup patches (like this one) as soon as<br>
you're satisfied with the review.<br>
<br></blockquote><div><br></div><div>Sounds good.<br> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Also, could you push a branch for v2 of your series? And put v2 in its<br>
name? That would help me review it since it no longer applies cleanly to<br>
master.<br></blockquote><div><br></div><div>Sure, the branch is located here:  <a href="http://cgit.freedesktop.org/~nchery/mesa/log/?h=ext_safev2">http://cgit.freedesktop.org/~nchery/mesa/log/?h=ext_safev2</a><br></div></div><br></div></div>