commit 530c7acf7aa11ff8db8743af0d62b39cf36bf539
author tsepez <tsepez@chromium.org> Tue Oct 25 14:36:49 2016 -0700
committer Commit bot <commit-bot@chromium.org> Tue Oct 25 14:36:49 2016 -0700
tree 7144c7cbaa3c565f3a2c310cb3434edac59123fa
parent cfcc11e53a60b3f56a9bbe13718ecb9b997384da

Avoid dubious use of comma operator in fde_csssyntax.h

Review-Url: https://codereview.chromium.org/2446113005

xfa/fde/css/fde_csssyntax.h

diff --git a/xfa/fde/css/fde_csssyntax.h b/xfa/fde/css/fde_csssyntax.h
index 294dfcb..af1d444 100644
--- a/xfa/fde/css/fde_csssyntax.h
+++ b/xfa/fde/css/fde_csssyntax.h
@@ -22,11 +22,11 @@
                          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();