
Thread
2009.01.22 16:23 "Re: [Tiff] libtiff 4.0.0beta3", by Edward Lam
Looks like I spoke too soon. I had previously only built libtiff with my custom build process and did not go through the usual nmake.
I did a pass using the canonical build instructions for MSVC and got some additional warnings similar to:
tif_lzw.c(389): warning C4244: '+=': conversion from 'tmsize_t' to 'long', possible loss of data tif_lzw.c(606): warning C4244: '+=': conversion from 'tmsize_t' to 'long', possible loss of data tif_lzw.c(715): warning C4244: '=': conversion from 'tmsize_t' to 'long', possible loss of data
As for the tools, tiffcrop fails to compile on Windows. stdint.h is not available on Windows. I only took a quick look and I'm not sure if it's necessary?
Here is the patch that I did to get it to compile on Windows:
============================================================ --- tiffcrop.c 2009-01-11 15:04:22.000000000 -0500 +++ ../../tiff-4.0.0beta3.new/tools/tiffcrop.c 2009-01-22 11:17:55.670500000 -0
500
@@ -117,7 +117,9 @@
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
+#if !defined(__WIN32__) && !defined(_MSC_VER)
#include <stdint.h>
+#endif
#include <math.h>
#include <ctype.h>
#include <limits.h>
@@ -130,6 +132,7 @@
#ifndef HAVE_GETOPT
extern int getopt(int, char**, char*);
+extern int optind;
#endif
#include "tiffio.h"
=============================================================
Regards,
-Edward
I justed tested this with a build on WinXP x64. Things seem to work fine with my own narrow regression tests.
The only warning I got while compiling was:
tif_dirinfo.c(508): warning C4133: 'function': incompatible types - from 'size_t *' to 'unsigned int *'
With Bob's help I have uploaded a snapshot of CVS head which I have dubbed libtiff 4.0.0 beta3.
http://download.osgeo.org/libtiff/tiff-4.0.0beta3.tar.gz
http://download.osgeo.org/libtiff/tiff-4.0.0beta3.zipI would appreciate folks trying this out and reporting outstanding issues via bugzilla. Ideally I'd like to prepare a libtiff 4.0.0 release candidate within the next month.