NotificationsYou must be signed in to change notification settings
Fork5
Star26

Commit0f427df

committed

Allow plpgsql IN parameters to be assigned to. Since the parameters are just

preinitialized local variables, this does not affect the function's semanticsas seen by callers; allowing assignment simply avoids the need to create morelocal variables in some cases. In any case we were being rather inconsistentsince only scalar parameters were getting marked constant.No documentation change, since parameters were never documented as beingmarked constant anyway.Steve Prentice

1 parent94f238c commit0f427dfCopy full SHA for 0f427df

File tree

1 file changed

-5

lines changed

src/pl/plpgsql/src
- pl_comp.c

1 file changed

-5

lines changed

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

Lines changed: 1 addition & 5 deletions

Original file line number	Diff line number	Diff line change
`@@ -8,7 +8,7 @@`
`8`	`8`	`*`
`9`	`9`	`*`
`10`	`10`	`* IDENTIFICATION`
`11`		`- * $PostgreSQL: pgsql/src/pl/plpgsql/src/pl_comp.c,v 1.137 2009/07/12 17:12:34 tgl Exp $`
	`11`	`+ * $PostgreSQL: pgsql/src/pl/plpgsql/src/pl_comp.c,v 1.138 2009/09/20 01:53:32 tgl Exp $`
`12`	`12`	`*`
`13`	`13`	`*-------------------------------------------------------------------------`
`14`	`14`	`*/`
`@@ -424,10 +424,6 @@ do_compile(FunctionCallInfo fcinfo,`
`424`	`424`	`if (argvariable->dtype==PLPGSQL_DTYPE_VAR)`
`425`	`425`	`{`
`426`	`426`	`argitemtype=PLPGSQL_NSTYPE_VAR;`
`427`		`-/* input argument vars are forced to be CONSTANT */`
`428`		`-if (argmode==PROARGMODE_IN\|\|`
`429`		`-argmode==PROARGMODE_VARIADIC)`
`430`		`-((PLpgSQL_var*)argvariable)->isconst= true;`
`431`	`427`	`}`
`432`	`428`	`else`
`433`	`429`	`{`

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!

Commit0f427df

File tree

1 file changed

1 file changed

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

0 commit comments