summaryrefslogtreecommitdiff
path: root/xfa/fde/css
diff options
context:
space:
mode:
authortsepez <tsepez@chromium.org>2016-10-25 14:36:49 -0700
committerCommit bot <commit-bot@chromium.org>2016-10-25 14:36:49 -0700
commit530c7acf7aa11ff8db8743af0d62b39cf36bf539 (patch)
tree7144c7cbaa3c565f3a2c310cb3434edac59123fa /xfa/fde/css
parentcfcc11e53a60b3f56a9bbe13718ecb9b997384da (diff)
downloadpdfium-530c7acf7aa11ff8db8743af0d62b39cf36bf539.tar.xz
Avoid dubious use of comma operator in fde_csssyntax.h
Review-Url: https://codereview.chromium.org/2446113005
Diffstat (limited to 'xfa/fde/css')
-rw-r--r--xfa/fde/css/fde_csssyntax.h10
1 files changed, 5 insertions, 5 deletions
diff --git a/xfa/fde/css/fde_csssyntax.h b/xfa/fde/css/fde_csssyntax.h
index 294dfcb540..af1d444144 100644
--- a/xfa/fde/css/fde_csssyntax.h
+++ b/xfa/fde/css/fde_csssyntax.h
@@ -22,11 +22,11 @@ class CFDE_CSSTextBuf : public CFX_Target {
int32_t iStreamOffset,
int32_t iMaxChars,
FX_BOOL& bEOS);
- FX_BOOL AppendChar(FX_WCHAR wch) {
- if (m_iDatLen >= m_iBufLen && !ExpandBuf(m_iBufLen * 2)) {
- return FALSE;
- }
- return (m_pBuffer[m_iDatLen++] = wch), TRUE;
+ bool AppendChar(FX_WCHAR wch) {
+ if (m_iDatLen >= m_iBufLen && !ExpandBuf(m_iBufLen * 2))
+ return false;
+ m_pBuffer[m_iDatLen++] = wch;
+ return true;
}
void Clear() { m_iDatPos = m_iDatLen = 0; }
void Reset();