On Fri, Feb 2, 2018 at 3:47 PM, Tsunakawa, Takayuki
<tsunakawa.ta...@jp.fujitsu.com> wrote:
> The fix is to add PGTYPES_free() in libpgtypes.dll, just like libpq has 
> PQfreemem() described here:

+ extern void PGTYPES_free(void *ptr);

It seems quite strange to repeat this in pgtypes_date.h,
pgtypes_interval.h and pgtypes_numeric.h.  I guess you might not want
to introduce a new common header file so that his can be back-patched
more easily?  Not sure if there is a project policy about that, but it
seems unfortunate to introduce maintenance burden by duplicating this.

+   <function>PGTYPES_free()/<function> instead of <function>free()</function>.

The "/" needs to move inside then closing tag.

Thomas Munro

Reply via email to