On 2015-02-15 12:31:10 -0500, Tom Lane wrote: > Andres Freund <and...@2ndquadrant.com> writes: > > I was thinking of adding BKI_FORCENOTNULL which would be > > specified on the attributes you want it. The FORCE in there representing > > that the default choice is overwritten. > > Where are you thinking of sticking that exactly, and will pgindent > do something sane with it?
Hm, I was thinking about /* extversion should never be null, but the others can be. */ text extversion PG_FORCENOTNULL; /* extension version name */ but pgindent then removes some of the space between text and extversion, making it text extversion PG_FORCENOTNULL; /* extension version name */ an alternative where it doesn't do that is text PG_FORCENOTNULL(extversion); /* extension version name */ Not sure what's the best way here. Greetings, Andres Freund -- Andres Freund 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