commit c207ee2c33e148176dd6671d641ba978b94ae7bf
author Lei Zhang <thestig@chromium.org> Fri Apr 28 20:11:23 2023 +0000
committer Pdfium LUCI CQ <pdfium-scoped@luci-project-accounts.iam.gserviceaccount.com> Fri Apr 28 20:11:23 2023 +0000
tree 5645a0dc8a15b386364d85dd583f5a47de7da5eb
parent 0703e04867f08970699fcf7938b2b0d7352b91d1

Enable -Wcovered-switch-default

When building PDFium with Clang, enable -Wcovered-switch-default.
Fix remaining warnings and remove an unnecessary variable initialization
along the way.

Bug: chromium:1393075
Change-Id: Ia7a439f2ca787f9e3dafb8d30fed2a38b7e29719
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/106670
Commit-Queue: Lei Zhang <thestig@chromium.org>
Reviewed-by: Nigi <nigi@chromium.org>

BUILD.gn

diff --git a/BUILD.gn b/BUILD.gn
index cb1da47..ca42418 100644
--- a/BUILD.gn
+++ b/BUILD.gn
@@ -288,6 +288,9 @@
       "/wd4577",
     ]
   }
+  if (is_clang) {
+    cflags += [ "-Wcovered-switch-default" ]
+  }
 }
 
 config("pdfium_strict_config") {