Hi Mark, >On Fri, Feb 12, 2021, at 20:56, Mark Rofail wrote: >Attachments: >anyarray_anyelement_operators-v2.patch
Some more code review comments: Comparing the v1 and v2 patch, I noticed this change in array_contains_elem(): + oprresult = DatumGetBool(FunctionCallInvoke(locfcinfo)); -+ if (oprresult) ++ if (!locfcinfo->isnull && oprresult) + return true; + } Is checking !locfcinfo->isnull due to something new in v2, or what is just a correction for a problem also in v1? /Joel
