Refactor `QPDFPageLabelDocumentHelper::Members` to use `std::unique_ptr`, simpli…

…fy logic, and enhance consistency with modern C++ practices.

Refactor `QPDFPageLabelDocumentHelper::Members` to use `std::unique_ptr`, simpli…
…fy logic, and enhance consistency with modern C++ practices.
m-holger
1 parent b00647d5
Showing 3 changed files with 22 additions and 30 deletions
include/qpdf/QPDFPageLabelDocumentHelper.hh
libqpdf/QPDFPageLabelDocumentHelper.cc
qpdf/qpdf.testcov
@@ -77,19 +77,7 @@ class QPDFPageLabelDocumentHelper: public QPDFDocumentHelper
         std::vector<QPDFObjectHandle>& new_labels);
   private:
-    class Members
-    {
-        friend class QPDFPageLabelDocumentHelper;
-
-      public:
-        ~Members() = default;
-
-      private:
-        Members() = default;
-        Members(Members const&) = delete;
-
-        std::shared_ptr<QPDFNumberTreeObjectHelper> labels;
-    };
+    class Members;
     std::shared_ptr<Members> m;
 };
@@ -2,37 +2,43 @@
 #include <qpdf/QTC.hh>
+class QPDFPageLabelDocumentHelper::Members
+{
+  public:
+    Members() = default;
+    Members(Members const&) = delete;
+    ~Members() = default;
+
+    std::unique_ptr<QPDFNumberTreeObjectHelper> labels;
+};
+
 QPDFPageLabelDocumentHelper::QPDFPageLabelDocumentHelper(QPDF& qpdf) :
     QPDFDocumentHelper(qpdf),
-    m(new Members())
+    m(std::make_shared<Members>())
 {
     QPDFObjectHandle root = qpdf.getRoot();
     if (root.hasKey("/PageLabels")) {
         m->labels =
-            std::make_shared<QPDFNumberTreeObjectHelper>(root.getKey("/PageLabels"), this->qpdf);
+            std::make_unique<QPDFNumberTreeObjectHelper>(root.getKey("/PageLabels"), this->qpdf);
     }
 }
 bool
 QPDFPageLabelDocumentHelper::hasPageLabels()
 {
-    return nullptr != m->labels;
+    return m->labels != nullptr;
 }
 QPDFObjectHandle
 QPDFPageLabelDocumentHelper::getLabelForPage(long long page_idx)
 {
-    QPDFObjectHandle result(QPDFObjectHandle::newNull());
     if (!hasPageLabels()) {
-        return result;
+        return QPDFObjectHandle::newNull();
     }
     QPDFNumberTreeObjectHelper::numtree_number offset = 0;
     QPDFObjectHandle label;
-    if (!m->labels->findObjectAtOrBelow(page_idx, label, offset)) {
-        return result;
-    }
-    if (!label.isDictionary()) {
-        return result;
+    if (!m->labels->findObjectAtOrBelow(page_idx, label, offset) || !label.isDictionary()) {
+        return QPDFObjectHandle::newNull();
     }
     QPDFObjectHandle S = label.getKey("/S");   // type (D, R, r, A, a)
     QPDFObjectHandle P = label.getKey("/P");   // prefix
@@ -43,7 +49,7 @@ QPDFPageLabelDocumentHelper::getLabelForPage(long long page_idx)
     }
     QIntC::range_check(start, offset);
     start += offset;
-    result = QPDFObjectHandle::newDictionary();
+    auto result = QPDFObjectHandle::newDictionary();
     result.replaceKey("/S", S);
     result.replaceKey("/P", P);
     result.replaceKey("/St", QPDFObjectHandle::newInteger(start));
@@ -81,21 +87,20 @@ QPDFPageLabelDocumentHelper::getLabelsForPageRange(
                 label.getKey("/St").getIntValue() - last.getKey("/St").getIntValue();
             long long int idx_delta = new_start_idx - last_idx.getIntValue();
             if (st_delta == idx_delta) {
-                QTC::TC("qpdf", "QPDFPageLabelDocumentHelper skip first");
                 skip_first = true;
             }
         }
     }
     if (!skip_first) {
-        new_labels.push_back(QPDFObjectHandle::newInteger(new_start_idx));
-        new_labels.push_back(label);
+        new_labels.emplace_back(QPDFObjectHandle::newInteger(new_start_idx));
+        new_labels.emplace_back(label);
     }
     long long int idx_offset = new_start_idx - start_idx;
     for (long long i = start_idx + 1; i <= end_idx; ++i) {
         if (m->labels->hasIndex(i) && (label = getLabelForPage(i)).isDictionary()) {
-            new_labels.push_back(QPDFObjectHandle::newInteger(i + idx_offset));
-            new_labels.push_back(label);
+            new_labels.emplace_back(QPDFObjectHandle::newInteger(i + idx_offset));
+            new_labels.emplace_back(label);
         }
     }
 }
@@ -338,7 +338,6 @@ QPDFAcroFormDocumentHelper annotation found 1
 QPDFJob keep files open n 0
 QPDFJob keep files open y 0
 QPDFJob automatically set keep files open 1
-QPDFPageLabelDocumentHelper skip first 0
 QPDFOutlineObjectHelper direct dest 0
 QPDFOutlineObjectHelper action dest 0
 QPDFOutlineObjectHelper named dest 0