Enable -Wextra-semi (or -Wall) in ICU4C
V8 builds with this flag, and there are some ICU header files that produce this warning. Would be nice to clean them up and enable this warning in the default ICU4C build.
We should also consider using -Wall and using a blacklist rather than a whitelist of warning codes.
Closing as duplicate of
is a stronger form of this. Without treating warning as error, -Wall doesn't help much.
Also somewhat related: (Adding a build bot that treats warnings as errors.)
-Wall is a moving target, but without it, we have to manually add warning flags, which we clearly don't do on a regular bases, and we get dozens of tickets, each for a different random warning that we aren't catching. Using -Wall with a blacklist of warnings we don't want to fix seems like it is easier to maintain.
-Wall is a moving target