Chromium Code Reviews| Index: xfa/src/fdp/src/css/fde_cssdatatable.cpp |
| diff --git a/xfa/src/fdp/src/css/fde_cssdatatable.cpp b/xfa/src/fdp/src/css/fde_cssdatatable.cpp |
| index 28b925dda21f34e7382ddc3f3e60becc251e48d4..a355433eeff8f3581355b8ffb128824c661babe0 100644 |
| --- a/xfa/src/fdp/src/css/fde_cssdatatable.cpp |
| +++ b/xfa/src/fdp/src/css/fde_cssdatatable.cpp |
| @@ -843,16 +843,18 @@ int32_t CFDE_CSSValueListParser::SkipTo(FX_WCHAR wch, |
| FX_BOOL bBrContinue) { |
| const FX_WCHAR* pStart = m_pCur; |
| if (!bBrContinue) { |
| - if (bWSSeparator) |
| - while (++m_pCur<m_pEnd&& * m_pCur != wch&& * m_pCur> ' ') |
| - ; |
| - else |
| - while (++m_pCur < m_pEnd && *m_pCur != wch) |
| - ; |
| + if (bWSSeparator) { |
| + while ((++m_pCur < m_pEnd) && (*m_pCur != wch) && (*m_pCur > ' ')) { |
|
dsinclair
2016/02/23 15:53:59
I added ()'s to this to keep git cl format from do
Tom Sepez
2016/02/23 17:44:19
Really? That's bad. I've not seen it do that befo
dsinclair
2016/02/23 19:14:01
Acknowledged.
|
| + } |
| + } else { |
| + while (++m_pCur < m_pEnd && *m_pCur != wch) { |
| + } |
| + } |
| + |
| } else { |
| int32_t iBracketCount = 0; |
| if (bWSSeparator) { |
| - while (m_pCur<m_pEnd&& * m_pCur != wch&& * m_pCur> ' ') { |
| + while ((m_pCur < m_pEnd) && (*m_pCur != wch) && (*m_pCur > ' ')) { |
| if (*m_pCur == '(') { |
| iBracketCount++; |
| } else if (*m_pCur == ')') { |