Skip to content

Commit f426f16

Browse files
committed
Drop unused PHP7ISAPI_EXPORTS
These definitions are usually used to distinguish between `dllexport` and `dllimport`, but there is no need for `dllimport`, and we don't use `dllexport` to specify the visible functions, but rather rely on a .def file, since the Microsoft ISAPI headers predeclare the functions, so the `dllexport` name mangling would cause link errors.
1 parent b73358d commit f426f16

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

sapi/isapi/config.w32

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ if (PHP_ISAPI == "yes") {
66
if (PHP_ZTS == "no") {
77
WARNING("ISAPI module requires an --enable-zts build of PHP");
88
} else {
9-
SAPI('isapi', 'php_isapi.c', 'php' + PHP_VERSION + 'isapi.dll', '/D PHP7ISAPI_EXPORTS /DZEND_ENABLE_STATIC_TSRMLS_CACHE=1');
9+
SAPI('isapi', 'php_isapi.c', 'php' + PHP_VERSION + 'isapi.dll', '/DZEND_ENABLE_STATIC_TSRMLS_CACHE=1');
1010
ADD_FLAG('LDFLAGS_ISAPI', '/DEF:sapi\\isapi\\php_isapi.def');
1111
}
1212
}

0 commit comments

Comments
 (0)