You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@pdfbox.apache.org by "ASF subversion and git services (JIRA)" <ji...@apache.org> on 2014/10/09 02:16:34 UTC
[jira] [Commented] (PDFBOX-2387) ImageIOUtil, JPEGUtil, TIFFUtil
and MetaUtil are not needed in "pdfbox"
[ https://issues.apache.org/jira/browse/PDFBOX-2387?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14164443#comment-14164443 ]
ASF subversion and git services commented on PDFBOX-2387:
---------------------------------------------------------
Commit 1630259 from [~jahewson] in branch 'pdfbox/trunk'
[ https://svn.apache.org/r1630259 ]
PDFBOX-2387: Move imageio utilities to pdfbox.tools
> ImageIOUtil, JPEGUtil, TIFFUtil and MetaUtil are not needed in "pdfbox"
> -----------------------------------------------------------------------
>
> Key: PDFBOX-2387
> URL: https://issues.apache.org/jira/browse/PDFBOX-2387
> Project: PDFBox
> Issue Type: Improvement
> Reporter: John Hewson
>
> The classes ImageIOUtil, JPEGUtil, TIFFUtil and MetaUtil are not needed in the "pdfbox" module. All usages of ImageIOUtil in the "pdfbox" module can be safely replaced with Java's ImageIO (I've tested this on my working copy). That's over 650 lines of imaging code which we don't need.
> ImageIOUtil and friends are only needed by classes in the "tools" module, which write image files to disk. Ultimately the ImageIOUtil class is just a bandaid for Java ImageIO's bugs, which while helpful, isn't a necessary part of the PDFBox core.
> Obviously ImageIOUtil is useful to downstream consumers of PDFBox, but it doesn't belong in the core "pdfbox" module, where it introduces unnecessary complexity, so we should move it into the "tools" module along with JPEGUtil, TIFFUtil and MetaUtil.
> We might want to consider doing this with other utility classes which probably don't belong in the "pdfbox" core such as TextToPDF, PrintPDF, WriteDecodedDoc, etc. - basically anything with a main() method. But we'll deal with that later.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)