| 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];
|
|
|