On 6/25/07, Alvaro Herrera <[EMAIL PROTECTED]> wrote:
Tom Lane wrote:
> Alvaro Herrera <[EMAIL PROTECTED]> writes:
> > This function seems to have an argument that is no longer used (probably
> > leftover from when it used to report an error message?).
>
> Yeah, I recall having left the argument in place because it seemed like
> we might want it again someday.  But that was for 7.4 which was awhile
> ago.  There's probably not a strong argument to keep it, but on the
> other hand is there a strong argument to remove it?

Other than removing cruft, nope, I don't see any ...

In any case, this is an exported symbol so maybe it's not a good idea to
mess with it.  OTOH I checked PL/R and PL/php and neither uses it, so
this may not be a problem at all.


FWIW, we remove the second argument in textToQualifiedNameList() two
years ago so i think this is just finishing what we already started,
IMHO anyway
http://archives.postgresql.org/pgsql-committers/2005-05/msg00318.php

--
regards,
Jaime Casanova

"Programming today is a race between software engineers striving to
build bigger and better idiot-proof programs and the universe trying
to produce bigger and better idiots.
So far, the universe is winning."
                                      Richard Cook

---------------------------(end of broadcast)---------------------------
TIP 3: Have you checked our extensive FAQ?

              http://www.postgresql.org/docs/faq

Reply via email to