NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit3b0ee7f

committed

Ensure that length argument of memcmp() isn't seen as negative.

I think this will shut up a weird warning from buildfarm memberserinus. Perhaps it'd be better to change tsCompareString'slength arguments to unsigned, but that seems more invasivethan is justified.Part of a general push to remove off-the-beaten-track warningswhere we can easily do so.

1 parent4c1a1a3 commit3b0ee7fCopy full SHA for 3b0ee7f

File tree

1 file changed

-1

lines changed

src/backend/utils/adt
- tsvector_op.c

1 file changed

-1

lines changed

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

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -1167,7 +1167,7 @@ tsCompareString(char a, int lena, char b, int lenb, bool prefix)`
`1167`	`1167`	`}`
`1168`	`1168`	`else`
`1169`	`1169`	`{`
`1170`		`-cmp=memcmp(a,b,Min(lena,lenb));`
	`1170`	`+cmp=memcmp(a,b,Min((unsignedint)lena, (unsignedint)lenb));`
`1171`	`1171`
`1172`	`1172`	`if (prefix)`
`1173`	`1173`	`{`

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!

Commit3b0ee7f

File tree

1 file changed

1 file changed

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

0 commit comments