commit 44d98b424db4f89ecedbfa403e39ffca29871afc
author Lei Zhang <thestig@chromium.org> Thu Dec 20 18:03:33 2018 +0000
committer Chromium commit bot <commit-bot@chromium.org> Thu Dec 20 18:03:33 2018 +0000
tree c2eb520a68b4c0b1250c9a5e66c43d93f470f93f
parent b78a068ced745b72f26b686cf8b3985c4705bd74

Remove a useless assignment in CPDF_Parser::ParseCrossRefV4().

Fix the variable name too.

Change-Id: I8e3fe9ba03dd9a14dca5089f5450220fc0b8ff56
Reviewed-on: https://pdfium-review.googlesource.com/c/47552
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>

core/fpdfapi/parser/cpdf_parser.cpp

diff --git a/core/fpdfapi/parser/cpdf_parser.cpp b/core/fpdfapi/parser/cpdf_parser.cpp
index dd885bc..5ddae2d 100644
--- a/core/fpdfapi/parser/cpdf_parser.cpp
+++ b/core/fpdfapi/parser/cpdf_parser.cpp
@@ -507,15 +507,14 @@
     return false;
   std::vector<CrossRefObjData> result_objects;
   while (1) {
-    FX_FILESIZE SavedPos = m_pSyntax->GetPos();
+    FX_FILESIZE saved_pos = m_pSyntax->GetPos();
     bool bIsNumber;
     ByteString word = m_pSyntax->GetNextWord(&bIsNumber);
-    if (word.IsEmpty()) {
+    if (word.IsEmpty())
       return false;
-    }
 
     if (!bIsNumber) {
-      m_pSyntax->SetPos(SavedPos);
+      m_pSyntax->SetPos(saved_pos);
       break;
     }
 
@@ -525,7 +524,6 @@
 
     uint32_t count = m_pSyntax->GetDirectNum();
     m_pSyntax->ToNextWord();
-    SavedPos = m_pSyntax->GetPos();
 
     if (!ParseAndAppendCrossRefSubsectionData(
             start_objnum, count, out_objects ? &result_objects : nullptr)) {