Re: Another pgindent gripe

On Thu, Feb 13, 2014 at 06:14:17PM -0500, Tom Lane wrote:
> I noticed that current pgindent wants to do this to a recently-added
> comment in psql/copy.c:
> 
> *************** handleCopyIn(PGconn *conn, FILE *copystr
> *** 627,633 ****
>                                         /*
>                                          * This code erroneously assumes '\.' on a line alone
>                                          * inside a quoted CSV string terminates the \copy.
> !                                        *
http://www.postgresql.org/message-id/E1TdNVQ-0001ju-GO@wrigleys.postgresql.org
>                                          */
>                                         if (strcmp(buf, "\\.\n") == 0 ||
>                                                 strcmp(buf, "\\.\r\n") == 0)
> --- 627,634 ----
>                                         /*
>                                          * This code erroneously assumes '\.' on a line alone
>                                          * inside a quoted CSV string terminates the \copy.
> !                                        * http://www.postgresql.org/message-id/E1TdNVQ-0001ju-GO@w
> !                                        * rigleys.postgresql.org
>                                          */
>                                         if (strcmp(buf, "\\.\n") == 0 ||
>                                                 strcmp(buf, "\\.\r\n") == 0)
> 
> Doesn't seem particularly friendly.

Yeah, that is ugly, and I was about to add a URL to a C comment myself
today.

I can set the column target width, but there doesn't seem to be any flag
that says to avoid breaking text to do the wrapping.

--  Bruce Momjian  <bruce@momjian.us>        http://momjian.us EnterpriseDB
http://enterprisedb.com
 + Everyone has their own god. +