Re: [Firebird-odbc-devel] More little fixes

[Paul R. <pr...@ib...>]

Carlos G.A. wrote:
> Hello:
> 
> More little fixes:
> 
> Note: SQL
> 
> 1. InfoItems.h
> 
> Change
> 
>     UITEM (SQL_MAX_USER_NAME_LEN, 0)
>     UITEM (SQL_MAX_SCHEMA_NAME_LEN, 0)

 > for

>     SITEM (SQL_MAX_USER_NAME_LEN, 0)
>     SITEM (SQL_MAX_SCHEMA_NAME_LEN, 0)
> 
> 
> 2. OdbcConnection.cpp
> 
> Add to SQLGetInfo this:
>     case SQL_MAX_USER_NAME_LEN:
>     case SQL_MAX_OWNER_NAME_LEN:
>         value = metaData->getMaxUserNameLength();
>         break;
> 
>     case SQL_MAX_SCHEMA_NAME_LEN:
>         value = metaData->getMaxSchemaNameLength();
>         break;
> 

There is a conflict between MAX_SCHEMA_NAME_LEN and 
SQL_MAX_OWNER_NAME_LEN. They are defined with the same value. Only one 
of them can exist in the same case statement. As schema are not actually 
supported by Firebird I would be inclined to leave 
SQL_MAX_SCHEMA_NAME_LEN out of sqlGetInfoItems and use the UITEM() macro 
in InfoItems.h


Thoughts?


Paul
-- 

Paul Reeves
http://www.ibphoenix.com
Supporting users of Firebird and InterBase