On Fri, 2008-05-02 at 23:22 +0200, Jan Urbański wrote: > While looking around vacuum.h (for my GSoC project) I found: > typedef struct VacAttrStats *VacAttrStatsP; > and then throughout the code sometimes VacAttrStats *foo is used and > sometimes VacAttrStatsP bar is used.
Personally I think we should be consistent about this and just use one form for a given type. (I don't really care which variant is preferred. Personally, I think that hiding pointer types inside a typedef is evil and wrong, but it is relatively common practice in the source...) -Neil -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers