Re: patch for cleanup protocol - Mailing list pgsql-odbc

From tomas@nocrew.org (Tomas Skäre)
Subject Re: patch for cleanup protocol
Date
Msg-id 803bkognw5.fsf@junk.nocrew.org
Whole thread Raw
In response to patch for cleanup protocol  (Ludek Finstrle <luf@pzkagis.cz>)
Responses Re: patch for cleanup protocol
List pgsql-odbc
Ludek Finstrle <luf@pzkagis.cz> writes:

> Hello,
>
>   protocol option is no longer used with libpq driver version.
> So I clean up the code. I try change both languages (English and Japanese).
> I didn't sure how to clean up faq.html so I don't do it.
>
> Please review and comment

I'm not qualified to tell if the changes are correct, but I just
noticed this one thing:

> diff -c psqlodbc.orig\dlg_specific.c psqlodbc.full\dlg_specific.c
> *** psqlodbc.orig\dlg_specific.c    Wed Nov 30 13:44:16 2005
> --- psqlodbc.full\dlg_specific.c    Mon Dec 19 23:33:09 2005
> ***************
> *** 65,72 ****
>
";%s=%s;%s=%s;%s=%s;%s=%s;%s=%s;%s=%s;%s=%s;%s=%d;%s=%d;%s=%d;%s=%d;%s=%d;%s=%d;%s=%d;%s=%d;%s=%d;%s=%d;%s=%d;%s=%d;%s=%d;%s=%d;%s=%d;%s=%s;%s=%d;%s=%d;%s=%d;%s=%d;%s=%d;%s=%d;%s=%d;%s=%d",
>                   INI_READONLY,
>                   ci->onlyread,
> -                 INI_PROTOCOL,
> -                 ci->protocol,
>                   INI_FAKEOIDINDEX,
>                   ci->fake_oid_index,
>                   INI_SHOWOIDCOLUMN,
> --- 65,70 ----

If you remove these arguments, you have to remove the corresponding
(second from start) "%s=%s" pair from the format string, otherwise
it'll use the wrong variables for the %d's further on.


Tomas

pgsql-odbc by date:

Previous
From: "Dave Page"
Date:
Subject: Re: patch for cleanup protocol
Next
From: "Andrus"
Date:
Subject: Re: [ psqlodbc-Bugs-1000481 ] VFP and SQLCancel