Avoid dubious use of comma operator in fde_csssyntax.h Review-Url: https://codereview.chromium.org/2446113005
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();