build: prefer USE_IPV6
macro internally (was: ENABLE_IPV6
)
#13349
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Before this patch, two macros were used to guard IPv6 features in curl
sources:
ENABLE_IPV6
andUSE_IPV6
. This patch makes the source usethe latter for consistency with other similar switches.
-DENABLE_IPV6
remains accepted for compatibility as a synonym for-DUSE_IPV6
, when passed to the compiler.ENABLE_IPV6
also remains the name of the CMake andMakefile.vc
options to control this feature.
Closes #13349
Ideally I would prefer to switch over all uses in C from
ENABLE_IPV6
toUSE_IPV6
, which fits the naming used for all other public feature macros.Then leave
ENABLE_IPV6
as a compatibility macro that merely setsUSE_IPV6
. Thoughts?