2007.10.04 21:34 "Re: [Tiff] Pink pages with tiff2pdf -j", by Frank Warmerdam
The tiff2pdf "-j" option creates JPEG data inside the PDF.
Mistaking RGB for YCbCr or vice-versa in JPEG compression typically shows up as pink and green images (which is what we are seeing here).
Does this ping anyone's memory about recent changes?
Folks,
tiff2pdf.c has seen quite a few fixes in the last couple years. It is likely this is already fixed. The following looks most promising out of the cvs log:
revision 1.40
date: 2007/07/03 15:47:05; author: dron; state: Exp; lines: +523 -510
Fix for TIFFTAG_JPEGTABLES tag fetching and significant upgrade of the whole utility as per bug http://bugzilla.remotesensing.org/show_bug.cgi?id=1560 Now we don't need tiffiop.h in tiff2pdf anymore and will open output PDF file using TIFFClientOpen() machinery as it is implemented by Leon Bottou.
The last few years logs look like:
revision 1.43
date: 2007/07/12 17:32:53; author: dron; state: Exp; lines: +3 -3
Mapping routines definitions fixed.
----------------------------
revision 1.42
date: 2007/07/12 17:25:12; author: dron; state: Exp; lines: +2 -1
Added missed extern optind.
revision 1.41
date: 2007/07/06 01:25:17; author: bfriesen; state: Exp; lines: +3 -2
* tools/tiff2pdf.c: Fix a compile problem when JPEG_SUPPORT is not defined.
----------------------------
revision 1.40
date: 2007/07/03 15:47:05; author: dron; state: Exp; lines: +523 -510
Fix for TIFFTAG_JPEGTABLES tag fetching and significant upgrade of the whole utility as per bug http://bugzilla.remotesensing.org/show_bug.cgi?id=1560 Now we don't need tiffiop.h in tiff2pdf anymore and will open output PDF file using TIFFClientOpen() machinery as it is implemented by Leon Bottou.
----------------------------
revision 1.39
date: 2007/04/27 16:47:14; author: dron; state: Exp; lines: +11 -13
Check the tmpfile() return status as per bug http://bugzilla.remotesensing.org/show_bug.cgi?id=154
----------------------------
revision 1.38
date: 2007/04/18 08:46:33; author: dron; state: Exp; lines: +10 -15
Avoid warnings.
----------------------------
revision 1.37
date: 2007/02/24 16:28:48; author: dron; state: Exp; lines: +43 -37
branches: 1.37.2;
Do not assume inches when the resolution units do not specified. As per bug http://bugzilla.remotesensing.org/show_bug.cgi?id=1366
----------------------------
revision 1.36
date: 2006/06/19 20:12:08; author: fwarmerdam; state: Exp; lines: +9 -17
Fix handling of -q values. (bug 587)
----------------------------
revision 1.35
date: 2006/06/08 11:27:11; author: dron; state: Exp; lines: +226 -155
More fixes for character type safety.
----------------------------
revision 1.34
date: 2006/06/08 10:45:35; author: dron; state: Exp; lines: +8 -7
Fixed buffer overflow condition in t2p_write_pdf_string() as per bug http://bugzilla.remotesensing.org/show_bug.cgi?id=1196
----------------------------
revision 1.33
date: 2006/04/21 15:09:34; author: dron; state: Exp; lines: +92 -92
Unified line ending characters (always use '\n') as per bug http://bugzilla.remotesensing.org/show_bug.cgi?id=1163
----------------------------
revision 1.32
date: 2006/04/20 12:36:23; author: dron; state: Exp; lines: +12 -1
Properly set the binary mode for stdin stream as per bug http://bugzilla.remotesensing.org/show_bug.cgi?id=1141
----------------------------
revision 1.31
date: 2006/04/20 12:04:35; author: dron; state: Exp; lines: +18 -19
Fixed typos as per bug http://bugzilla.remotesensing.org/show_bug.cgi?id=1139
----------------------------
revision 1.30
date: 2006/03/21 16:37:51; author: dron; state: Exp; lines: +6 -8
Fixed win32 I/O functions usage as per bug http://bugzilla.remotesensing.org/show_bug.cgi?id=1127
----------------------------
revision 1.29
date: 2006/03/20 15:29:53; author: dron; state: Exp; lines: +5 -5
Avoid warnings.
----------------------------
revision 1.28
date: 2006/02/26 17:55:42; author: dron; state: Exp; lines: +16 -21
Functions t2p_sample_rgbaa_to_rgb() and t2p_sample_rgba_to_rgb() was used in place of each other, that was resulted in problems with RGBA images with associated alpha. As per bug http://bugzilla.remotesensing.org/show_bug.cgi?id=1097
----------------------------
revision 1.27
date: 2006/02/15 13:33:59; author: dron; state: Exp; lines: +47 -141
Do not create output file until all option checks will be done. As per bug http://bugzilla.remotesensing.org/show_bug.cgi?id=1072
----------------------------
revision 1.26
date: 2006/02/07 14:28:49; author: fwarmerdam; state: Exp; lines: +9 -1
fix for non-YCbCr jpeg compressed files
----------------------------
revision 1.25
date: 2005/12/27 12:24:07; author: dron; state: Exp; lines: +6 -1
Avoid warnings.
----------------------------
revision 1.24
date: 2005/09/20 11:19:38; author: dron; state: Exp; lines: +5 -1
Added missed 'break' statement as per bug http://bugzilla.remotesensing.org/show_bug.cgi?id=932
----------------------------
revision 1.23
date: 2005/09/13 13:40:48; author: dron; state: Exp; lines: +72 -60
Avoid warnings.
----------------------------
revision 1.22
date: 2005/06/23 13:28:33; author: dron; state: Exp; lines: +17 -13
Print two characters per loop in the t2p_write_pdf_trailer(). As per bug http://bugzilla.remotesensing.org/show_bug.cgi?id=594
----------------------------
revision 1.21
date: 2005/05/05 20:52:57; author: dron; state: Exp; lines: +26 -12
Calculate the tilewidth properly; added new option '-b' to use interpolation in output PDF files (Bruno Ledoux).
----------------------------
revision 1.20
date: 2005/03/18 09:47:34; author: dron; state: Exp; lines: +34 -36
Fixed problem with alpha channel handling as per bug http://bugzilla.remotesensing.org/show_bug.cgi?id=794
----------------------------
revision 1.19
date: 2004/10/28 13:32:28; author: fwarmerdam; state: Exp; lines: +10 -4
provide explicit unsigned char casts for a few values to avoid warnings
----------------------------
revision 1.18
date: 2004/10/10 11:38:34; author: dron; state: Exp; lines: +5 -2
Move _TIFFmemset() behind the pointer check in t2p_init().
----------------------------
revision 1.17
date: 2004/10/09 14:15:07; author: dron; state: Exp; lines: +12 -2
Fixed TransferFunction tag handling reported by Ross A. Finlayson.
----------------------------
revision 1.16
date: 2004/09/10 11:50:57; author: dron; state: Exp; lines: +34 -30
Fixed reading TransferFunction tag as per bug http://bugzilla.remotesensing.org/show_bug.cgi?id=590
----------------------------
revision 1.15
date: 2004/08/25 18:34:55; author: dron; state: Exp; lines: +4 -7
Work out getopt problems.
revision 1.14
date: 2004/08/25 13:43:14; author: dron; state: Exp; lines: +27 -10
Initialize arrays properly.
revision 1.13
date: 2004/08/24 07:48:36; author: dron; state: Exp; lines: +18 -10
More fixes for bug http://bugzilla.remotesensing.org/show_bug.cgi?id=590 from Ross Finlayson.
----------------------------
revision 1.12
date: 2004/08/23 12:00:30; author: dron; state: Exp; lines: +41 -40
Fixed a bunch of problems as per bug http://bugzilla.remotesensing.org/show_bug.cgi?id=590
----------------------------
revision 1.11
date: 2004/08/21 08:09:49; author: dron; state: Exp; lines: +10 -2
More fixes from Ross.
----------------------------
revision 1.10
date: 2004/08/20 19:23:25; author: dron; state: Exp; lines: +49 -22
Applied patch from Ross Finlayson that checks that the input file has compression, photometric interpretation, etcetra, tags or if not than a more
descriptive error is returned.
revision 1.9
date: 2004/06/04 13:46:25; author: dron; state: Exp; lines: +18 -12
Avoid warnings.
----------------------------
revision 1.8
date: 2004/05/26 09:24:07; author: dron; state: Exp; lines: +68 -64
Get rid of __T() macro; avoid warnings.
----------------------------
revision 1.7
date: 2004/04/20 14:54:05; author: dron; state: Exp; lines: +33 -17
Fixed problem with unaligned access as per bug http://bugzilla.remotesensing.org/show_bug.cgi?id=555
----------------------------
revision 1.6
date: 2004/04/20 14:24:31; author: dron; state: Exp; lines: +4 -64
Obsoleted configuration switches removed.
----------------------------
Best regards,
--
---------------------------------------+--------------------------------------
I set the clouds in motion - turn up | Frank Warmerdam, warmerdam@pobox.com
light and sound - activate the windows | http://pobox.com/~warmerdam
and watch the world go round - Rush | President OSGeo, http://osgeo.org