shithub: opusfile

Download patch

ref: d75915786f465892f5eadcd93444f51a32b9ad1c
parent: cc5f107c13b8d18e3a8ebfc7e07282b4f35d72b0
author: Timothy B. Terriberry <tterribe@xiph.org>
date: Tue Jan 8 00:04:41 EST 2013

Use fseeko64/ftello64 for mingw32.

It turns out i686-pc-mingw32 does define these functions, and they
 are always available (unlike _fseeki64/_ftelli64).
This means we can build and link without requiring
 i686-w64-mingw32.
The resulting binary still doesn't run in wine for me, but that may
 be a personal problem.

--- a/src/stream.c
+++ b/src/stream.c
@@ -56,7 +56,18 @@
 static int op_fseek(void *_stream,opus_int64 _offset,int _whence){
 #if defined(_MSC_VER)
   return _fseeki64((FILE *)_stream,_offset,_whence);
+#elif defined(__MINGW32__)
+  /*i686-pc-mingw32 does not have fseeko() and requires
+     __MSVCRT_VERSION__>=0x800 for _fseeki64(), which screws up linking with
+     other libraries (that don't use MSVCRT80 from MSVC 2005 by default).
+    i686-w64-mingw32 does have fseeko() and respects _FILE_OFFSET_BITS, but I
+     don't know how to detect that at compile time.
+    We don't need to use fopen64(), as this just dispatches to fopen() in
+     mingw32.*/
+  return fseeko64((FILE *)_stream,(off64_t)_offset,_whence);
 #else
+  /*This function actually conforms to the SUSv2 and POSIX.1-2001, so we prefer
+     it except in the two special-cases above.*/
   return fseeko((FILE *)_stream,(off_t)_offset,_whence);
 #endif
 }
@@ -64,7 +75,18 @@
 static opus_int64 op_ftell(void *_stream){
 #if defined(_MSC_VER)
   return _ftelli64((FILE *)_stream);
+#elif defined(__MINGW32__)
+  /*i686-pc-mingw32 does not have ftello() and requires
+     __MSVCRT_VERSION__>=0x800 for _ftelli64(), which screws up linking with
+     other libraries (that don't use MSVCRT80 from MSVC 2005 by default).
+    i686-w64-mingw32 does have ftello() and respects _FILE_OFFSET_BITS, but I
+     don't know how to detect that at compile time.
+    We don't need to use fopen64(), as this just dispatches to fopen() in
+     mingw32.*/
+  return ftello64((FILE *)_stream);
 #else
+  /*This function actually conforms to the SUSv2 and POSIX.1-2001, so we prefer
+     it except in the two special-cases above.*/
   return ftello((FILE *)_stream);
 #endif
 }