Thanks for comments, Euler! > ... is hard to understand. Instead, you could separate the conditional > expression into a variable.
Fixed the patch to be more descriptive and to avoid repeating same computation over and over again. See v2 of the patch attached. > I don't think those are mandatory. Why do you think that? Should I remove them or maybe send as separate patch? Cheers, Roma
fix_drop_operator_reset_oprcom_and_oprnegate_fields_v2.patch
Description: Binary data
-- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
