diff options
| author | tsepez <tsepez@chromium.org> | 2016-10-25 14:36:49 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2016-10-25 14:36:49 -0700 |
| commit | 530c7acf7aa11ff8db8743af0d62b39cf36bf539 (patch) | |
| tree | 7144c7cbaa3c565f3a2c310cb3434edac59123fa /xfa/fde/css/fde_csssyntax.h | |
| parent | cfcc11e53a60b3f56a9bbe13718ecb9b997384da (diff) | |
| download | pdfium-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/fde_csssyntax.h')
| -rw-r--r-- | xfa/fde/css/fde_csssyntax.h | 10 |
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(); |