From 007e6c0f9559412788b903bcb6a7601e220c3be8 Mon Sep 17 00:00:00 2001 From: Tom Sepez Date: Fri, 26 Feb 2016 14:31:56 -0800 Subject: Remove some FX_BOOLs Grepping for FX_BOOL and |==| on the same line gives a very strong clue that the expression won't be out of range of the |bool| type iteself. R=thestig@chromium.org Review URL: https://codereview.chromium.org/1736323003 . --- fpdfsdk/include/fsdk_baseannot.h | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) (limited to 'fpdfsdk/include/fsdk_baseannot.h') diff --git a/fpdfsdk/include/fsdk_baseannot.h b/fpdfsdk/include/fsdk_baseannot.h index c0ed529782..8cee331f64 100644 --- a/fpdfsdk/include/fsdk_baseannot.h +++ b/fpdfsdk/include/fsdk_baseannot.h @@ -36,19 +36,19 @@ class CPDFSDK_DateTime { CPDFSDK_DateTime& operator=(const CPDFSDK_DateTime& datetime); CPDFSDK_DateTime& operator=(const FX_SYSTEMTIME& st); - FX_BOOL operator==(CPDFSDK_DateTime& datetime); - FX_BOOL operator!=(CPDFSDK_DateTime& datetime); - FX_BOOL operator>(CPDFSDK_DateTime& datetime); - FX_BOOL operator>=(CPDFSDK_DateTime& datetime); - FX_BOOL operator<(CPDFSDK_DateTime& datetime); - FX_BOOL operator<=(CPDFSDK_DateTime& datetime); + bool operator==(const CPDFSDK_DateTime& datetime) const; + bool operator!=(const CPDFSDK_DateTime& datetime) const; + bool operator>(const CPDFSDK_DateTime& datetime) const; + bool operator>=(const CPDFSDK_DateTime& datetime) const; + bool operator<(const CPDFSDK_DateTime& datetime) const; + bool operator<=(const CPDFSDK_DateTime& datetime) const; operator time_t(); CPDFSDK_DateTime& FromPDFDateTimeString(const CFX_ByteString& dtStr); CFX_ByteString ToCommonDateTimeString(); CFX_ByteString ToPDFDateTimeString(); void ToSystemTime(FX_SYSTEMTIME& st); - CPDFSDK_DateTime ToGMT(); + CPDFSDK_DateTime ToGMT() const; CPDFSDK_DateTime& AddDays(short days); CPDFSDK_DateTime& AddSeconds(int seconds); -- cgit v1.2.3