Re: Cosmetic changes

#1

Euler Taveira de Oliveira <eulerto (AT) yahoo (DOT) com.br> writes:

Quote:

This small patch correct a hint style and change from multiple to
single line comment because gettext seems not to like multiple line
comments. Then we could see the "translator: ..." in po files too.

I believe most of these changes are fixing comments that were written as
single lines to begin with, and were changed to multiline format by
pgindent which has extremely narrow (ahem) ideas about the appropriate
right margin for code and comments. Bruce, could we *please* fix
pgindent to not enforce a 72-column limit? 72 hasn't been interesting
since people stopped punching FORTRAN code into cards. 79 would be a
reasonable line length limit.

regards, tom lane

---------------------------(end of broadcast)---------------------------
TIP 6: Have you searched our list archives?

http://archives.postgresql.org

#2

Tom Lane wrote:

Quote:

Euler Taveira de Oliveira <eulerto (AT) yahoo (DOT) com.br> writes:
This small patch correct a hint style and change from multiple to
single line comment because gettext seems not to like multiple line
comments. Then we could see the "translator: ..." in po files too.

I believe most of these changes are fixing comments that were written as
single lines to begin with, and were changed to multiline format by
pgindent which has extremely narrow (ahem) ideas about the appropriate
right margin for code and comments. Bruce, could we *please* fix
pgindent to not enforce a 72-column limit? 72 hasn't been interesting
since people stopped punching FORTRAN code into cards. 79 would be a
reasonable line length limit.

It was set to 75. Updated to 79. I assume you don't want a rerun now.

--
Bruce Momjian | http://candle.pha.pa.us
pgman (AT) candle (DOT) pha.pa.us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073

---------------------------(end of broadcast)---------------------------
TIP 1: subscribe and unsubscribe commands go to majordomo (AT) postgresql (DOT) org

Thread Tools
Show Printable Version Email this Page
Display Modes
Linear Mode Switch to Hybrid Mode Switch to Threaded Mode

Re: Cosmetic changes

comp.databases.postgresql.patches comp.databases.postgresql.patches

Re: Cosmetic changes - 10-01-2004 , 01:21 AM

Re: Cosmetic changes - 10-01-2004 , 08:11 PM