Jeff Davis <pg...@j-davis.com> writes:
> This is just to make the variable names in this function consistent with
> the rest of the file (and less confusing). No functional impact, so I'll
> commit soon unless someone objects.

If I'm reading this right, this creates two different variables pointing
at the same object.  That does not sound like an improvement from a
readability standpoint.

If you want to have just *one* variable but change its name and type,
I'd be ok with that.

                        regards, tom lane


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to