rpbissonnette / thebigpicture

Automatically exported from code.google.com/p/thebigpicture
0 stars 0 forks source link

Tiff writer is incomplete #10

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
The current Tiff writer only writes the metadata, and doesn't copy the
actual image data.

Original issue reported on code.google.com by p.edel...@gmail.com on 15 Mar 2007 at 6:57

GoogleCodeExporter commented 9 years ago

Original comment by p.edel...@gmail.com on 16 Mar 2007 at 10:49

GoogleCodeExporter commented 9 years ago
This is fixed in revision 23, save for one problem:
RowsPerStrip, StripOffsets, and StripByteCounts can be both of type Short and 
Long,
thus occupying different sizes in the IFD. Since we're not altering image data,
RowsPerStrip and StripByteCounts will remain the same. StripOffsets however, 
*might*
change the datatype when a lot of metadata pushes it over the 65k limit. This 
should
be accounted for.

Original comment by p.edel...@gmail.com on 21 Mar 2007 at 7:38

GoogleCodeExporter commented 9 years ago
Actually, this problem does not occur, since we first calculate the offsets and 
then
let the IFD class decide what data type it should be encoded in. RowsPerStrip 
and
StripByteCounts will be copied verbatim, but they don't have anything to do 
with the
offset, so we're safe.

This issue requires further testing before we can close it -> beta1.

Original comment by p.edel...@gmail.com on 21 Mar 2007 at 7:42