commit ad9ec040ef2832bf5e22140022887f228e20a3fb
author Lei Zhang <thestig@chromium.org> Wed Oct 05 22:41:21 2022 +0000
committer Pdfium LUCI CQ <pdfium-scoped@luci-project-accounts.iam.gserviceaccount.com> Wed Oct 05 22:41:21 2022 +0000
tree 1338dc8bb72d61519f6d306b2c356b8680e43164
parent 8176850eb13ae2ea3bfbf4adf6e59cb863f37c2d

Delete unused CPDFSDK_AnnotReverseIteration.

Remove CPDFSDK_AnnotReverseIteration since it has no users now. Fold
CPDFSDK_AnnotForwardIteration into CPDFSDK_AnnotIteration.

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

fpdfsdk/cpdfsdk_annotiteration.cpp

diff --git a/fpdfsdk/cpdfsdk_annotiteration.cpp b/fpdfsdk/cpdfsdk_annotiteration.cpp
index 0869d0e..aa736bb 100644
--- a/fpdfsdk/cpdfsdk_annotiteration.cpp
+++ b/fpdfsdk/cpdfsdk_annotiteration.cpp
@@ -7,46 +7,30 @@
 #include "fpdfsdk/cpdfsdk_annotiteration.h"
 
 #include <algorithm>
-#include <utility>
 
 #include "fpdfsdk/cpdfsdk_annot.h"
 #include "fpdfsdk/cpdfsdk_pageview.h"
 
-CPDFSDK_AnnotIteration::CPDFSDK_AnnotIteration(CPDFSDK_PageView* pPageView,
-                                               bool bReverse) {
-  // Copying/sorting ObservedPtrs is expensive, so do it once at the end.
-  std::vector<CPDFSDK_Annot*> copiedList = pPageView->GetAnnotList();
-  std::stable_sort(copiedList.begin(), copiedList.end(),
+CPDFSDK_AnnotIteration::CPDFSDK_AnnotIteration(CPDFSDK_PageView* pPageView) {
+  // Copying ObservedPtrs is expensive, so do it once at the end.
+  std::vector<CPDFSDK_Annot*> copied_list = pPageView->GetAnnotList();
+  std::stable_sort(copied_list.begin(), copied_list.end(),
                    [](const CPDFSDK_Annot* p1, const CPDFSDK_Annot* p2) {
                      return p1->GetLayoutOrder() < p2->GetLayoutOrder();
                    });
 
   CPDFSDK_Annot* pTopMostAnnot = pPageView->GetFocusAnnot();
   if (pTopMostAnnot) {
-    auto it = std::find(copiedList.begin(), copiedList.end(), pTopMostAnnot);
-    if (it != copiedList.end()) {
-      copiedList.erase(it);
-      copiedList.insert(copiedList.begin(), pTopMostAnnot);
+    auto it = std::find(copied_list.begin(), copied_list.end(), pTopMostAnnot);
+    if (it != copied_list.end()) {
+      copied_list.erase(it);
+      copied_list.insert(copied_list.begin(), pTopMostAnnot);
     }
   }
-  if (bReverse)
-    std::reverse(copiedList.begin(), copiedList.end());
 
-  m_List.reserve(copiedList.size());
-  for (auto* pAnnot : copiedList)
+  m_List.reserve(copied_list.size());
+  for (auto* pAnnot : copied_list)
     m_List.emplace_back(pAnnot);
 }
 
 CPDFSDK_AnnotIteration::~CPDFSDK_AnnotIteration() = default;
-
-CPDFSDK_AnnotForwardIteration::CPDFSDK_AnnotForwardIteration(
-    CPDFSDK_PageView* pPageView)
-    : CPDFSDK_AnnotIteration(pPageView, /*bReverse=*/false) {}
-
-CPDFSDK_AnnotForwardIteration::~CPDFSDK_AnnotForwardIteration() = default;
-
-CPDFSDK_AnnotReverseIteration::CPDFSDK_AnnotReverseIteration(
-    CPDFSDK_PageView* pPageView)
-    : CPDFSDK_AnnotIteration(pPageView, /*bReverse=*/true) {}
-
-CPDFSDK_AnnotReverseIteration::~CPDFSDK_AnnotReverseIteration() = default;