Peter Eisentraut <peter.eisentraut@2ndquadrant.com> writes:
> While reviewing some unrelated code, I noticed that we are handling
> error conditions from Python API functions such as PyList_New() and
> PyDict_New() in pretty random ways or not at all. Here is a patch to
> fix that.
This needs some adjustment in the wake of 687f096ea.
I'm confused by the places that change PLy_elog calls to pass NULL:
- PLy_elog(ERROR, "could not create globals");
+ PLy_elog(ERROR, NULL);
How is that an improvement? Otherwise it looks reasonable.
regards, tom lane