Re: pgindent vs. git whitespace check

Поиск
Список
Период
Сортировка
От Bruce Momjian
Тема Re: pgindent vs. git whitespace check
Дата
Msg-id ZCRy5p0z+RP0oakf@momjian.us
обсуждение исходный текст
Ответ на Re: pgindent vs. git whitespace check  (Daniel Gustafsson <daniel@yesql.se>)
Ответы Re: pgindent vs. git whitespace check  (Daniel Gustafsson <daniel@yesql.se>)
Список pgsql-hackers
On Thu, Feb 23, 2023 at 09:36:00AM +0100, Daniel Gustafsson wrote:
> > On 23 Feb 2023, at 05:48, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> 
> > For my own taste, I really don't have any objection to // in isolation --
> > the problem with it is just that we've got megabytes of code in the other
> > style.  I fear it'd look really ugly to have an intermixture of // and /*
> > comment styles.  
> 
> We could use the "use the style of surrounding code (comments)" approach - when
> changing an existing commented function use the style already present; when
> adding a net new function a choice can be made (unless we mandate a style).  It
> will still look ugly, but it will be less bad than mixing within the same
> block.
> 
> > Mass conversion of /* to // style would answer that,
> > but would also create an impossible back-patching problem.
> 
> Yeah, that sounds incredibly invasive.

I am replying late here but ...

We would have to convert all supported branches, and tell all forks to
do the same (hopefully at the same time).  The new standard would then
be for all single-line comments to use // instead of /* ... */.

-- 
  Bruce Momjian  <bruce@momjian.us>        https://momjian.us
  EDB                                      https://enterprisedb.com

  Embrace your flaws.  They make you human, rather than perfect,
  which you will never be.



В списке pgsql-hackers по дате отправления:

Предыдущее
От: Peter Eisentraut
Дата:
Сообщение: Re: Transparent column encryption
Следующее
От: Alexander Korotkov
Дата:
Сообщение: Re: POC: Lock updated tuples in tuple_update() and tuple_delete()