From: | Richard Guo <guofenglinux(at)gmail(dot)com> |
---|---|
To: | Japin Li <japinli(at)hotmail(dot)com> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, John Naylor <john(dot)naylor(at)enterprisedb(dot)com>, "pgsql-hackers(at)lists(dot)postgresql(dot)org" <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: Typo in pg_db_role_setting.h |
Date: | 2022-08-02 03:06:01 |
Message-ID: | CAMbWs489+u6P_9qMjABsse0dNNBr36MA1SX5Ss7yZ7TD86mfKQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Mon, Aug 1, 2022 at 10:42 PM Japin Li <japinli(at)hotmail(dot)com> wrote:
>
> On Mon, 01 Aug 2022 at 22:16, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> > John Naylor <john(dot)naylor(at)enterprisedb(dot)com> writes:
> >> You are correct, but I wonder if it'd be better to just drop the comment
> >> entirely. I checked a couple other random headers with function
> >> declarations and they didn't have such a comment, and it's kind of
> obvious
> >> what they're for.
> >
> > Some places have these, some don't. It's probably more useful where
> > a header foo.h is declaring functions that aren't in the obviously
> > corresponding foo.c file, or live in multiple files. In this case
> > I agree it's not adding much.
> >
>
> Attached patch to remove this comment. Please take a look.
I'm not sure that we should remove such comments. And a rough search
shows that there are much more places with this kind of comments, such
as below:
nbtxlog transam readfuncs walreceiver buffile bufmgr fd latch pmsignal
procsignal sinvaladt logtape rangetypes
Thanks
Richard
From | Date | Subject | |
---|---|---|---|
Next Message | houzj.fnst@fujitsu.com | 2022-08-02 03:20:55 | RE: Support logical replication of DDLs |
Previous Message | Amit Langote | 2022-08-02 03:05:55 | Re: PG 15 (and to a smaller degree 14) regression due to ExprEvalStep size |