public inbox for [email protected]  

help / color / mirror / Atom feed

From: Bruce Momjian <[email protected]>
To: Peter Smith <[email protected]>
Cc: Michael Paquier <[email protected]>
Cc: Alvaro Herrera <[email protected]>
Cc: Tom Lane <[email protected]>
Cc: PostgreSQL Hackers <[email protected]>
Subject: Re: [PATCH] postgresql.conf.sample comment alignment.
Date: Tue, 31 Oct 2023 08:51:48 -0400
Message-ID: <[email protected]> (raw)
In-Reply-To: <CAHut+Pv0CCx0M+pg=GfapBXxUnukiUFUsO7uS2WU6x-kd1nN9A@mail.gmail.com>
References: <[email protected]>
	<[email protected]>
	<[email protected]>
	<CAHut+Pv0CCx0M+pg=GfapBXxUnukiUFUsO7uS2WU6x-kd1nN9A@mail.gmail.com>

On Thu, Aug  4, 2022 at 12:42:38PM +1000, Peter Smith wrote:
> On Thu, Aug 4, 2022 at 11:09 AM Michael Paquier <[email protected]> wrote:
> >
> > On Wed, Aug 03, 2022 at 12:58:04PM +0200, Alvaro Herrera wrote:
> > > On 2022-Aug-01, Tom Lane wrote:
> > >> One idea for avoiding confusion is to legislate that we won't
> > >> use tabs at all in this file (which we could enforce via
> > >> .gitattributes, I think).
> > >
> > > +1.
> >
> > That's not the first time this 4- or 8-character tab issue is popping
> > up around here, so enforcing spaces and having a rule sounds like a
> > good idea at the end.
> >
> 
> Well, it was only assumed that I had probably confused 4- 8- tabs, but
> I don't think I did, so the tabbing issue did not really "pop up"
> here.
> 
> e.g. you can see some of the existing alignments I'd suggested
> modifying here [1]
> - #shared_preload_libraries = '' # (change requires restart)
> - #idle_in_transaction_session_timeout = 0 # in milliseconds, 0 is
> disable <- (moved comments of the neighbours to keep them all aligned)
> - etc.
> 
> I'm not saying replacing the tabs with spaces isn't a good idea - I
> also agree probably it is, but that's a different problem to the
> alignments I was trying to correct with the patch

Patch applied to master.  Perhaps someday we will adjust tabs, but for
now, this is an improvements.  I made a few small adjustments myself.

-- 
  Bruce Momjian  <[email protected]>        https://momjian.us
  EDB                                      https://enterprisedb.com

  Only you can decide what is important to you.

view thread (2+ messages)

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: [PATCH] postgresql.conf.sample comment alignment.
  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