NotificationsYou must be signed in to change notification settings
Fork5
Star26

Commit8722a1a

committed

Use the proper macro to convert a bool to a Datum.

The original coding wasvar->value = (Datum) state;which is bogus, and then in commit2f0f7b4it was "corrected" tovar->value = PointerGetDatum(state);which is a faithful translation but still wrong.This seems purely cosmetic, though, so no need for a back-patch.Pavel Stehule

1 parent4a90181 commit8722a1aCopy full SHA for 8722a1a

File tree

1 file changed

-3

lines changed

src/pl/plpgsql/src
- pl_exec.c

1 file changed

-3

lines changed

`‎src/pl/plpgsql/src/pl_exec.c`

Lines changed: 2 additions & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -5850,8 +5850,7 @@ exec_simple_recheck_plan(PLpgSQL_expr expr, CachedPlan cplan)`
`5850`	`5850`	`}`
`5851`	`5851`
`5852`	`5852`	`/* ----------`
`5853`		`- * exec_set_foundSet the global found variable`
`5854`		`- *to true/false`
	`5853`	`+ * exec_set_foundSet the global found variable to true/false`
`5855`	`5854`	`* ----------`
`5856`	`5855`	`*/`
`5857`	`5856`	`staticvoid`
`@@ -5860,7 +5859,7 @@ exec_set_found(PLpgSQL_execstate *estate, bool state)`
`5860`	`5859`	`PLpgSQL_var*var;`
`5861`	`5860`
`5862`	`5861`	`var= (PLpgSQL_var*) (estate->datums[estate->found_varno]);`
`5863`		`-var->value=PointerGetDatum(state);`
	`5862`	`+var->value=BoolGetDatum(state);`
`5864`	`5863`	`var->isnull= false;`
`5865`	`5864`	`}`
`5866`	`5865`

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!

Commit8722a1a

File tree

1 file changed

1 file changed

`‎src/pl/plpgsql/src/pl_exec.c`

0 commit comments