Skip to content

Commit

Permalink
build.rs: Remove some previously-added C compiler warnings.
Browse files Browse the repository at this point in the history
Remove "-Winline" in particular because it has caused problems for
multiple people trying to port *ring* to new targets.

Remove '-Wextra", trusting the C compiler writers' judgement about
what should be in "-Wall."

Remove some others since they're more-or-less no longer relevant as we
decrease the C code in *ring*.
  • Loading branch information
briansmith committed Feb 17, 2024
1 parent 68bf136 commit 81fed3c
Showing 1 changed file with 0 additions and 7 deletions.
7 changes: 0 additions & 7 deletions build.rs
Original file line number Diff line number Diff line change
Expand Up @@ -114,16 +114,10 @@ fn cpp_flags(compiler: &cc::Tool) -> &'static [&'static str] {
"-fvisibility=hidden",
"-std=c1x", // GCC 4.6 requires "c1x" instead of "c11"
"-Wall",
"-Wextra",
"-Wbad-function-cast",
"-Wcast-align",
"-Wcast-qual",
"-Wconversion",
"-Wenum-compare",
"-Wfloat-equal",
"-Wformat=2",
"-Winline",
"-Winvalid-pch",
"-Wmissing-field-initializers",
"-Wmissing-include-dirs",
"-Wnested-externs",
Expand All @@ -134,7 +128,6 @@ fn cpp_flags(compiler: &cc::Tool) -> &'static [&'static str] {
"-Wstrict-prototypes",
"-Wundef",
"-Wuninitialized",
"-Wwrite-strings",
];
NON_MSVC_FLAGS
} else {
Expand Down

0 comments on commit 81fed3c

Please sign in to comment.