From c9a9fe9c2f8837bdfdd2b0e58fce9a82b0a64881 Mon Sep 17 00:00:00 2001
From: Jay Berkenbilt <ejb@ql.org>
Date: Thu, 26 Dec 2013 11:51:50 -0500
Subject: Avoid traversing same object twice when copying objects

This is a performance fix.  The output is unchanged.

Fixes #28.
---
 libqpdf/QPDF.cc | 5 +++++
 1 file changed, 5 insertions(+)

(limited to 'libqpdf')

diff --git a/libqpdf/QPDF.cc b/libqpdf/QPDF.cc
index 543b34b7..51f39abd 100644
--- a/libqpdf/QPDF.cc
+++ b/libqpdf/QPDF.cc
@@ -1755,6 +1755,11 @@ QPDF::reserveObjects(QPDFObjectHandle foreign, ObjCopier& obj_copier,
             QTC::TC("qpdf", "QPDF loop reserving objects");
             return;
         }
+        if (obj_copier.object_map.find(foreign_og) != obj_copier.object_map.end())
+        {
+            QTC::TC("qpdf", "QPDF already reserved object");
+            return;
+        }
         QTC::TC("qpdf", "QPDF copy indirect");
         obj_copier.visiting.insert(foreign_og);
         std::map<QPDFObjGen, QPDFObjectHandle>::iterator mapping =
-- 
cgit v1.2.3-54-g00ecf