Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(166)

Unified Diff: xfa/src/fdp/src/css/fde_cssdatatable.cpp

Issue 1730553002: Fixing whitespace lint errors. (Closed) Base URL: https://pdfium.googlesource.com/pdfium.git@master
Patch Set: Created 4 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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 == ')') {

Powered by Google App Engine
This is Rietveld 408576698