Tatsuo Ishii <[EMAIL PROTECTED]> writes:
>> Breaking the type_sanity test is not acceptable.  Put in a second C
>> function.

> Are you talking opr_sanity?

Sorry, yes, too little caffeine ...

> What is evil with a polymorphic function?

(1) It's creating a false match --- your proposed entry in the opr_sanity
results has nothing at all to do with what the test is looking for.

(2) Refactoring to have two separate C functions will make the code
clearer, and not noticeably longer.

                        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