Escape patch applied

Started by Bruce Momjianabout 21 years ago3 messagespatches

bruce@momjian.us

about 21 years ago

I have applied the E'' escape patch to CVS head.

I am attaching a smaller version that doesn't have the
escape_string_warning GUC and none of the adjustments to our code to use
E''.

I am thinking I should apply this to 8.0.X and 7.4.X? OK? Earlier
versions?

-- 
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.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

Andrew Dunstan

andrew@dunslane.net

about 21 years ago

In reply to: Bruce Momjian (#1)

Re: Escape patch applied

Bruce Momjian wrote:

I have applied the E'' escape patch to CVS head.

You missed one regression fix: int8-exp-three-digits.out:

================== pgsql.26432/src/test/regress/regression.diffs ===================
*** ./expected/int8-exp-three-digits.out	Sun Jun 26 02:04:42 2005
--- ./results/int8.out	Sun Jun 26 03:34:16 2005
***************
*** 280,286 ****
              |  -4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 . 0 0 0
  (5 rows)

! SELECT '' AS to_char_16, to_char(q2, '99999 "text" 9999 "9999" 999 "\\"text between quote marks\\"" 9999') FROM INT8_TBL;
   to_char_16 |                          to_char                          
  ------------+-----------------------------------------------------------
              |       text      9999     "text between quote marks"   456
--- 280,286 ----
              |  -4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 . 0 0 0
  (5 rows)

! SELECT '' AS to_char_16, to_char(q2, E'99999 "text" 9999 "9999" 999 "\\"text between quote marks\\"" 9999') FROM INT8_TBL;
to_char_16 | to_char
------------+-----------------------------------------------------------
| text 9999 "text between quote marks" 456

======================================================================

cheers

andrew

Bruce Momjian

bruce@momjian.us

about 21 years ago

In reply to: Andrew Dunstan (#2)

Re: Escape patch applied

Got it. I couldn't find any more.

---------------------------------------------------------------------------

Andrew Dunstan wrote:

Bruce Momjian wrote:

I have applied the E'' escape patch to CVS head.

You missed one regression fix: int8-exp-three-digits.out:
================== pgsql.26432/src/test/regress/regression.diffs ===================
*** ./expected/int8-exp-three-digits.out	Sun Jun 26 02:04:42 2005
--- ./results/int8.out	Sun Jun 26 03:34:16 2005
***************
*** 280,286 ****
|  -4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 . 0 0 0
(5 rows)
! SELECT '' AS to_char_16, to_char(q2, '99999 "text" 9999 "9999" 999 "\\"text between quote marks\\"" 9999') FROM INT8_TBL;
to_char_16 |                          to_char                          
------------+-----------------------------------------------------------
|       text      9999     "text between quote marks"   456
--- 280,286 ----
|  -4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 . 0 0 0
(5 rows)
! SELECT '' AS to_char_16, to_char(q2, E'99999 "text" 9999 "9999" 999 "\\"text between quote marks\\"" 9999') FROM INT8_TBL;
to_char_16 | to_char
------------+-----------------------------------------------------------
| text 9999 "text between quote marks" 456

======================================================================

cheers

andrew

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

http://archives.postgresql.org

-- 
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.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

Escape patch applied

Attachments: