Refactor null object handling across helpers to use the `Null` class for consist…

…ent and simplified temporary null management. Clean up unused test coverage markers.

Refactor null object handling across helpers to use the `Null` class for consist…
…ent and simplified temporary null management. Clean up unused test coverage markers.
m-holger
1 parent 4181ec92
Showing 5 changed files with 10 additions and 19 deletions
libqpdf/QPDFAcroFormDocumentHelper.cc
libqpdf/QPDFAnnotationObjectHelper.cc
libqpdf/QPDFEmbeddedFileDocumentHelper.cc
libqpdf/QPDFFileSpecObjectHelper.cc
qpdf/qpdf.testcov
@@ -252,16 +252,15 @@ QPDFFormFieldObjectHelper
 QPDFAcroFormDocumentHelper::getFieldForAnnotation(QPDFAnnotationObjectHelper h)
 {
     QPDFObjectHandle oh = h.getObjectHandle();
-    QPDFFormFieldObjectHelper result(QPDFObjectHandle::newNull());
     if (!oh.isDictionaryOfType("", "/Widget")) {
-        return result;
+        return Null::temp();
     }
     analyze();
     QPDFObjGen og(oh.getObjGen());
     if (m->annotation_to_field.contains(og)) {
-        result = m->annotation_to_field[og];
+        return m->annotation_to_field[og];
     }
-    return result;
+    return Null::temp();
 }
 void
@@ -56,20 +56,16 @@ QPDFAnnotationObjectHelper::getAppearanceStream(std::string const&amp; which, std::s
             // appearance stream when /AP is a dictionary, but files have been seen in the wild
             // where Appearance State is `/N` and `/AP` is a stream. Therefore, if `which` points to
             // a stream, disregard state and just use the stream. See qpdf issue #949 for details.
-            QTC::TC("qpdf", "QPDFAnnotationObjectHelper AP stream");
             return ap_sub;
         }
-        if (ap_sub.isDictionary() && (!desired_state.empty())) {
-            QTC::TC("qpdf", "QPDFAnnotationObjectHelper AP dictionary");
+        if (ap_sub.isDictionary() && !desired_state.empty()) {
             QPDFObjectHandle ap_sub_val = ap_sub.getKey(desired_state);
             if (ap_sub_val.isStream()) {
-                QTC::TC("qpdf", "QPDFAnnotationObjectHelper AP sub stream");
                 return ap_sub_val;
             }
         }
     }
-    QTC::TC("qpdf", "QPDFAnnotationObjectHelper AP null");
-    return QPDFObjectHandle::newNull();
+    return Null::temp();
 }
 std::string
 #include <qpdf/QPDFEmbeddedFileDocumentHelper.hh>
 #include <qpdf/QPDFNameTreeObjectHelper.hh>
+#include <qpdf/QPDFObjectHandle_private.hh>
 #include <qpdf/QPDF_private.hh>
 // File attachments are stored in the /EmbeddedFiles (name tree) key of the /Names dictionary from
@@ -143,11 +144,10 @@ QPDFEmbeddedFileDocumentHelper::removeEmbeddedFile(std::string const&amp; name)
     if (iter == m->embedded_files->end()) {
         return false;
     }
-    auto oh = iter->second;
-    iter.remove();
-    if (oh.isIndirect()) {
-        qpdf.replaceObject(oh.getObjGen(), QPDFObjectHandle::newNull());
+    if (iter->second.indirect()) {
+        qpdf.replaceObject(iter->second, Null());
     }
+    iter.remove();
     return true;
 }
@@ -76,7 +76,7 @@ QPDFFileSpecObjectHelper::getEmbeddedFileStream(std::string const&amp; key)
             }
         }
     }
-    return QPDFObjectHandle::newNull();
+    return Null::temp();
 }
 QPDFObjectHandle
@@ -181,10 +181,6 @@ QPDFFormFieldObjectHelper TU absent 0
 QPDFFormFieldObjectHelper TM absent 0
 QPDFFormFieldObjectHelper Q present 1
 QPDFFormFieldObjectHelper DA present 1
-QPDFAnnotationObjectHelper AP stream 0
-QPDFAnnotationObjectHelper AP dictionary 0
-QPDFAnnotationObjectHelper AP sub stream 0
-QPDFAnnotationObjectHelper AP null 0
 QPDFAcroFormDocumentHelper field found 1
 QPDFAcroFormDocumentHelper annotation found 1
 QPDFJob automatically set keep files open 1