ropensci / qpdf

Split, Combine and Compress PDF files
https://docs.ropensci.org/qpdf
Other
57 stars 10 forks source link

Removed pages.size() from split_pdf #6

Closed SamPortnow closed 7 months ago

SamPortnow commented 5 years ago

The addition of pages.size() creates an error in large pdfs, and it doesn't seem necessary.

zeileis commented 7 months ago

Jeroen has suggested a better solution in this commit: https://github.com/ropensci/qpdf/commit/098c6fa1c8fe7f3d6031b407a669b2b2240b7e75 So I think that this PR is not needed anymore and can be closed.

jeroen commented 7 months ago

Fixed in qpdf 1.3.3