From: | "Dave Page" <dpage(at)vale-housing(dot)co(dot)uk> |
---|---|
To: | "Marko Ristola" <Marko(dot)Ristola(at)kolumbus(dot)fi>, "Merlin Moncure" <merlin(dot)moncure(at)rcsonline(dot)com> |
Cc: | "Eric E" <whalesuit(at)bonbon(dot)net>, <pgsql-odbc(at)postgresql(dot)org> |
Subject: | Re: Lowercase folding - simple patch? |
Date: | 2005-08-01 19:23:52 |
Message-ID: | E7F85A1B5FF8D44C8A1AF6885BC9A0E4AC96B4@ratbert.vale-housing.co.uk |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-odbc |
> -----Original Message-----
> From: pgsql-odbc-owner(at)postgresql(dot)org
> [mailto:pgsql-odbc-owner(at)postgresql(dot)org] On Behalf Of Marko Ristola
> Sent: 01 August 2005 17:28
> To: Merlin Moncure
> Cc: Eric E; pgsql-odbc(at)postgresql(dot)org
> Subject: Re: [ODBC] Lowercase folding - simple patch?
>
>
> Does somebody know, wether the following code is correct:
>
> odbcapi.c:
>
>
> RETCODE SQL_API
> SQLColumns(HSTMT StatementHandle, ...)
> {
> ...
>
> if (SC_is_lower_case(stmt, conn)) /* case-insensitive
> identifier */
> ifallupper = FALSE;
>
> In above there is one hidden problem:
> SC_is_lower_case() is implemented as follows:
>
> statement.h:#define SC_is_lower_case(a, b)
> (a->options.metadata_id,
> b->connInfo.lower_case_identifier)
>
>
> GCC compiler complaints (-Wall enabled) about the definition:
>
> odbcapi.c:140: warning: left-hand operand of comma expression
> has no effect
>
> So this means, that a->options.metadata_id is not used at
> all. Maybe the
> comma
> should be replaced with && or || ?
I *think* && is appropriate, however I'm too busy too look into this in
depth right now. Any chance you can confirm this?
(a->options.metadata_id is attribute SQL_ATTR_METADATA_ID btw).
Regards, Dave.
From | Date | Subject | |
---|---|---|---|
Next Message | Kevin Grittner | 2005-08-01 19:34:52 | Re: problem odbc 8.0 with EOModeler |
Previous Message | YL | 2005-08-01 18:47:59 | Re: problem odbc 8.0 with EOModeler |