diff options
author | m-holger <m-holger@kubitscheck.org> | 2022-05-21 16:18:15 +0200 |
---|---|---|
committer | Jay Berkenbilt <jberkenbilt@users.noreply.github.com> | 2022-05-21 22:06:29 +0200 |
commit | 6c69a747b9f7a801be2ad58985f35886bd38239e (patch) | |
tree | a35f002d4564ad1811044335c34f1b510a6774b8 /qpdf/test_parsedoffset.cc | |
parent | 70ccd807c45f477d6caf73b9390ba3acda53d1db (diff) | |
download | qpdf-6c69a747b9f7a801be2ad58985f35886bd38239e.tar.zst |
Code clean up: use range-style for loops wherever possible
Remove variables obsoleted by commit 4f24617.
Diffstat (limited to 'qpdf/test_parsedoffset.cc')
-rw-r--r-- | qpdf/test_parsedoffset.cc | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/qpdf/test_parsedoffset.cc b/qpdf/test_parsedoffset.cc index 26a50875..b822b0e3 100644 --- a/qpdf/test_parsedoffset.cc +++ b/qpdf/test_parsedoffset.cc @@ -51,8 +51,7 @@ walk( result[stream_number].push_back(p); if (obj.isArray()) { - std::vector<QPDFObjectHandle> array = obj.getArrayAsVector(); - for (auto& oh: array) { + for (auto& oh: obj.getArrayAsVector()) { if (!oh.isIndirect()) { // QPDF::GetAllObjects() enumerates all indirect objects. // So only the direct objects are recursed here. @@ -60,8 +59,7 @@ walk( } } } else if (obj.isDictionary()) { - std::set<std::string> keys = obj.getKeys(); - for (auto const& key: keys) { + for (auto const& key: obj.getKeys()) { QPDFObjectHandle item = obj.getKey(key); if (!item.isIndirect()) { // QPDF::GetAllObjects() enumerates all indirect objects. @@ -81,10 +79,9 @@ process( { QPDF qpdf; qpdf.processFile(fn.c_str()); - std::vector<QPDFObjectHandle> objs = qpdf.getAllObjects(); std::map<QPDFObjGen, QPDFXRefEntry> xrefs = qpdf.getXRefTable(); - for (auto const& oh: objs) { + for (auto const& oh: qpdf.getAllObjects()) { if (xrefs.count(oh.getObjGen()) == 0) { std::cerr << oh.getObjectID() << "/" << oh.getGeneration() << " is not found in xref table" << std::endl; |