On Sun, Nov 13, 2016 at 9:48 AM, Andrew Dunstan <and...@dunslane.net> wrote: > I'm not convinced the line prefix part is necessary, though. What I'm > thinking of is something like this: > > PROCDATA( oid=1242 name=boolin isstrict=t volatile=i parallel=s nargs=1 > rettype=bool argtypes="cstring" src=boolin );
I liked Tom's format a lot better. If we put this in a separate file rather than in the header, which I favor, the PROCDATA stuff is just noise. On the other hand, having the name as the first thing on the line seems *excellent* for readability. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers