Re: Let's use libpq for everything - Mailing list pgsql-odbc

From Michael Paquier
Subject Re: Let's use libpq for everything
Date
Msg-id CAB7nPqQEELm+pOEaq1nG165Mq5jRt9uiPFixB8Wpari23rOayg@mail.gmail.com
Whole thread Raw
In response to Re: Let's use libpq for everything  (Heikki Linnakangas <hlinnakangas@vmware.com>)
Responses Re: Let's use libpq for everything
List pgsql-odbc
On Tue, Dec 23, 2014 at 5:53 AM, Heikki Linnakangas
<hlinnakangas@vmware.com> wrote:
> I've merged this with latest changes in the master branch, and did some
> error handling fixes. Latest version is again attached here, and also
> available in github
> (https://github.com/hlinnaka/psqlodbc/tree/libpq-integration4).

Some minor comments after scanning the code:
1) The description below should be prepareParametersNoDesc.
+/*
+ * Process the original SQL query.
+ */
+RETCODE        prepareParametersNoDesc(StatementClass *stmt)
+{
+       QueryParse      query_org, *qp;
+       QueryBuild      query_crt, *qb;
+
+inolog("prepareParametersNoSync\n");
2) In psqlodbc.vcproj, you can safely remove the references to USE_LIBPQ.
3) s/Extrace/Extract
ISTM that this code is clean, I haven't found flaws in the new logic
where code is replaced with libpq stuff.
Regards,
--
Michael


pgsql-odbc by date:

Previous
From: Heikki Linnakangas
Date:
Subject: Re: Let's use libpq for everything
Next
From: Heikki Linnakangas
Date:
Subject: Re: Let's use libpq for everything