Re: [Mingw-w64-public] Patch for ffmpeg

> I don`t see that this patch is valid. I would request here more > details about reasoning. I would be curious to learn that we have > useless code in repro for years > > Kai Current code works with msvcrt.dll in Win32 + _USE_32BIT_TIME_T (which is defined by default in Win32). It doesn't

Re: [Mingw-w64-public] Patch for ffmpeg

On 12/3/2015 03:32, Mateusz wrote: > I've prepared small patch for _mingw_stat64.h and mkstemp.c files. > > First problem is that in _mingw_stat64.h header file #define section for > _USE_32BIT_TIME_T case is switched from > #define new_name msvcrt_function > to > #define msvcrt_function

Re: [Mingw-w64-public] Patch for ffmpeg

I don`t see that this patch is valid. I would request here more details about reasoning. I would be curious to learn that we have useless code in repro for years Kai Am 29.12.2015 10:45 schrieb "JonY" : > On 12/3/2015 03:32, Mateusz wrote: > > I've prepared small

Re: [Mingw-w64-public] Patch for ffmpeg

I execute command gcc -dumpspecs >/local/specs.org and then edit /local/specs.org file. I find 'msvcrt' word and delete all text except '*libgcc:' section (which contains 'msvcrt' word). In my GCC it looks like this: *libgcc: %{mthreads:-lmingwthrd} -lmingw32 -lgcc -lmoldname

Re: [Mingw-w64-public] Patch for ffmpeg

On Thu, Dec 3, 2015 at 12:35 AM, Mateusz wrote: > Sorry for previous patch -- I often link to msvcr120.dll instead of > msvcrt.dll. > I didn't know that was possible. How do you do that and why would you? What is the advantage? -- ˙uʍop-ǝpısdn sı ɹoʇıuoɯ ɹnoʎ 'sıɥʇ

[Mingw-w64-public] Patch for ffmpeg

I've prepared small patch for _mingw_stat64.h and mkstemp.c files. First problem is that in _mingw_stat64.h header file #define section for _USE_32BIT_TIME_T case is switched from #define new_name msvcrt_function to #define msvcrt_function new_name which is nonsense. ffmpeg in file

Re: [Mingw-w64-public] Patch for ffmpeg

Sorry for previous patch -- I often link to msvcr120.dll instead of msvcrt.dll. For normal linking to msvcrt.dll the patch should be bigger -- we should add _fstat32i64, _stat32i64, _wstat32i64, _ftime32 aliases to 32-bit version of libmsvcrt.a (these functions are present in libmsvcr120.a).