diff options
author | Jay Berkenbilt <ejb@ql.org> | 2023-09-03 14:12:31 +0200 |
---|---|---|
committer | Jay Berkenbilt <ejb@ql.org> | 2023-09-03 14:12:31 +0200 |
commit | c6a3d83fb5958684436c8b607509079da22193f3 (patch) | |
tree | 4f43966f9856316798240a3232acae46a7f2ef19 | |
parent | fa3cdaf1b46dcbb583393f93a02f89207ba21a88 (diff) | |
download | qpdf-c6a3d83fb5958684436c8b607509079da22193f3.tar.zst |
Revert "Check for warnings in files used by --pages"
m-holger's fix is better
This reverts commit fa3cdaf1b46dcbb583393f93a02f89207ba21a88.
-rw-r--r-- | ChangeLog | 6 | ||||
-rw-r--r-- | libqpdf/QPDFJob.cc | 5 | ||||
-rw-r--r-- | qpdf/qtest/page-errors.test | 2 | ||||
-rw-r--r-- | qpdf/qtest/qpdf/page-missing-mediabox.out | 1 |
4 files changed, 1 insertions, 13 deletions
@@ -1,9 +1,3 @@ -2023-09-03 Jay Berkenbilt <ejb@ql.org> - - * Bug fix: with --pages, if one of the external files had warnings - but the main file did not, the warning was previously not taken - into consideration when determining the exit status. - 2023-09-02 Jay Berkenbilt <ejb@ql.org> * Bug fix from M. Holger: allow fix-qdf to read from pipe. Fixes #1010. diff --git a/libqpdf/QPDFJob.cc b/libqpdf/QPDFJob.cc index adce2358..b8ae7787 100644 --- a/libqpdf/QPDFJob.cc +++ b/libqpdf/QPDFJob.cc @@ -2561,11 +2561,6 @@ QPDFJob::handlePageSpecs(QPDF& pdf, std::vector<std::unique_ptr<QPDF>>& page_hea } } } - for (auto const& p: page_spec_qpdfs) { - if (!p.second->getWarnings().empty()) { - m->warnings = true; - } - } } void diff --git a/qpdf/qtest/page-errors.test b/qpdf/qtest/page-errors.test index 3f14f24e..60894f96 100644 --- a/qpdf/qtest/page-errors.test +++ b/qpdf/qtest/page-errors.test @@ -22,7 +22,7 @@ $td->runtest("handle page no with contents", $td->NORMALIZE_NEWLINES); $td->runtest("handle page with missing MediaBox", {$td->COMMAND => "qpdf --static-id --empty --pages page-no-content.pdf -- out.pdf"}, - {$td->FILE => "page-missing-mediabox.out", $td->EXIT_STATUS => 3}, + {$td->FILE => "page-missing-mediabox.out", $td->EXIT_STATUS => 0}, $td->NORMALIZE_NEWLINES); $td->runtest("check output", {$td->FILE => "out.pdf"}, diff --git a/qpdf/qtest/qpdf/page-missing-mediabox.out b/qpdf/qtest/qpdf/page-missing-mediabox.out index 91e85707..9db62ed6 100644 --- a/qpdf/qtest/qpdf/page-missing-mediabox.out +++ b/qpdf/qtest/qpdf/page-missing-mediabox.out @@ -1,2 +1 @@ WARNING: page-no-content.pdf, object 4 0 at offset 288: kid 1 (from 0) MediaBox is undefined; setting to letter / ANSI A -qpdf: operation succeeded with warnings; resulting file may have some problems |