Hello, this is new version of the patch.
# This patch is based on the commit
# 2bbd88f8f841b01efb073972b60d4dc1ff1f6fd0 @ Feb 13 to avoid the
# compile error caused by undeclared LEAKPROOF in kwlist.h.
> It must free the PGresults that PQgetResult() returns.
I'm sorry. It slipped out of my mind. Add PQclear() for the
return value.
> Also, please fix 2 issues mentined here:
- PQsetRowProcessorErrMsg() now handles msg as const string.
- Changed the order of the parameters of the type PQrowProcessor.
New order is (PGresult *res, PGrowValue *columns, void *params).
# PQsetRowProcessorErrMsg outside of callback is not implemented.
- Documentation and dblink are modified according to the changes
above.
By the way, I would like to ask you one question. What is the
reason why void* should be head or tail of the parameter list?
regards,
--
Kyotaro Horiguchi
NTT Open Source Software Center
In response to
Responses
pgsql-hackers by date
| Next: | From: Peter Eisentraut | Date: 2012-02-23 11:08:51 |
| Subject: overriding current_timestamp |
| Previous: | From: Daniel Farina | Date: 2012-02-23 09:58:44 |
| Subject: pg_stat_statements normalization: re-review |