Skip to content

build: replace -pedantic with -Wpedantic when supported - #20010

Closed
vszakats wants to merge 2 commits into
curl:masterfrom
vszakats:wpedantic
Closed

vszakats wants to merge 2 commits into
curl:masterfrom
vszakats:wpedantic

Conversation

@vszakats

@vszakats vszakats commented Dec 17, 2025 •

Copy link
Copy Markdown
Member

To use the modern form when possible. The modern option also allows
using the no- prefix to disable.

Supported by: gcc 4.8+, clang 3.2+ (= appleclang 4.2+)

This also automatically changes -clang:-pedantic to -Wpedantic in
clang-cl builds.

Refs:
llvm/llvm-project@9877f68
https://gcc.gnu.org/onlinedocs/gcc-4.8.5/gcc/Warning-Options.html


https://github.com/curl/curl/pull/20010/files?w=1

@vszakats vszakats added the build label Dec 17, 2025
@vszakats
vszakats marked this pull request as draft December 17, 2025 14:24
@vszakats
vszakats marked this pull request as ready for review December 17, 2025 14:38
@vszakats vszakats closed this in 7e2b15e Dec 18, 2025
@vszakats
vszakats deleted the wpedantic branch December 18, 2025 00:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

1 participant