From 1bb23d0545dfe2d651cb22b6135d99c1c9ef85d5 Mon Sep 17 00:00:00 2001
From: m-holger <m-holger@kubitscheck.org>
Date: Sat, 10 Dec 2022 19:48:07 +0000
Subject: Refactor QPDF_Array::insertItem and rename to insert

---
 libqpdf/QPDF_Array.cc | 31 ++++++++++++-------------------
 1 file changed, 12 insertions(+), 19 deletions(-)

(limited to 'libqpdf/QPDF_Array.cc')

diff --git a/libqpdf/QPDF_Array.cc b/libqpdf/QPDF_Array.cc
index 5a80cdf5..7633266e 100644
--- a/libqpdf/QPDF_Array.cc
+++ b/libqpdf/QPDF_Array.cc
@@ -236,31 +236,24 @@ QPDF_Array::setFromVector(std::vector<std::shared_ptr<QPDFObject>>&& v)
     }
 }
 
-void
-QPDF_Array::insertItem(int at, QPDFObjectHandle const& item)
+bool
+QPDF_Array::insert(int at, QPDFObjectHandle const& item)
 {
-    if (sparse) {
+    int sz = size();
+    if (at < 0 || at > sz) {
         // As special case, also allow insert beyond the end
-        if ((at < 0) || (at > sp_elements.size())) {
-            throw std::logic_error(
-                "INTERNAL ERROR: bounds error accessing QPDF_Array element");
-        }
-        sp_elements.insert(at, item);
+        return false;
+    } else if (at == sz) {
+        push_back(item);
     } else {
-        // As special case, also allow insert beyond the end
-        size_t idx = QIntC::to_size(at);
-        if ((at < 0) || (at > QIntC::to_int(elements.size()))) {
-            throw std::logic_error(
-                "INTERNAL ERROR: bounds error accessing QPDF_Array element");
-        }
-        if (idx == elements.size()) {
-            // Allow inserting to the last position
-            elements.push_back(item.getObj());
+        checkOwnership(item);
+        if (sparse) {
+            sp_elements.insert(at, item);
         } else {
-            int n = int(idx);
-            elements.insert(elements.cbegin() + n, item.getObj());
+            elements.insert(elements.cbegin() + at, item.getObj());
         }
     }
+    return true;
 }
 
 void
-- 
cgit v1.2.3-70-g09d2