<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Oct 22, 2015 at 3:06 AM, 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>On 20 October 2015 at 16:43, Nanley Chery <<a href="mailto:nanleychery@gmail.com" target="_blank">nanleychery@gmail.com</a>> wrote:<br>
> On Tue, Oct 20, 2015 at 8:16 AM, Marek Olšák <<a href="mailto:maraeo@gmail.com" target="_blank">maraeo@gmail.com</a>> wrote:<br>
>><br>
>> Also, the FIXME comment should be on its own line.<br>
>><br>
><br>
> I moved it aside to make editing the table easier. However, since the<br>
> formatting of the<br>
> table is unlikely to change much after this series, I agree that I should<br>
> move it back to<br>
> its original position.<br>
><br>
</span>Actually the designated initalisers should be fine in core mesa. If in<br>
doubt wrt MSVC compat, just grep MSVC.*COMPAT through whole of mesa.<br>
The 2013 version adds support for this feature.<br>
Alternatively feel free to ask Brian/Jose, as they have a fair bit of<br>
experience in the area.<br>
<br></blockquote><div><br></div><div>That's good news. Thanks for the information.<br> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
That aside, I'm in favour of keeping the comments as is. The editing<br>
comment does not apply imho.<br>
<span><font color="#888888"><br></font></span></blockquote><div><br></div><div>Since, there isn't a unanimous opinion on this, I'll leave the FIXME<br>to save some rebasing time. Are you referring to my git comment?<br></div><div>Or to my previous reply about why I moved the FIXME?<br></div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span><font color="#888888">
-Emil<br>
</font></span></blockquote></div><br></div></div>