<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Nov 18, 2015 at 12:33 PM, Emil Velikov <span dir="ltr"><<a href="mailto:emil.l.velikov@gmail.com" target="_blank">emil.l.velikov@gmail.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">Hi Nanley,<br>
<span><br>
On 18 November 2015 at 20:29, Nanley Chery <<a href="mailto:nanleychery@gmail.com" target="_blank">nanleychery@gmail.com</a>> wrote:<br>
> From: Nanley Chery <<a href="mailto:nanley.g.chery@intel.com" target="_blank">nanley.g.chery@intel.com</a>><br>
><br>
> Help developers understand the table's organization.<br>
><br>
> Suggested-by: Jason Ekstrand <<a href="mailto:jason@jlekstrand.net" target="_blank">jason@jlekstrand.net</a>><br>
> Signed-off-by: Nanley Chery <<a href="mailto:nanley.g.chery@intel.com" target="_blank">nanley.g.chery@intel.com</a>><br>
</span>Can you bring back the comments that were in here, please. They might<br>
have cause issues with sed prior but things should be fine now. Also<br>
move the existing FIXME as originally, as mentioned by Marek and<br>
myself.<br>
<br></blockquote><div><br></div><div>There's one more organizational change that's possible - sorting the table alphabetically. The original comments would probably be unnecessary if we choose to perform that change.<br><br>When first adding ASTC to the table, I did not place the extensions in the right rows partly because the existing alphabetical sorting was inconsistent. By completely sorting the table it becomes a easier to determine where to place new extensions.<br><br>If the sorted order is checked with a gtest, it  would then be possible to know if the table is sorted and, if desired, perform a bsearch on the table. I'll send out a patch to test interest on this. I have a wip branch that does what I've described (minus the bsearch) : <a href="http://cgit.freedesktop.org/~nchery/mesa/log/?h=ext_sort" target="_blank">http://cgit.freedesktop.org/~nchery/mesa/log/?h=ext_sort</a></div><div><br></div><div>Wrt to the FIXME, I was confused by your original comment and thought that you actually wanted the FIXME on the same line as the extension. Hence, my comment about there being no unanimous opinion on the change. I wasn't sure what action to take in the midst of the perceived disagreement, but now that I understand your comment, I'll send out the patch for it.<br><br></div><div>Regards,<br></div><div>Nanley<br></div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Thanks<br>
<span><font color="#888888">Emil<br>
</font></span></blockquote></div><br></div></div>