[ https://issues.apache.org/jira/browse/PDFBOX-1975?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13931438#comment-13931438 ]
Tilman Hausherr edited comment on PDFBOX-1975 at 3/12/14 7:00 AM: ------------------------------------------------------------------ I added JAI and levigo to the parent POM for test purposes only in rev 1576588, to the pdfbox POM in rev 1576598. I added levigo to enable a JBIG2 test in the near future, i.e. adding a non copyrighted PDF file with JBIG2 to the 3 existing test files, and checking that the result isn't empty. was (Author: tilman): I added JAI and levigo to the POM for test purposes only in rev 1576588. I added levigo to enable a JBIG2 test in the near future, i.e. adding a non copyrighted PDF file with JBIG2 to the 3 existing test files, and checking that the result isn't empty. > Improve TestImageIOUtils unit tests to check image resolution and compression > ----------------------------------------------------------------------------- > > Key: PDFBOX-1975 > URL: https://issues.apache.org/jira/browse/PDFBOX-1975 > Project: PDFBox > Issue Type: Task > Components: Utilities > Affects Versions: 2.0.0 > Reporter: Tilman Hausherr > Assignee: Tilman Hausherr > Priority: Minor > Labels: imageio, test, tiff > Fix For: 2.0.0 > > > Because of the problems with recent changes (see PDFBOX-1963), I will improve > the unit tests so that image resolution and compression is checked. > I found out that JPEGs don't have a resolution, BMP had the wrong resolution. > The fault wasn't in the java TIFF writer as I thought before, it is in the > java PNG writer, which uses the PixelSize values wrongly, i.e. it interprets > them as "pixels per mm" instead of "mm per pixel" as per specification. The > JPEG writer throws an exception "JFIF APP0 must be first marker after SOI". > The BMP writer can set the resolution, but the BMP reader doesn't read it. > (Some of this might be different depending on the version) -- This message was sent by Atlassian JIRA (v6.2#6252)