SEO

#1 (**permalink**) 04-17-2008, 11:22 PM

Jaime Casanova wrote:
> i found out that the function textToQualifiedNameList doesn't use the
> second argument it receive (caller). i suppose in the past was used
> and now it is useless, if that is the case here is a patch removing.
> Or are any reasons to keep that argument?

None that I can see. I'll apply the patch later today, barring any
objections.

-Neil

---------------------------(end of broadcast)---------------------------
TIP 4: Don't 'kill -9' the postmaster

#2 (**permalink**) 04-17-2008, 11:22 PM

Neil Conway <neilc@samurai.com> writes:
> Jaime Casanova wrote:
>> Or are any reasons to keep that argument?

> None that I can see. I'll apply the patch later today, barring any
> objections.

Looking at the 7.3 sources, the argument was used to support the
then-existing convention that an error message include the name
of the reporting routine. This does seem obsolete now.

regards, tom lane

---------------------------(end of broadcast)---------------------------
TIP 4: Don't 'kill -9' the postmaster

Thread Tools
Show Printable Version Email this Page
Display Modes
Linear Mode Switch to Hybrid Mode Switch to Threaded Mode

SEO

Re: Remove second argument from textToQualifiedNameList