All driver managers support SQLDataSources now too

So we don't need the define?

Author: NattyNarwhal

ext/odbc/config.m4

@@ -131,9 +131,6 @@ AS_VAR_IF([PHP_ODBCVER], [no],
 dnl Extension setup
 if test -n "$ODBC_TYPE"; then
   PHP_EVAL_LIBLINE([$ODBC_LFLAGS $ODBC_LIBS], [ODBC_SHARED_LIBADD])
-  AC_DEFINE([HAVE_SQLDATASOURCES], [1],
-    [Define to 1 if ODBC library has 'SQLDataSources', as a function or
-    macro.])
 
   AC_DEFINE([HAVE_UODBC], [1],
     [Define to 1 if the PHP extension 'odbc' is available.])

ext/odbc/config.w32

@@ -9,7 +9,6 @@ if (PHP_ODBC == "yes") {
 	&& CHECK_HEADER_ADD_INCLUDE("sqlext.h", "CFLAGS_ODBC")) {
 		EXTENSION("odbc", "php_odbc.c odbc_utils.c", PHP_ODBC_SHARED, "/DZEND_ENABLE_STATIC_TSRMLS_CACHE=1");
 		AC_DEFINE("HAVE_UODBC", 1, "Define to 1 if the PHP extension 'odbc' is available.");
-		AC_DEFINE("HAVE_SQLDATASOURCES", 1, "Define to 1 if ODBC library has 'SQLDataSources', as a function or macro.");
 		if ("no" == PHP_ODBCVER) {
 			AC_DEFINE("ODBCVER", "0x0350", "The highest supported ODBC version.", false);
 		} else if ("0" != PHP_ODBCVER) {

ext/odbc/odbc.stub.php

@@ -341,9 +341,7 @@ function odbc_execute(Odbc\Result $statement, array $params = []): bool {}
 
     function odbc_cursor(Odbc\Result $statement): string|false {}
 
-#ifdef HAVE_SQLDATASOURCES
     function odbc_data_source(Odbc\Connection $odbc, int $fetch_type): array|null|false {}
-#endif
 
     function odbc_exec(Odbc\Connection $odbc, string $query): Odbc\Result|false {}
 

ext/odbc/odbc_arginfo.h

@@ -1199,7 +1199,6 @@ PHP_FUNCTION(odbc_cursor)
 }
 /* }}} */
 
-#ifdef HAVE_SQLDATASOURCES
 /* {{{ Return information about the currently connected data source */
 PHP_FUNCTION(odbc_data_source)
 {
@@ -1255,7 +1254,6 @@ PHP_FUNCTION(odbc_data_source)
 
 }
 /* }}} */
-#endif /* HAVE_SQLDATASOURCES */
 
 /* {{{ Prepare and execute an SQL statement */
 /* XXX Use flags */