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

Reply via email to