Re: PL/Python patch for Universal Newline Support - Mailing list pgsql-patches

From Neil Conway
Subject Re: PL/Python patch for Universal Newline Support
Date
Msg-id 423E071C.4010901@samurai.com
Whole thread Raw
In response to PL/Python patch for Universal Newline Support  (Michael Fuhr <mike@fuhr.org>)
Responses Re: PL/Python patch for Universal Newline Support  (Tom Lane <tgl@sss.pgh.pa.us>)
Re: PL/Python patch for Universal Newline Support  (Michael Fuhr <mike@fuhr.org>)
List pgsql-patches
Michael Fuhr wrote:
> Should the PL/Python documentation mention this behavior?

Isn't this the behavior the user would expect? If so, I guess it's okay
not to document it.

> How should I submit regression tests?

Yes, please.

> *** src/pl/plpython/plpython.c    17 Dec 2004 02:14:48 -0000    1.58
> --- src/pl/plpython/plpython.c    19 Mar 2005 04:29:55 -0000
> ***************
> *** 1206,1215 ****
>
>       while (*sp != '\0')
>       {
> !         if (*sp == '\n')
>           {
> !             *mp++ = *sp++;
>               *mp++ = '\t';
>           }
>           else
>               *mp++ = *sp++;
> --- 1206,1219 ----
>
>       while (*sp != '\0')
>       {
> !         if (*sp == '\r' && *(sp + 1) == '\n')
> !             sp++;
> !
> !         if (*sp == '\n' || *sp == '\r')
>           {
> !             *mp++ = '\n';
>               *mp++ = '\t';
> +             sp++;
>           }
>           else
>               *mp++ = *sp++;

Does this work for "\r\n" embedded in string literals?

-Neil

pgsql-patches by date:

Previous
From: Neil Conway
Date:
Subject: Re: [patch 0/6] pgcrypto update
Next
From: Neil Conway
Date:
Subject: Re: HeapTupleSatisfiesUpdate result as enum