Subtract instead of adding the negation.

core/fpdfdoc/doc_vt.cpp

Index: core/fpdfdoc/doc_vt.cpp
diff --git a/core/fpdfdoc/doc_vt.cpp b/core/fpdfdoc/doc_vt.cpp
index b728c68af61fade85c2e699a3e35c581edb46437..20c48ef571193d479f7e86581d8c46a4be6cbb49 100644
--- a/core/fpdfdoc/doc_vt.cpp
+++ b/core/fpdfdoc/doc_vt.cpp
@@ -372,7 +372,7 @@ CPVT_FloatRect CTypeset::CharArray() {
     pLine->m_LineInfo.fLineWidth = x - pLine->m_LineInfo.fLineX;
     pLine->m_LineInfo.fLineAscent = fLineAscent;
     pLine->m_LineInfo.fLineDescent = fLineDescent;
-    y += (-fLineDescent);
+    y -= fLineDescent;
   }
   return m_rcRet = CPVT_FloatRect(0, 0, x, y);
 }
@@ -640,7 +640,7 @@ void CTypeset::SplitLines(FX_BOOL bTypeset, FX_FLOAT fFontSize) {
           m_pSection->AddLine(line);
         }
         fMaxY += (fLineAscent + m_pVT->GetLineLeading(m_pSection->m_SecInfo));
-        fMaxY += (-fLineDescent);
+        fMaxY -= fLineDescent;
         fMaxX = std::max(fLineWidth, fMaxX);
         nLineHead = i;
         fLineWidth = 0.0f;
@@ -666,7 +666,7 @@ void CTypeset::SplitLines(FX_BOOL bTypeset, FX_FLOAT fFontSize) {
         m_pSection->AddLine(line);
       }
       fMaxY += (fLineAscent + m_pVT->GetLineLeading(m_pSection->m_SecInfo));
-      fMaxY += (-fLineDescent);
+      fMaxY -= fLineDescent;
       fMaxX = std::max(fLineWidth, fMaxX);
     }
   } else {
@@ -688,8 +688,8 @@ void CTypeset::SplitLines(FX_BOOL bTypeset, FX_FLOAT fFontSize) {
       line.fLineDescent = fLineDescent;
       m_pSection->AddLine(line);
     }
-    fMaxY += (m_pVT->GetLineLeading(m_pSection->m_SecInfo) + fLineAscent +
-              (-fLineDescent));
+    fMaxY += (m_pVT->GetLineLeading(m_pSection->m_SecInfo) + fLineAscent -

[Tom Sepez, 2016/04/29 22:32:41]

nit: overparenthesized.

[Lei Zhang, 2016/04/30 01:01:14]

Done.

+              fLineDescent);
   }
   m_rcRet = CPVT_FloatRect(0, 0, fMaxX, fMaxY);
 }
@@ -761,7 +761,7 @@ void CTypeset::OutputLines() {
             fPosX += m_pVT->GetWordWidth(*pWord);
           }
         }
-        fPosY += (-pLine->m_LineInfo.fLineDescent);
+        fPosY -= pLine->m_LineInfo.fLineDescent;
       }
     }
   }