<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Nov 18, 2015 at 4:17 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"><div class=""><div class="h5">On 18 November 2015 at 22:39, Nanley Chery <<a href="mailto:nanleychery@gmail.com">nanleychery@gmail.com</a>> wrote:<br>
> On Wed, Nov 18, 2015 at 12:33 PM, Emil Velikov <<a href="mailto:emil.l.velikov@gmail.com">emil.l.velikov@gmail.com</a>><br>
> wrote:<br>
>><br>
>> Hi Nanley,<br>
>><br>
>> On 18 November 2015 at 20:29, Nanley Chery <<a href="mailto:nanleychery@gmail.com">nanleychery@gmail.com</a>> wrote:<br>
>> > From: Nanley Chery <<a href="mailto:nanley.g.chery@intel.com">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">jason@jlekstrand.net</a>><br>
>> > Signed-off-by: Nanley Chery <<a href="mailto:nanley.g.chery@intel.com">nanley.g.chery@intel.com</a>><br>
>> 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>
><br>
> There's one more organizational change that's possible - sorting the table<br>
> alphabetically. The original comments would probably be unnecessary if we<br>
> choose to perform that change.<br>
><br>
> When first adding ASTC to the table, I did not place the extensions in the<br>
> right rows partly because the existing alphabetical sorting was<br>
> inconsistent. By completely sorting the table it becomes a easier to<br>
> determine where to place new extensions.<br>
><br>
> If the sorted order is checked with a gtest, it would then be possible to<br>
> know if the table is sorted and, if desired, perform a bsearch on the table.<br>
> I'll send out a patch to test interest on this. I have a wip branch that<br>
> does what I've described (minus the bsearch) :<br>
> <a href="http://cgit.freedesktop.org/~nchery/mesa/log/?h=ext_sort" rel="noreferrer" target="_blank">http://cgit.freedesktop.org/~nchery/mesa/log/?h=ext_sort</a><br>
><br>
</div></div>Sure, with sorting in place we can ignore the previous (original)<br>
comments. The bsearch will bring some benefit only as one overrides<br>
extensions or there is some other place that I'm missing ?<br>
<span class=""><br></span></blockquote><div><br></div><div>That's the only place where it is applicable unfortunately. For that reason, I did not implement the code for that functionality. Most of the benefit IMO comes from helping developers insert new extensions in the correct positions.<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"><span class="">
> Wrt to the FIXME, I was confused by your original comment and thought that<br>
> you actually wanted the FIXME on the same line as the extension. Hence, my<br>
> comment about there being no unanimous opinion on the change. I wasn't sure<br>
> what action to take in the midst of the perceived disagreement, but now that<br>
> I understand your comment, I'll send out the patch for it.<br>
><br>
</span>Hmm ? /me rereads his message<br>
<br></blockquote><div><br></div><div>Sorry for not providing the context: <a href="http://lists.freedesktop.org/archives/mesa-dev/2015-October/098182.html">http://lists.freedesktop.org/archives/mesa-dev/2015-October/098182.html</a><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Please don't be shy/afraid/too polite to call me and others out. If we<br>
have X "for" and Y "against" just draw the line and (explicitly) say<br>
which one you're going with. Otherwise one can never know if they came<br>
across (were understood) correctly.<br>
<br></blockquote><div><br></div><div>I originally did say that I would leave the FIXME, but I'll try to improve in this aspect of my communication.<br></div><div> <br></div><div>Regards,<br></div><div>Nanley<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">
Thanks<br>
<span class=""><font color="#888888">Emil<br>
</font></span></blockquote></div><br></div></div>