NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit1613de8

committed

Fix compiler warning on MSVC

MSVC does not understand that ereport(ERROR) does not return, so justreturn the first enum PartitionStrategy value to keep the compiler fromcomplaining about the missing return.Discussion:https://postgr.es/m/20221104161934.GB16921@telsasoft.com

1 parent0e758ae commit1613de8Copy full SHA for 1613de8

File tree

1 file changed

-0

lines changed

src/backend/parser
- gram.y

1 file changed

-0

lines changed

`‎src/backend/parser/gram.y`

Lines changed: 3 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -18428,10 +18428,13 @@ parsePartitionStrategy(char *strategy)`
`18428`	`18428`	`return PARTITION_STRATEGY_RANGE;`
`18429`	`18429`	`else if (pg_strcasecmp(strategy, "hash") == 0)`
`18430`	`18430`	`return PARTITION_STRATEGY_HASH;`
	`18431`	`+`
`18431`	`18432`	`ereport(ERROR,`
`18432`	`18433`	`(errcode(ERRCODE_INVALID_PARAMETER_VALUE),`
`18433`	`18434`	`errmsg("unrecognized partitioning strategy \"%s\"",`
`18434`	`18435`	`strategy)));`
	`18436`	`+return PARTITION_STRATEGY_LIST;/* keep compiler quiet*/`
	`18437`	`+`
`18435`	`18438`	`}`
`18436`	`18439`
`18437`	`18440`	`/*`

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!

Commit1613de8

File tree

1 file changed

1 file changed

`‎src/backend/parser/gram.y`

0 commit comments