Re: [HACKERS] Preliminary results for proposed new pgindentimplementation
От | Andres Freund |
---|---|
Тема | Re: [HACKERS] Preliminary results for proposed new pgindentimplementation |
Дата | |
Msg-id | 20170616180212.4yk44vso3n24bfns@alap3.anarazel.de обсуждение исходный текст |
Ответ на | Re: [HACKERS] Preliminary results for proposed new pgindentimplementation (Bruce Momjian <bruce@momjian.us>) |
Ответы |
Re: [HACKERS] Preliminary results for proposed new pgindent implementation
|
Список | pgsql-hackers |
On 2017-06-16 13:44:30 -0400, Bruce Momjian wrote: > On Fri, Jun 16, 2017 at 01:34:01PM -0400, Tom Lane wrote: > > > I could live with both of these proposed > > > changes, the selection of the changes you posted looks like it could be > > > improved by code changes, but that's obviously a large amount of work. > > > > In the end, the only thing that fixes this sort of stuff is to be more > > rigid about making the code fit into 80 columns to begin with. I get > > the impression though that a lot of people work in editor windows that > > are wider than that, so the code looks fine to them when it slops over > > a bit. > > Yes, it is all about <80 column output. The current pgindent does > everything possible to accomplish that --- the question is whether we > want uglier code to do it. For me personally the misindentation is way uglier than a too long line. I think a number of those long-lines are there because pgindent sometimes re-indents lines that are continuations of previous ones pretty far, making it hard to reduce indentation. - Andres
В списке pgsql-hackers по дате отправления: