diff options
| author | Tom Sepez <tsepez@chromium.org> | 2017-09-15 15:43:11 -0700 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2017-09-15 22:54:34 +0000 |
| commit | 7b1e53c0ef09da360304c6f4397082d09a945a7e (patch) | |
| tree | 0167576535814d271950c4aea24071c4e7b6afce /fpdfsdk/cpdfsdk_widget.h | |
| parent | 134ac9105586407eb3b1e06001101ff893dd4a31 (diff) | |
| download | pdfium-7b1e53c0ef09da360304c6f4397082d09a945a7e.tar.xz | |
Use unsigned types for app age, value age in widgets.chromium/3218chromium/3217
Then if they roll over, it doesn't matter, since we only check for change.
And then we can pull a silly check. Then remove some no-op calls where we
didn't use the result.
Change-Id: I35ba470b42fb8c32a6984999e0311b21729791ca
Reviewed-on: https://pdfium-review.googlesource.com/14210
Commit-Queue: Tom Sepez <tsepez@chromium.org>
Reviewed-by: Lei Zhang <thestig@chromium.org>
Diffstat (limited to 'fpdfsdk/cpdfsdk_widget.h')
| -rw-r--r-- | fpdfsdk/cpdfsdk_widget.h | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/fpdfsdk/cpdfsdk_widget.h b/fpdfsdk/cpdfsdk_widget.h index 5c63400443..d33413afae 100644 --- a/fpdfsdk/cpdfsdk_widget.h +++ b/fpdfsdk/cpdfsdk_widget.h @@ -130,8 +130,8 @@ class CPDFSDK_Widget : public CPDFSDK_BAAnnot { void ClearAppModified(); bool IsAppModified() const; - int32_t GetAppearanceAge() const; - int32_t GetValueAge() const; + uint32_t GetAppearanceAge() const { return m_nAppearanceAge; } + uint32_t GetValueAge() const { return m_nValueAge; } bool IsWidgetAppearanceValid(CPDF_Annot::AppearanceMode mode); void DrawAppearance(CFX_RenderDevice* pDevice, @@ -149,8 +149,8 @@ class CPDFSDK_Widget : public CPDFSDK_BAAnnot { private: CFX_UnownedPtr<CPDFSDK_InterForm> const m_pInterForm; bool m_bAppModified; - int32_t m_nAppAge; - int32_t m_nValueAge; + uint32_t m_nAppearanceAge; + uint32_t m_nValueAge; #ifdef PDF_ENABLE_XFA mutable CFX_UnownedPtr<CXFA_FFWidget> m_hMixXFAWidget; |