Skip to content

Commit c666884

Browse files
committed
Remove wrappers for SQLColAttribute
We don't need to support the old way of doing it.
1 parent 7432c18 commit c666884

File tree

2 files changed

+6
-8
lines changed

2 files changed

+6
-8
lines changed

ext/odbc/php_odbc.c

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -676,10 +676,10 @@ void odbc_bindcols(odbc_result *result)
676676
bool char_extra_alloc = false;
677677
colfieldid = SQL_COLUMN_DISPLAY_SIZE;
678678

679-
rc = PHP_ODBC_SQLCOLATTRIBUTE(result->stmt, (SQLUSMALLINT)(i+1), PHP_ODBC_SQL_DESC_NAME,
679+
rc = SQLColAttribute(result->stmt, (SQLUSMALLINT)(i+1), SQL_DESC_NAME,
680680
result->values[i].name, sizeof(result->values[i].name), &colnamelen, 0);
681681
result->values[i].coltype = 0;
682-
rc = PHP_ODBC_SQLCOLATTRIBUTE(result->stmt, (SQLUSMALLINT)(i+1), SQL_COLUMN_TYPE,
682+
rc = SQLColAttribute(result->stmt, (SQLUSMALLINT)(i+1), SQL_COLUMN_TYPE,
683683
NULL, 0, NULL, &result->values[i].coltype);
684684

685685
/* Don't bind LONG / BINARY columns, so that fetch behaviour can
@@ -710,7 +710,7 @@ void odbc_bindcols(odbc_result *result)
710710
/* TODO: Check this is the intended behaviour */
711711
ZEND_FALLTHROUGH;
712712
default:
713-
rc = PHP_ODBC_SQLCOLATTRIBUTE(result->stmt, (SQLUSMALLINT)(i+1), colfieldid,
713+
rc = SQLColAttribute(result->stmt, (SQLUSMALLINT)(i+1), colfieldid,
714714
NULL, 0, NULL, &displaysize);
715715
if (rc != SQL_SUCCESS) {
716716
displaysize = 0;
@@ -832,7 +832,7 @@ void odbc_column_lengths(INTERNAL_FUNCTION_PARAMETERS, int type)
832832
RETURN_FALSE;
833833
}
834834

835-
PHP_ODBC_SQLCOLATTRIBUTE(result->stmt, (SQLUSMALLINT)pv_num, (SQLUSMALLINT) (type?SQL_COLUMN_SCALE:SQL_COLUMN_PRECISION), NULL, 0, NULL, &len);
835+
SQLColAttribute(result->stmt, (SQLUSMALLINT)pv_num, (SQLUSMALLINT) (type?SQL_COLUMN_SCALE:SQL_COLUMN_PRECISION), NULL, 0, NULL, &len);
836836

837837
RETURN_LONG(len);
838838
}
@@ -1830,7 +1830,7 @@ PHP_FUNCTION(odbc_result)
18301830
fieldsize = result->longreadlen;
18311831
}
18321832
} else {
1833-
PHP_ODBC_SQLCOLATTRIBUTE(result->stmt, (SQLUSMALLINT)(field_ind + 1),
1833+
SQLColAttribute(result->stmt, (SQLUSMALLINT)(field_ind + 1),
18341834
(SQLUSMALLINT)((sql_c_type == SQL_C_BINARY) ? SQL_COLUMN_LENGTH :
18351835
SQL_COLUMN_DISPLAY_SIZE),
18361836
NULL, 0, NULL, &fieldsize);
@@ -2577,7 +2577,7 @@ PHP_FUNCTION(odbc_field_type)
25772577
RETURN_FALSE;
25782578
}
25792579

2580-
PHP_ODBC_SQLCOLATTRIBUTE(result->stmt, (SQLUSMALLINT)pv_num, SQL_COLUMN_TYPE_NAME, tmp, 31, &tmplen, NULL);
2580+
SQLColAttribute(result->stmt, (SQLUSMALLINT)pv_num, SQL_COLUMN_TYPE_NAME, tmp, 31, &tmplen, NULL);
25812581
RETURN_STRING(tmp);
25822582
}
25832583
/* }}} */

ext/odbc/php_odbc_includes.h

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -264,10 +264,8 @@ void odbc_sql_error(ODBC_SQL_ERROR_PARAMS);
264264

265265
#define IS_SQL_LONG(x) (x == SQL_LONGVARBINARY || x == SQL_LONGVARCHAR || x == SQL_WLONGVARCHAR)
266266

267-
#define PHP_ODBC_SQLCOLATTRIBUTE SQLColAttribute
268267
#define PHP_ODBC_SQLALLOCSTMT(hdbc, phstmt) SQLAllocHandle(SQL_HANDLE_STMT, hdbc, phstmt)
269268

270-
#define PHP_ODBC_SQL_DESC_NAME SQL_DESC_NAME
271269
#define IS_SQL_BINARY(x) (x == SQL_BINARY || x == SQL_VARBINARY || x == SQL_LONGVARBINARY)
272270

273271
PHP_ODBC_API ZEND_EXTERN_MODULE_GLOBALS(odbc)

0 commit comments

Comments
 (0)