On 3 November 2015 at 00:29, Matt Turner <matts...@gmail.com> wrote: > Generated by > > sed -i -e 's/\.bits\././g' *.c *.h *.cpp > sed -i -e 's/dw1\.//g' *.c *.h *.cpp > > and then reverting changes to comments in gen7_blorp.cpp and > brw_fs_generator.cpp. > > There wasn't any utility offered by forcing the programmer to list these > to access their fields. Removing them will reduce churn in future > commits. > > This is C11 (and gcc has apparently supported it for sometime > "compatibility with other compilers") > > See https://gcc.gnu.org/onlinedocs/gcc/Unnamed-Fields.html FYI on old versions of GCC where this extension is not enabled by default, we flip in on. We're using -std=gnu99, although perhaps we should use -fms-extensions - not a big deal either way.
Btw I had an almost identical patch locally :-) -Emil _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev