vcl/source/filter/ipdf/pdfread.cxx | 17 ++++------------- 1 file changed, 4 insertions(+), 13 deletions(-)
New commits: commit 512e58a3962b403df9d995fdb4ca8d3f7c6768eb Author: Miklos Vajna <vmik...@collabora.com> AuthorDate: Tue Apr 9 21:48:03 2019 +0200 Commit: Tomaž Vajngerl <qui...@gmail.com> CommitDate: Sat Mar 21 15:41:04 2020 +0100 vcl PDF import: clean up CompatibleWriter It's cleaner to avoid member functions and constructors when we inherit from a C struct and we pass it to pdfium's C API. Change-Id: Iac9ab99b3d1b66241bb49d3a7707e242078ab5df Reviewed-on: https://gerrit.libreoffice.org/70472 Tested-by: Jenkins Reviewed-by: Miklos Vajna <vmik...@collabora.com> (cherry picked from commit 3c24774d85a8c0ac03faa4ba2b82c8c9dfe0a2b3) Reviewed-on: https://gerrit.libreoffice.org/c/core/+/90832 Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoff...@gmail.com> Reviewed-by: Tomaž Vajngerl <qui...@gmail.com> diff --git a/vcl/source/filter/ipdf/pdfread.cxx b/vcl/source/filter/ipdf/pdfread.cxx index 9815e89a1a2b..edd354f19e5b 100644 --- a/vcl/source/filter/ipdf/pdfread.cxx +++ b/vcl/source/filter/ipdf/pdfread.cxx @@ -30,22 +30,10 @@ namespace /// Callback class to be used with FPDF_SaveWithVersion(). struct CompatibleWriter : public FPDF_FILEWRITE { -public: - CompatibleWriter(); - static int WriteBlockCallback(FPDF_FILEWRITE* pFileWrite, const void* pData, - unsigned long nSize); - SvMemoryStream m_aStream; }; -CompatibleWriter::CompatibleWriter() -{ - FPDF_FILEWRITE::version = 1; - FPDF_FILEWRITE::WriteBlock = CompatibleWriter::WriteBlockCallback; -} - -int CompatibleWriter::WriteBlockCallback(FPDF_FILEWRITE* pFileWrite, const void* pData, - unsigned long nSize) +int CompatibleWriterCallback(FPDF_FILEWRITE* pFileWrite, const void* pData, unsigned long nSize) { auto pImpl = static_cast<CompatibleWriter*>(pFileWrite); pImpl->m_aStream.WriteBytes(pData, nSize); @@ -182,6 +170,9 @@ bool getCompatibleStream(SvStream& rInStream, SvStream& rOutStream, sal_uInt64 n return false; CompatibleWriter aWriter; + aWriter.version = 1; + aWriter.WriteBlock = &CompatibleWriterCallback; + // 15 means PDF-1.5. if (!FPDF_SaveWithVersion(pPdfDocument, &aWriter, 0, 15)) return false; _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits