On Fri, Oct 23, 2015 at 12:22 AM, Nanley Chery <nanleych...@gmail.com> wrote: > > > On Thu, Oct 22, 2015 at 3:06 AM, Emil Velikov <emil.l.veli...@gmail.com> > wrote: >> >> On 20 October 2015 at 16:43, Nanley Chery <nanleych...@gmail.com> wrote: >> > On Tue, Oct 20, 2015 at 8:16 AM, Marek Olšák <mar...@gmail.com> wrote: >> >> >> >> Also, the FIXME comment should be on its own line. >> >> >> > >> > I moved it aside to make editing the table easier. However, since the >> > formatting of the >> > table is unlikely to change much after this series, I agree that I >> > should >> > move it back to >> > its original position. >> > >> Actually the designated initalisers should be fine in core mesa. If in >> doubt wrt MSVC compat, just grep MSVC.*COMPAT through whole of mesa. >> The 2013 version adds support for this feature. >> Alternatively feel free to ask Brian/Jose, as they have a fair bit of >> experience in the area. >> > > That's good news. Thanks for the information. > >> >> That aside, I'm in favour of keeping the comments as is. The editing >> comment does not apply imho. >> > > Since, there isn't a unanimous opinion on this, I'll leave the FIXME > to save some rebasing time. Are you referring to my git comment? > Or to my previous reply about why I moved the FIXME?
Yeah, feel free to fix the FIXME later in the series. Marek _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev