diff options
author | Jay Berkenbilt <ejb@ql.org> | 2019-08-22 23:53:20 +0200 |
---|---|---|
committer | Jay Berkenbilt <ejb@ql.org> | 2019-08-22 23:55:16 +0200 |
commit | 4b2e72c4cd7dc9bc17ad78ca983ff884c1e1ee69 (patch) | |
tree | af9dc8355ef29d6fd96a771ad9bb9de7f6ce70b6 /libqpdf/SparseOHArray.cc | |
parent | 3f3dbe22eaae56df48af304c35a1c0b5dd976135 (diff) | |
download | qpdf-4b2e72c4cd7dc9bc17ad78ca983ff884c1e1ee69.tar.zst |
Test for direct, rather than resolved nulls in parser
Just because we know an indirect reference is null, doesn't mean we
shouldn't keep it indirect.
Diffstat (limited to 'libqpdf/SparseOHArray.cc')
-rw-r--r-- | libqpdf/SparseOHArray.cc | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/libqpdf/SparseOHArray.cc b/libqpdf/SparseOHArray.cc index 76c7c224..05b44ee0 100644 --- a/libqpdf/SparseOHArray.cc +++ b/libqpdf/SparseOHArray.cc @@ -15,7 +15,7 @@ SparseOHArray::size() const void SparseOHArray::append(QPDFObjectHandle oh) { - if (! oh.isResolvedNull()) + if (! oh.isDirectNull()) { this->elements[this->n_elements] = oh; } @@ -73,7 +73,7 @@ SparseOHArray::setAt(size_t idx, QPDFObjectHandle oh) { throw std::logic_error("bounds error setting item in SparseOHArray"); } - if (oh.isResolvedNull()) + if (oh.isDirectNull()) { this->elements.erase(idx); } |