NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitb64a754

committed

From: "Thomas G. Lockhart" <lockhart@alumni.caltech.edu>

For substr() and substring() on the text data type, the relevant code is invarlena.c. You are right, there is a problem. I have a patch which I willapply to the source tree soon. The copy enclosed below probably does notpreserve tabs correctly so cannot be applied directly; the relevant changeis simply changing the ">=" to ">"...

1 parent6ac2528 commitb64a754Copy full SHA for b64a754

File tree

1 file changed

-2

lines changed

src/backend/utils/adt
- varlena.c

1 file changed

-2

lines changed

`‎src/backend/utils/adt/varlena.c`

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,7 @@`
`7`	`7`	`*`
`8`	`8`	`*`
`9`	`9`	`* IDENTIFICATION`
`10`		`- * $Header: /cvsroot/pgsql/src/backend/utils/adt/varlena.c,v 1.31 1998/02/26 04:37:24 momjian Exp $`
	`10`	`+ * $Header: /cvsroot/pgsql/src/backend/utils/adt/varlena.c,v 1.32 1998/03/15 08:07:01 scrappy Exp $`
`11`	`11`	`*`
`12`	`12`	`*-------------------------------------------------------------------------`
`13`	`13`	`*/`
`@@ -291,7 +291,7 @@ text_substr(text *string, int32 m, int32 n)`
`291`	`291`	`len=VARSIZE(string)-VARHDRSZ;`
`292`	`292`
`293`	`293`	`/* m will now become a zero-based starting position */`
`294`		`-if (m >=len)`
	`294`	`+if (m>len)`
`295`	`295`	`{`
`296`	`296`	`m=0;`
`297`	`297`	`n=0;`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commitb64a754

File tree

1 file changed

1 file changed

`‎src/backend/utils/adt/varlena.c`

0 commit comments