newline before endstream fix for object streams (fixes #205)

Jay Berkenbilt
1 parent b8ccbff4
Showing 4 changed files with 13 additions and 1 deletions
ChangeLog
libqpdf/QPDFWriter.cc
qpdf/qtest/qpdf.test
qpdf/qtest/qpdf/newline-before-endstream-nl-objstm.pdf
+2018-05-12  Jay Berkenbilt  <ejb@ql.org>
+
+	* In newline before endstream mode, an extra newline was not
+	inserted prior to the endstream that ends object streams.
+	Fixes #205.
+
 2018-04-15  Jay Berkenbilt  <ejb@ql.org>
 	* Arbitrarily limit the depth of data structures represented by
@@ -1907,6 +1907,10 @@ QPDFWriter::writeObjectStream(QPDFObjectHandle object)
     pushEncryptionFilter();
     writeBuffer(stream_buffer);
     popPipelineStack();
+    if (this->m->newline_before_endstream)
+    {
+        writeString("\n");
+    }
     writeString("endstream");
     this->m->cur_data_key.clear();
     closeObject(new_id);
@@ -964,7 +964,7 @@ $td-&gt;runtest(&quot;check output&quot;,
 show_ntests();
 # ----------
 $td->notify("--- Newline before endstream ---");
-$n_tests += 10;
+$n_tests += 12;
 # From issue 133, http://verapdf.org/software/ is an open source
 # package that can verify PDF/A compliance. This could potentially be
@@ -975,6 +975,8 @@ foreach my $d (
     ['--qdf', 'qdf', 'qdf'],
     ['--newline-before-endstream', 'newline', 'nl'],
     ['--qdf --newline-before-endstream', 'newline and qdf', 'nl-qdf'],
+    ['--object-streams=generate --newline-before-endstream',
+     'newline and object streams', 'nl-objstm'],
     )
 {
     my ($flags, $description, $suffix) = @$d;