Re: [bug fix] ECPG: freeing memory for pgtypes crashes on Windows

At Sun, 25 Mar 2018 22:15:52 +0900, "MauMau" <maumau307(at)gmail(dot)com> wrote in <B3BEB35436E3471095762969E2FCEDD6(at)tunaPC>
> And thank you for your review. All modifications are done.

Thank you for the new version. I marked this as "Ready for
Committer" with one change.

- Windows requires this since different versions (MT/non-MT and
DEBUG/RELEASE?) of CRT are not compatible on malloc/free, which
is the same reason for PQfreemem().

- It applies on the master HEAD cleanly. Compiled with no
error. (Except for having some warnings with MSB8018 *1 for
some mb modules and seemingly harmless C4818 for many files and
this patch is not to blame.)

- Documentation looks fine.

- The change on regtest looks fine and ran sucessfully on both
Linux and Windows (vcregress ecpgcheck). But it doesn't prove
anything about the different versions of CRT library. (The
same can be said about PQfreemem())

- Style looks fine with one exception that extern "C" is
increasing indentation, so I fixed that in the attached
version.

*1: "The intermediate directory contains files shared from
another project" for pg_archivecleanup, pg_stat_statements,
pg_isolation_regress, latin2_and_win1250, utf8_and_cyrillic,
utf8_and_iso8859 and utf8_and_sjis2004.

regards,

--
Kyotaro Horiguchi
NTT Open Source Software Center