diff options
author | Jay Berkenbilt <ejb@ql.org> | 2021-01-02 18:05:09 +0100 |
---|---|---|
committer | Jay Berkenbilt <ejb@ql.org> | 2021-01-02 20:08:17 +0100 |
commit | bedf35d6a55a9092485d56002b50bc9003ce7931 (patch) | |
tree | 80af63d963a4e23ea3c88a29dc607b18cdf99679 /include | |
parent | a139d2b36da39fbfb018ef6973e9316a64a4ca6c (diff) | |
download | qpdf-bedf35d6a55a9092485d56002b50bc9003ce7931.tar.zst |
Bug fix: avoid extraneous pipeline finish calls with multiple contents
Avoid calling finish() multiple times on the pipeline passed to
pipeContentStreams. This commit also fixes a bug in which qpdf was not
exiting with the proper exit status if warnings found while splitting
pages; this was exposed by a test case that changed.
Diffstat (limited to 'include')
-rw-r--r-- | include/qpdf/QPDFPageObjectHelper.hh | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/include/qpdf/QPDFPageObjectHelper.hh b/include/qpdf/QPDFPageObjectHelper.hh index 93745446..1152a7a5 100644 --- a/include/qpdf/QPDFPageObjectHelper.hh +++ b/include/qpdf/QPDFPageObjectHelper.hh @@ -199,12 +199,7 @@ class QPDFPageObjectHelper: public QPDFObjectHelper // Pipe a page's contents through the given pipeline. This method // works whether the contents are a single stream or an array of - // streams. Call on a page object. Please note that if there is an - // array of content streams, p->finish() is called after each - // stream. If you pass a pipeline that doesn't allow write() to be - // called after finish(), you can wrap it in an instance of - // Pl_Concatenate and then call manualFinish() on the - // Pl_Concatenate pipeline at the end. + // streams. QPDF_DLL void pipePageContents(Pipeline* p); |