On 04.01.2011 16:20, Philip Martin wrote: > danie...@apache.org writes: > >> Author: danielsh >> Date: Thu Dec 30 20:13:50 2010 >> New Revision: 1053996 >> >> URL: http://svn.apache.org/viewvc?rev=1053996&view=rev >> Log: >> Once and for all, name all our anonymous struct/enum typedefs. >> >> Follows up on r1040058, and with thanks to Danny Trebbien. >> >> * everywhere: >> Change 'typedef struct {} foo_t;' to 'typedef struct foo_t {} foo_t;'. > If there is a consensus that this is a good thing to do then we should > add it to our coding guidelines
I frankly see no reason at all to do this "everywhere", it's just unnecessary code churn. The struct tags are only useful if the structure references itself (via pointers), and our practice for such cases was to declare the typedef first, and the struct itself below it, and use the typedef name in the struct definition. So unless someone can explain the reasoning why all these anonymous structs and enums etc. should have names -- on technical grounds, not some stylistic hand-waving -- then please revert. -- Brane
