2005.03.29 23:08 "[Tiff] [patch] little things", by Ron

2005.03.30 13:48 "Re: [Tiff] [patch] little things", by Andrey Kiselev

Hi, Ron,

I've got a couple of little things first off, all re current cvs HEAD. Unless I'm mistaken, we can't just add the exif tags in a neat block to tif_dirinfo.c, we rely on that array being ordered by tag and bsearch it later. The first hunk below fixes that for the exif tags that were added, I've not checked the DNG ones that were also added. The second hunk changes find by name to use lsearch, since the array is ordered by key not name.

I'm also using the following in the TIFFClientOpen replacement I posted previously, (so it won't apply cleanly to HEAD except by hand). The problem it fixes is pathological but potentially real I think. If you try to write a tiff to a filename that already exists, we try to read the header from the file instead of just writing the one we have. If the byte sex of the header on file is different to the tiff you think you are about to write, it would seem things might go pear shaped. That may not be the only way, so its simpler and safer to just not try to read it if we are writing I think, as a minimal patch something like so:

Thanks! Your patches applied.

Andrey V. Kiselev
ICQ# 26871517