Skip to content

Windows build: Use GREP_HEADER() instead of CHECK_FUNC_IN_HEADER()#21315

Open
petk wants to merge 1 commit intophp:masterfrom
petk:patch-windows-CHECK_FUNC_IN_HEADER
Open

Windows build: Use GREP_HEADER() instead of CHECK_FUNC_IN_HEADER()#21315
petk wants to merge 1 commit intophp:masterfrom
petk:patch-windows-CHECK_FUNC_IN_HEADER

Conversation

@petk
Copy link
Member

@petk petk commented Feb 27, 2026

Following #21191 this removes the following unused compile definitions:

  • HAVE_OSSL_SET_MAX_THREADS
  • HAVE_ARGON2ID_HASH_RAW

The CHECK_FUNC_IN_HEADER() function defines the 'HAVE_' compile definitions to 0 or 1, but these aren't used in the code. Defining such preprocessor macros makes it difficult to track and sync with other build systems.

This removes the following unused compile definitions:

- HAVE_OSSL_SET_MAX_THREADS
- HAVE_ARGON2ID_HASH_RAW

The CHECK_FUNC_IN_HEADER() function defines the 'HAVE_<FUNCTION>'
compile definitions to 0 or 1, but these aren't used in the code.
Defining such preprocessor macros makes it difficult to track and sync
with other build systems.
@petk petk force-pushed the patch-windows-CHECK_FUNC_IN_HEADER branch from 8c6bbba to 074ca9a Compare March 3, 2026 19:08
@petk petk marked this pull request as ready for review March 3, 2026 19:09
@petk petk requested a review from bukka as a code owner March 3, 2026 19:09
@petk
Copy link
Member Author

petk commented Mar 4, 2026

Merge coming up soon here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants