Use a switch() in CPDF_ColorSpace::ComponentsForFamily().
Change-Id: I33c5ae559bd149d29b3b5b0e8488f75eed742dc6
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/56295
Commit-Queue: Lei Zhang <thestig@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
diff --git a/core/fpdfapi/page/cpdf_colorspace.cpp b/core/fpdfapi/page/cpdf_colorspace.cpp
index 431fa1e..15e772f 100644
--- a/core/fpdfapi/page/cpdf_colorspace.cpp
+++ b/core/fpdfapi/page/cpdf_colorspace.cpp
@@ -527,12 +527,17 @@
// static
uint32_t CPDF_ColorSpace::ComponentsForFamily(int family) {
- if (family == PDFCS_DEVICERGB)
- return 3;
- if (family == PDFCS_DEVICEGRAY)
- return 1;
- ASSERT(family == PDFCS_DEVICECMYK);
- return 4;
+ switch (family) {
+ case PDFCS_DEVICEGRAY:
+ return 1;
+ case PDFCS_DEVICERGB:
+ return 3;
+ case PDFCS_DEVICECMYK:
+ return 4;
+ default:
+ NOTREACHED();
+ return 4;
+ }
}
size_t CPDF_ColorSpace::GetBufSize() const {