Re: Lowercase folding - simple patch?

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.

Responses

Browse pgsql-odbc by date

  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