From: | Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Larry Rosenman <ler(at)lerctr(dot)org>, PostgreSQL Hackers List <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: coding style guidelines? |
Date: | 2000-11-17 02:06:36 |
Message-ID: | 200011170206.VAA11826@candle.pha.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
> Larry Rosenman <ler(at)lerctr(dot)org> writes:
> > Is there any guidelines on the formatting of the C code in
> > PG? As I was working on guc-file.l yesterday, I noticed
> > some things with LONG lines (I broke some of them up).
> > I was wondering if there were formal standards?
>
> Brace layout, comment layout and indentation are all brought into line
> by pg_indent, which Bruce runs at least once per release cycle.
> However, I don't think pg_indent will consider breaking non-comment lines
> into multiple lines, so it's up to the code author to be reasonable in
> that area.
It does wrap >80 lines.
>
> My own practice is to try to make the code look nice in an 80-column
> window.
>
> BTW, if you are writing a comment that you don't want to have
> reformatted by pg_indent's rather braindead reformatter, protect it
> with some dashes:
>
> /*----------
> * This text will not get reformatted.
> *----------
> */
>
>
> > Also, do we care about extraneous #include's?
>
> Not very much. You have to be particularly cautious about removing
> system-header #includes, since what looks redundant on your platform
> may not be redundant for other platforms. I think Bruce has a tool
> to look for unnecessary includes of our own header files, but it
> doesn't risk trying to remove system headers.
Yes, it does not touch system includes.
--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 853-3000
+ If your life is a hard drive, | 830 Blythe Avenue
+ Christ can be your backup. | Drexel Hill, Pennsylvania 19026
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2000-11-17 02:08:20 | Re: [rfc] new CREATE FUNCTION (and more) |
Previous Message | Philip Warner | 2000-11-17 02:00:20 | Re: [rfc] new CREATE FUNCTION (and more) |