[MPlayer-dev-eng] Re: [PATCH] Kill some _LARGEFILE_SOURCE #ifdefs

[Loren Merritt]

On Thu, 30 Jun 2005, Oded Shimon wrote:

> On Thu, Jun 30, 2005 at 03:47:31PM +0200, Tobias Diedrich wrote:
>> Reimar D?ffinger wrote:
>>
>>> No good, since ll prefix doesn't exist on e.g. MinGW AFAIK.
>>
>> My mingw does support it:
>>
>> ranma at melchior:~$ dpkg -l mingw32
>> Desired=Unknown/Install/Remove/Purge/Hold
>> | Status=Not/Installed/Config-files/Unpacked/Failed-config/Half-installed
>> |/ Err?=(none)/Hold/Reinst-required/X=both-problems (Status,Err: uppercase=bad)
>> ||/ Name           Version        Description
>> +++-==============-==============-============================================
>> ii  mingw32        3.4.2.20040916 Minimalist GNU win32 (cross) compiler
>> ranma at melchior:~/tmp$ cat test.c
>> #include <stdio.h>
>>
>> int main(int argc, char **argv)
>> {
>>         long long bla = 20;
>>         printf("bla=%lld\n", bla);
>>
>>         return 0;
>> }
>> ranma at melchior:~/tmp$ i586-mingw32msvc-gcc -Wall test.c -o test.exe
>> ranma at melchior:~/tmp$ wine test.exe
>> bla=20
>> Wine exited with a successful status
>> ranma at melchior:~/tmp$
>
> If i'm not mistaken, isn't that the stupidest possible way to check that?..
> Wine probably uses glibc's printf, i doubt the implemented their own printf
> just so it would have slightly less features to be identical with windows's
> printf...

That code would work on little-endian even if printf ignores the %ll.

~> cat test.c
  #include <stdio.h>
  int main(int argc, char **argv)
  {
          long long bla = 20;
          printf("bla=%lld=%lld\n", bla, bla);
          return 0;
  }
~> gcc -mno-cygwin test.c -o test
~> ./test
bla=20=0
~> gcc test.c -o test
~> ./test
bla=20=20
~>

--Loren Merritt