Add method QPDFObject::getObjGen and remove QPDFObjectHandle::og

m-holger
1 parent 5033e3b2
Showing 3 changed files with 9 additions and 7 deletions
include/qpdf/QPDFObject.hh
include/qpdf/QPDFObjectHandle.hh
libqpdf/QPDFObjectHandle.cc
@@ -103,6 +103,12 @@ class QPDFObject
     {
         return value->qpdf;
     }
+    QPDFObjGen
+    getObjGen() const
+    {
+        return value->og;
+    }
+
     void
     setDescription(QPDF* qpdf, std::string const& description)
     {
@@ -1584,7 +1584,6 @@ class QPDFObjectHandle
         QPDF* qpdf,
         QPDFObjGen const& og,
         std::shared_ptr<QPDFObject> const& obj) :
-        og(og),
         obj(obj)
     {
     }
@@ -1640,7 +1639,6 @@ class QPDFObjectHandle
     // Moving members of QPDFObjectHandle into a smart pointer incurs
     // a substantial performance penalty since QPDFObjectHandle
     // objects are copied around so frequently.
-    QPDFObjGen og;
     std::shared_ptr<QPDFObject> obj;
 };
@@ -1864,19 +1862,19 @@ class QPDFObjectHandle::QPDFArrayItems
 inline QPDFObjGen
 QPDFObjectHandle::getObjGen() const
 {
-    return og;
+    return isInitialized() ? obj->getObjGen() : QPDFObjGen();
 }
 inline int
 QPDFObjectHandle::getObjectID() const
 {
-    return og.getObj();
+    return getObjGen().getObj();
 }
 inline int
 QPDFObjectHandle::getGeneration() const
 {
-    return og.getGen();
+    return getObjGen().getGen();
 }
 inline bool
@@ -2282,8 +2282,6 @@ QPDFObjectHandle::copyObject(
                                " reserved object handle direct");
     }
-    og = QPDFObjGen();
-
     std::shared_ptr<QPDFObject> new_obj;
     if (isBool() || isInteger() || isName() || isNull() || isReal() ||