Re: [PR] Add method to merge PDF from InputStream source [pdfbox]

2024-12-22 Thread via GitHub
asfgit closed pull request #198: Add method to merge PDF from InputStream source URL: https://github.com/apache/pdfbox/pull/198 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment.

Re: [PR] Add method to merge PDF from InputStream source [pdfbox]

2024-12-21 Thread via GitHub
lehmi commented on PR #198: URL: https://github.com/apache/pdfbox/pull/198#issuecomment-2558173231 I fully support @THausherr. We are using the RandomAccessRead interfaces to give the user maximum flexibility w.r.t. the used access method and possible optimization -- This is an automated

Re: [PR] Add method to merge PDF from InputStream source [pdfbox]

2024-12-19 Thread via GitHub
THausherr commented on PR #198: URL: https://github.com/apache/pdfbox/pull/198#issuecomment-2555629989 I'd prefer to improve the javadoc. Also for `Loader.loadPDF()` which has the same problem. -- This is an automated message from the Apache Git Service. To respond to the message, please

Re: [PR] Add method to merge PDF from InputStream source [pdfbox]

2024-12-19 Thread via GitHub
aaronhagopian commented on code in PR #198: URL: https://github.com/apache/pdfbox/pull/198#discussion_r1892909779 ## pdfbox/src/main/java/org/apache/pdfbox/multipdf/PDFMergerUtility.java: ## @@ -16,10 +16,7 @@ */ package org.apache.pdfbox.multipdf; -import java.io.File; Re

[PR] Add method to merge PDF from InputStream source [pdfbox]

2024-12-19 Thread via GitHub
aaronhagopian opened a new pull request, #198: URL: https://github.com/apache/pdfbox/pull/198 Introduced a new `addSource` method that accepts an `InputStream` as a parameter, enabling merging of PDFs directly from streams keeping the developer from having to understand the RandomAccessRead