2010/12/17 Alvaro Herrera <alvhe...@commandprompt.com>: > Excerpts from Pavel Stehule's message of vie dic 17 07:02:00 -0300 2010: >> Hello >> >> This patch remove redundant rows from PL/pgSQL executor (-89 lines). >> Doesn't change a functionality. > > Hmm I'm not sure but I think the new code has some of the result values > inverted. Did you test this thoroughly? I think this would be a nice > simplification because the repetitive coding is ugly and confusing, but > I'm nervous about the unstated assumption that all loop structs are > castable to the new struct. Seems like it could be easily broken in the > future. >
All regress tests was successful. A common structure isn't a new. There is same for FOR loops, there is some similar in parser yylval, and it is only explicit description of used construction for stmt structures. I should not to modify any other structure. But I am not strong in this. A interface can be changed and enhanced about pointer to label. Just I am not satisfied from current state, where same things are implemented with minimal difference. Pavel > -- > Álvaro Herrera <alvhe...@commandprompt.com> > The PostgreSQL Company - Command Prompt, Inc. > PostgreSQL Replication, Consulting, Custom Development, 24x7 support > -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers