diff options
| author | Tom Sepez <tsepez@chromium.org> | 2018-06-12 20:49:55 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2018-06-12 20:49:55 +0000 |
| commit | 62d8ebeee58c5da29102e61d25b28967a1b2aa04 (patch) | |
| tree | 0889b83248921397be31899fcc7e860a075bfb07 /fpdfsdk | |
| parent | 6eb7939300d1bc7c31afd5086c1b93d4a7628481 (diff) | |
| download | pdfium-62d8ebeee58c5da29102e61d25b28967a1b2aa04.tar.xz | |
Comment some more #else // PDF_ENABLE_XFA
This is an anti-pattern when XFA can be configured at runtime.
Make it easier to find some more remaining cases.
Change-Id: I1ee8d95b2fd52be5e37ef4e859e3e9179ea25975
Reviewed-on: https://pdfium-review.googlesource.com/34990
Commit-Queue: Tom Sepez <tsepez@chromium.org>
Reviewed-by: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'fpdfsdk')
| -rw-r--r-- | fpdfsdk/cpdfsdk_widget.cpp | 2 | ||||
| -rw-r--r-- | fpdfsdk/cpdfsdk_widget.h | 2 | ||||
| -rw-r--r-- | fpdfsdk/fpdf_view_embeddertest.cpp | 4 |
3 files changed, 4 insertions, 4 deletions
diff --git a/fpdfsdk/cpdfsdk_widget.cpp b/fpdfsdk/cpdfsdk_widget.cpp index 40e04e8772..cd863bda2c 100644 --- a/fpdfsdk/cpdfsdk_widget.cpp +++ b/fpdfsdk/cpdfsdk_widget.cpp @@ -465,7 +465,7 @@ WideString CPDFSDK_Widget::GetValue(bool bDisplay) const { : XFA_VALUEPICTURE_Edit); } } -#else +#else // PDF_ENABLE_XFA WideString CPDFSDK_Widget::GetValue() const { #endif // PDF_ENABLE_XFA CPDF_FormField* pFormField = GetFormField(); diff --git a/fpdfsdk/cpdfsdk_widget.h b/fpdfsdk/cpdfsdk_widget.h index 43482ea7d5..4d85facb7d 100644 --- a/fpdfsdk/cpdfsdk_widget.h +++ b/fpdfsdk/cpdfsdk_widget.h @@ -72,7 +72,7 @@ class CPDFSDK_Widget : public CPDFSDK_BAAnnot { int GetSelectedIndex(int nIndex) const; #ifdef PDF_ENABLE_XFA WideString GetValue(bool bDisplay = true) const; -#else +#else // PDF_ENABLE_XFA WideString GetValue() const; #endif // PDF_ENABLE_XFA WideString GetDefaultValue() const; diff --git a/fpdfsdk/fpdf_view_embeddertest.cpp b/fpdfsdk/fpdf_view_embeddertest.cpp index cef5e9a0d4..1eedda3d4b 100644 --- a/fpdfsdk/fpdf_view_embeddertest.cpp +++ b/fpdfsdk/fpdf_view_embeddertest.cpp @@ -79,9 +79,9 @@ TEST_F(FPDFViewEmbeddertest, EmptyDocument) { { #ifdef PDF_ENABLE_XFA const unsigned long kExpected = static_cast<uint32_t>(-1); -#else +#else // PDF_ENABLE_XFA const unsigned long kExpected = 0; -#endif +#endif // PDF_ENABLE_XFA EXPECT_EQ(kExpected, FPDF_GetDocPermissions(document())); } |