Index: fpdfsdk/src/fsdk_baseform.cpp |
diff --git a/fpdfsdk/src/fsdk_baseform.cpp b/fpdfsdk/src/fsdk_baseform.cpp |
index 9f8b35dcb117301afb2365e71b9599e5689be115..3a304986d494cebc23d851ba879f852e39ef1a25 100644 |
--- a/fpdfsdk/src/fsdk_baseform.cpp |
+++ b/fpdfsdk/src/fsdk_baseform.cpp |
@@ -1519,7 +1519,7 @@ CPDFSDK_InterForm::CPDFSDK_InterForm(CPDFSDK_Document* pDocument) |
ASSERT(m_pInterForm != NULL); |
m_pInterForm->SetFormNotify(this); |
- for (int i = 0; i < 6; i++) |
+ for (int i = 0; i < kNumFieldTypes; ++i) |
m_bNeedHightlight[i] = FALSE; |
m_iHighlightAlpha = 0; |
} |
@@ -2239,20 +2239,22 @@ int CPDFSDK_InterForm::AfterFormImportData(const CPDF_InterForm* pForm) { |
} |
FX_BOOL CPDFSDK_InterForm::IsNeedHighLight(int nFieldType) { |
- if (nFieldType < 1 || nFieldType > 6) |
+ if (nFieldType < 1 || nFieldType > kNumFieldTypes) |
return FALSE; |
return m_bNeedHightlight[nFieldType - 1]; |
} |
void CPDFSDK_InterForm::RemoveAllHighLight() { |
- memset((void*)m_bNeedHightlight, 0, 6 * sizeof(FX_BOOL)); |
+ for (int i = 0; i < kNumFieldTypes; ++i) |
+ m_bNeedHightlight[i] = FALSE; |
} |
+ |
void CPDFSDK_InterForm::SetHighlightColor(FX_COLORREF clr, int nFieldType) { |
- if (nFieldType < 0 || nFieldType > 6) |
+ if (nFieldType < 0 || nFieldType > kNumFieldTypes) |
return; |
switch (nFieldType) { |
case 0: { |
- for (int i = 0; i < 6; i++) { |
+ for (int i = 0; i < kNumFieldTypes; ++i) { |
m_aHighlightColor[i] = clr; |
m_bNeedHightlight[i] = TRUE; |
} |
@@ -2267,7 +2269,7 @@ void CPDFSDK_InterForm::SetHighlightColor(FX_COLORREF clr, int nFieldType) { |
} |
FX_COLORREF CPDFSDK_InterForm::GetHighlightColor(int nFieldType) { |
- if (nFieldType < 0 || nFieldType > 6) |
+ if (nFieldType < 0 || nFieldType > kNumFieldTypes) |
return FXSYS_RGB(255, 255, 255); |
if (nFieldType == 0) |
return m_aHighlightColor[0]; |