Bug 36224 - [PATCH] Support for CCITTFaxDecode filter (TIFF images) in PDF Renderer
Summary: [PATCH] Support for CCITTFaxDecode filter (TIFF images) in PDF Renderer
Status: CLOSED FIXED
Alias: None
Product: Fop - Now in Jira
Classification: Unclassified
Component: images (show other bugs)
Version: trunk
Hardware: Other other
: P2 normal
Target Milestone: ---
Assignee: fop-dev
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-08-17 15:04 UTC by Manuel Mall
Modified: 2012-04-01 07:08 UTC (History)
0 users



Attachments
Patch file (45.38 KB, patch)
2005-08-17 15:05 UTC, Manuel Mall
Details | Diff
New class - src\java\org\apache\fop\pdf\CCFFilter.java (1.51 KB, text/plain)
2005-08-17 15:06 UTC, Manuel Mall
Details
TIFF file with CCITT Group 4 encoding for testing (8.45 KB, application/octet-stream)
2005-08-17 15:10 UTC, Manuel Mall
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Manuel Mall 2005-08-17 15:04:21 UTC
This patch ports the 0.20.5 CCITTFaxDecode filter functionality and its support
of direct embedding on certain TIFF images to the trunk.

It also cleans up some of the image handling with respect to consistent closing
of the input stream to avoid dangling open files.

There is most likely still a fundamental problem with respect to image decoding
and the image cache left. As renderers differ in their ability to handle an
image in its raw format or decoded the format, the image bitmap stored (raw or
decoded) depends on the first renderer loading it. If in a subsequent invocation
a different renderer get the same image from the cache it may not be in the
format that the second renderer expects leading to undesirable results. I'll
leave that for a later time.
Comment 1 Manuel Mall 2005-08-17 15:05:10 UTC
Created attachment 16079 [details]
Patch file
Comment 2 Manuel Mall 2005-08-17 15:06:06 UTC
Created attachment 16080 [details]
New class - src\java\org\apache\fop\pdf\CCFFilter.java
Comment 3 Manuel Mall 2005-08-17 15:10:26 UTC
Created attachment 16081 [details]
TIFF file with CCITT Group 4 encoding for testing

Not sure where this file should go. It really belongs to some renderer testing
resources. May be it can go into test/resources/images if this is the
repository for all test images being it for layout or for render testing.
Comment 4 Jeremias Maerki 2005-08-21 18:49:35 UTC
Applied. Thanks a lot for the patch. It looked very good.

It should be possible to write a JUnit test case to check for the caching 
problem you think could be around. I haven't tested for that one, yet.

As a side note: Direct CCITT support could also be added to the PostScript 
renderer as it supports this filter, too. Probably quite easy now.
Comment 5 Glenn Adams 2012-04-01 07:08:40 UTC
batch transition pre-FOP1.0 resolved+fixed bugs to closed+fixed