Re: [HACKERS] bugfix for cursor arguments in named notation

2012-04-05 Thread Tom Lane
Yeb Havinga writes: > On 2012-04-04 17:10, Tom Lane wrote: >> I think a better way would be to temporarily set >> plpgsql_IdentifierLookup to IDENTIFIER_LOOKUP_DECLARE, > Thank you for looking at it. Attached is a patch that implements your > suggestion. Oh, sorry, I assumed you'd seen that I c

Re: [HACKERS] bugfix for cursor arguments in named notation

2012-04-05 Thread Yeb Havinga
On 2012-04-04 17:10, Tom Lane wrote: Yeb Havinga writes: Using a cursor argument name equal to another plpgsql variable results in the error: cursor .. has no argument named "" I think a better way would be to temporarily set plpgsql_IdentifierLookup to IDENTIFIER_LOOKUP_DECLARE, so as to supp

Re: [HACKERS] bugfix for cursor arguments in named notation

2012-04-04 Thread Tom Lane
Yeb Havinga writes: > Using a cursor argument name equal to another plpgsql variable results > in the error: > cursor .. has no argument named "" > The attached patch fixes that. > Instead of solving the issue like is done in the patch, another way > would be to expose internal_yylex() so that

[HACKERS] bugfix for cursor arguments in named notation

2012-04-04 Thread Yeb Havinga
Using a cursor argument name equal to another plpgsql variable results in the error: cursor .. has no argument named "" The attached patch fixes that. Instead of solving the issue like is done in the patch, another way would be to expose internal_yylex() so that could be used instead of yylex