On 27/11/16 21:47, Andres Freund wrote: > Hi, > >>> +typedef struct SlabBlockData *SlabBlock; /* forward reference */ >>> +typedef struct SlabChunkData *SlabChunk; >>> >>> Can we please not continue hiding pointers behind typedefs? It's a bad >>> pattern, and that it's fairly widely used isn't a good excuse to >>> introduce further usages of it. >>> >> >> Why is it a bad pattern? > > It hides what is passed by reference, and what by value, and it makes it > a guessing game whether you need -> or . since you don't know whether > it's a pointer or the actual object. All to save a * in parameter and > variable declaration?... >
FWIW I don't like that pattern either although it's used in many parts of our code-base. -- Petr Jelinek http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training & Services -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers