Remove an unnecessary call in CPDF_TransferFuncDIB ctor.
Instead of clearing `m_palette`, just DCHECK() that it starts out empty.
Change-Id: Ic5d7c32ebdb30c825c0a40465779709f4bea0f38
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/75250
Commit-Queue: Lei Zhang <thestig@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
diff --git a/core/fpdfapi/page/cpdf_transferfuncdib.cpp b/core/fpdfapi/page/cpdf_transferfuncdib.cpp
index 9ccfb12..4f03127 100644
--- a/core/fpdfapi/page/cpdf_transferfuncdib.cpp
+++ b/core/fpdfapi/page/cpdf_transferfuncdib.cpp
@@ -11,6 +11,7 @@
#include "build/build_config.h"
#include "core/fpdfapi/page/cpdf_transferfunc.h"
#include "core/fpdfapi/parser/cpdf_dictionary.h"
+#include "third_party/base/check.h"
#include "third_party/base/compiler_specific.h"
CPDF_TransferFuncDIB::CPDF_TransferFuncDIB(
@@ -27,8 +28,8 @@
m_bpp = GetBppFromFormat(format);
m_AlphaFlag = GetAlphaFlagFromFormat(format);
m_Pitch = (m_Width * m_bpp + 31) / 32 * 4;
- m_palette.clear();
m_Scanline.resize(m_Pitch);
+ DCHECK(m_palette.empty());
}
CPDF_TransferFuncDIB::~CPDF_TransferFuncDIB() = default;