public inbox for [email protected]
help / color / mirror / Atom feedFrom: Álvaro Herrera <[email protected]>
To: Tom Lane <[email protected]>
Cc: solaimurugan vellaipandiyan <[email protected]>
Cc: Aleksander Alekseev <[email protected]>
Cc: Payal Singh <[email protected]>
Cc: [email protected]
Subject: Re: Review - Patch for pg_bsd_indent: improve formatting of multiline comments
Date: Mon, 11 May 2026 17:50:41 +0200
Message-ID: <[email protected]> (raw)
In-Reply-To: <[email protected]>
On 2026-May-10, Tom Lane wrote:
> Attached are a proposed v8 of the patch, plus two diff files showing
> the effects. v7-0001.diff.nocfbot is what the v7 patch does with
> today's HEAD (it's the same as before). The v8 patch makes all those
> changes and in addition makes the ones shown in v8-0001.diff.nocfbot.
> I think those are pretty much all improvements, except that it kind
> of messes up Martin Utesch's ASCII-art signatures in the geqo files.
> That's because there are some lines starting with '*' and some with
> '='. This is another place where I doubt it's worth the trouble to
> try to make pgindent handle the case nicely; I propose just manually
> adding leading '*'s to those comments before running pgindent.
Just passing by, but I think this comment should be handled manually as
well, as I doubt we want it to end up this way:
> diff --git a/src/backend/utils/adt/tsrank.c b/src/backend/utils/adt/tsrank.c
> index d35e5528d0a..6b4bf8eb13b 100644
> --- a/src/backend/utils/adt/tsrank.c
> +++ b/src/backend/utils/adt/tsrank.c
> @@ -337,12 +337,12 @@ calc_rank_or(const float *w, TSVector t, TSQuery q)
> }
> }
> /*
> - limit (sum(1/i^2),i=1,inf) = pi^2/6
> - resj = sum(wi/i^2),i=1,noccurrence,
> - wi - should be sorted desc,
> - don't sort for now, just choose maximum weight. This should be corrected
> - Oleg Bartunov
> -*/
> + * limit (sum(1/i^2),i=1,inf) = pi^2/6
> + * resj = sum(wi/i^2),i=1,noccurrence,
> + * wi - should be sorted desc,
> + * don't sort for now, just choose maximum weight. This should be corrected
> + * Oleg Bartunov
> + */
> res = res + (wjm + resj - wjm / ((jm + 1) * (jm + 1))) / 1.64493406685;
>
> entry++;
(Not that I understand what this is trying to tell me, mind)
--
Álvaro Herrera PostgreSQL Developer — https://www.EnterpriseDB.com/
"¿Cómo puedes confiar en algo que pagas y que no ves,
y no confiar en algo que te dan y te lo muestran?" (Germán Poo)
view thread (11+ messages) latest in thread
reply
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: [email protected]
Cc: [email protected], [email protected], [email protected], [email protected], [email protected], [email protected]
Subject: Re: Review - Patch for pg_bsd_indent: improve formatting of multiline comments
In-Reply-To: <[email protected]>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox