On 22.06.2017 15:07, Kai Tietz via Mingw-w64-public wrote:
> 2017-06-20 3:52 GMT+02:00 Liu Hao <lh_mo...@126.com>:
>> On 2017/6/20 1:40, Jacek Caban wrote:
>>> Please review.
>>>
>>> Signed-off-by: Jacek Caban <ja...@codeweavers.com>
>&
There were some discussions about recent libstdc++ breackage, but I
didn't see any fix committed, so here is my proposal. The first patch is
just about doing (read: breaking) things in the right place instead of a
random unrelated header. Please review.
Signed-off-by: Jacek Caban <
is not a good place to do that. In the future, when libstdc++ is
fixed, we may change guards to be version-dependent.
Signed-off-by: Jacek Caban <ja...@codeweavers.com>
---
mingw-w64-headers/include/sal.h | 8 +++-
1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/mingw-w64-h
will be defined.
Seems like this would be better:
#if defined(__cplusplus) && defined(__GNUC__)
// Don't define __in and __out because they conflict with libstdc++.
#else
#define __in
#define __out
#endif
--David
On Mon, May 29, 2017 at 12:43 PM, Jacek Caban <ja...@codeweavers.com>
Signed-off-by: Jacek Caban <ja...@codeweavers.com>
---
mingw-w64-headers/include/versionhelpers.h | 8
1 file changed, 8 insertions(+)
diff --git a/mingw-w64-headers/include/versionhelpers.h b/mingw-w64-headers/include/versionhelpers.h
index 25ea414..178e788 100644
--- a/min
Hi Martin,
On 9/23/17 4:50 PM, Martin Storsjö wrote:
We already did this when targeting ucrtbase.dll, to avoid using
__mingw_vprintf and __mingw_vwprintf from libmingwex, since those
hardcode references to the wrong (pre-ucrtbase) stdout/stdin
FILE objects.
It should be fine to do the same on
Hi Martin,
On 9/23/17 4:50 PM, Martin Storsjö wrote:
--- a/mingw-w64-crt/crt/pseudo-reloc.c
+++ b/mingw-w64-crt/crt/pseudo-reloc.c
@@ -145,11 +145,13 @@ __report_error (const char *msg, ...)
#else
va_list argp;
va_start (argp, msg);
+ /* Intentionally avoid optimizing fprintf with a
On 10/11/17 5:15 AM, Shinchiro Shinchiro wrote:
Is there a reason why latest pull from wine doesn't include new directx
file
like d3d11_3 and such?
They were not really excluded, but simply not added. The update was huge
already, I wanted to do that separately. I committed it now, please
On 11.10.2017 12:25, Shinchiro Shinchiro wrote:
> On Wed, Oct 11, 2017 at 5:23 PM, Jacek Caban wrote:
>
> They were not really excluded, but simply not added. The update
> was huge already, I wanted to do that separately. I committed it
> now, please give it a try.
&g
On 11.10.2017 12:47, Leif AMO wrote:
> On 2017-10-10 17:13, Jacek Caban wrote:
>> Importing from Wine is scripted, but the script for that is not
>> public for quite a while. We used to have it in experimental branch
>> of SVN, but it didn't survive move to Git. I
On 11.10.2017 14:18, Shinchiro Shinchiro wrote:
> On Wed, Oct 11, 2017 at 6:38 PM, Jacek Caban wrote:
>
> Oh, right, I didn't mean to commit that at all. The problem is
> that mingw-w64 version of widl crashes on that IDL. It needs an
> investigation (and proba
Hi Martin,
On 07.09.2017 14:47, Martin Storsjö wrote:
> With clang, there's no mechanism with spec files. For compiling, it's
> easy to add the -D__MSVCRT_VERSION__ yourself, but for linking,
> -lmsvcrt is hardcoded. I send this patch for that yesterday:
>
> https://reviews.llvm.org/D37530
Please review.
Signed-off-by: Jacek Caban <ja...@codeweavers.com>
---
mingw-w64-headers/include/sal.h | 3 +++
1 file changed, 3 insertions(+)
diff --git a/mingw-w64-headers/include/sal.h b/mingw-w64-headers/include/sal.h
index 9088d5ab..0a2df8b4 100644
--- a/mingw-w64-headers/include
Hi Martin,
On 11.09.2017 14:48, Martin Storsjö wrote:
> #undef __mingw_ovr
> #if defined (__GNUC__)
> #define __mingw_ovr static __attribute__ ((__unused__)) __inline__ __cdecl
> @@ -497,7 +544,11 @@ int wscanf(const wchar_t *__format, ...)
> {
>register int __retval;
>
Please review.
Signed-off-by: Jacek Caban <ja...@codeweavers.com>
---
mingw-w64-headers/crt/stdio.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/mingw-w64-headers/crt/stdio.h b/mingw-w64-headers/crt/stdio.h
index 34aae9ad..0d9cf1c7 100644
--- a/mingw-w64-heade
Hi Tom,
On 10/17/17 7:16 AM, Tom Ritter wrote:
Attached is a patch that I think, maybe, resolves the issue in
https://bugzilla.mozilla.org/show_bug.cgi?id=1372958#c10
The attachment didn't make it to the mailing list.
Thanks,
Jacek
On 11/28/17 10:30 AM, Kai Tietz via Mingw-w64-public wrote:
Hello Martin,
patch is ok. Please go ahead and commit, if Jacek has no objections.
The patch looks good to me as well. Thanks a lot for taking care of that!
Jacek
Hi Martin,
On 11/24/17 11:03 AM, Martin Storsjö wrote:
---
mingw-w64-headers/crt/conio.h | 214 +-
1 file changed, 209 insertions(+), 5 deletions(-)
Since we already know that we often need such functions in
libucrtbase.a, wouldn't it make more
Signed-off-by: Jacek Caban <ja...@codeweavers.com>
---
mingw-w64-crt/Makefile.am | 2 +-
mingw-w64-crt/def-include/msvcrt-common.def.in | 2 ++
mingw-w64-crt/stdio/vsnprintf_s.c | 16
3 files changed, 3 insertions(+), 17 deletions(-)
On 24.11.2017 20:45, Martin Storsjö wrote:
> On Fri, 24 Nov 2017, Jacek Caban wrote:
>
>>
>> It's properly forwarded by crt import libs.
>>
>> Signed-off-by: Jacek Caban <ja...@codeweavers.com>
>> ---
>> mingw-w64-crt/Makefile.am
On 24.11.2017 11:37, Martin Storsjö wrote:
> MSVC doesn't provide these functions as global symbols at the linker
> level, but there you must include a header to make them work.
>
> Additionally, for the wchar versions, you can currently override the
> behaviour by setting
It's properly forwarded by crt import libs.
Signed-off-by: Jacek Caban <ja...@codeweavers.com>
---
mingw-w64-crt/Makefile.am | 2 +-
mingw-w64-crt/stdio/vsnprintf.c | 13 -
2 files changed, 1 insertion(+), 14 deletions(-)
delete mode 100644 mingw-w64-crt/stdio/vsnpr
Signed-off-by: Jacek Caban <ja...@codeweavers.com>
---
mingw-w64-headers/crt/corecrt_startup.h | 23 +++
mingw-w64-headers/crt/process.h | 1 +
2 files changed, 24 insertions(+)
create mode 100644 mingw-w64-headers/crt/corecrt_startup.h
diff --git a/min
Signed-off-by: Jacek Caban <ja...@codeweavers.com>
---
mingw-w64-crt/crt/ucrtbase_compat.c | 8
mingw-w64-crt/include/internal.h| 12
2 files changed, 20 deletions(-)
diff --git a/mingw-w64-crt/crt/ucrtbase_compat.c b/mingw-w64-crt/crt/ucrtbase_compat.c
index cb
This is a workaround, widl can't handle parametrized macro as #if
expression. Ideally, widl preprocessor should support this syntax.
Signed-off-by: Jacek Caban <ja...@codeweavers.com>
---
mingw-w64-headers/crt/_mingw.h.in | 11 ++-
1 file changed, 6 insertions(+), 5 deletions(-)
On 18.12.2017 21:50, Martin Storsjö wrote:
> On Mon, 18 Dec 2017, Jacek Caban wrote:
>
>> This is a workaround, widl can't handle parametrized macro as #if
>> expression. Ideally, widl preprocessor should support this syntax.
>>
>> Signed-off-by: Jace
On 18.12.2017 21:56, Martin Storsjö wrote:
> On Mon, 18 Dec 2017, Jacek Caban wrote:
>
>>
>> It requires adding compatibility code for x86_64 msvcrt version. I don't
>> have arm setup to test, but I assume it's present there.
>
> Based on the msvcrt.dll from Win10
It requires adding compatibility code for x86_64 msvcrt version. I don't
have arm setup to test, but I assume it's present there.
Signed-off-by: Jacek Caban <ja...@codeweavers.com>
---
mingw-w64-crt/Makefile.am | 1 +
mingw-w64-crt/crt/crtexe.c | 2 +-
mingw-w64-c
it again.
Signed-off-by: Jacek Caban <ja...@codeweavers.com>
---
mingw-w64-headers/wine-import.sh | 189
+++
1 file changed, 189 insertions(+)
create mode 100755 mingw-w64-headers/wine-import.sh
diff --git a/mingw-w64-headers/wine-import.sh b/min
On 18.12.2017 21:57, Martin Storsjö wrote:
> On Mon, 18 Dec 2017, Jacek Caban wrote:
>
>> We were unable to do that for a long time, because of additional
>> mingw-w64 patches that were committed on top of upstream version. Those
>> patches started implementing winrt
Looks good to me.
Thanks,
Jacek
Signed-off-by: Jacek Caban <ja...@codeweavers.com>
--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/sl
Looks good as well.
Thanks,
Jacek
Signed-off-by: Jacek Caban <ja...@codeweavers.com>
--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/sl
.
That said, I propose to use upstream Wine upstream version and resurrect
import script in the tree. Dropped patches (needed for some winrt IDLs)
are easy to get from Git (just use a commit prior to this one). We could
even create a separated branch for that, if desired.
Signed-off-by: Jacek Caban <
On 16.11.2017 14:59, Martin Storsjö wrote:
> On Thu, 16 Nov 2017, Jacek Caban wrote:
>
>> Hi Martin,
>>
>> The patch looks generally good to me. I'm fine with committing it.
>>
>> As a side note, I expect this patch to also fix winpthread (built with
>&g
On 16.11.2017 14:40, Martin Storsjö wrote:
> On Thu, 16 Nov 2017, Jacek Caban wrote:
>
>> Hi Martin,
>>
>> On 15.11.2017 21:06, Martin Storsjö wrote:
>>> diff --git a/mingw-w64-crt/crt/ucrtbase_compat.c
>>> b/mingw-w64-crt/crt/ucrtbase_compat.c
>>>
Hi Martin,
On 15.11.2017 21:06, Martin Storsjö wrote:
> diff --git a/mingw-w64-crt/crt/ucrtbase_compat.c
> b/mingw-w64-crt/crt/ucrtbase_compat.c
> index c02c476..4f840ea 100644
> --- a/mingw-w64-crt/crt/ucrtbase_compat.c
> +++ b/mingw-w64-crt/crt/ucrtbase_compat.c
> @@ -24,6 +24,7 @@
> #include
Hi Martin,
The patch looks generally good to me. I'm fine with committing it.
As a side note, I expect this patch to also fix winpthread (built with
toolchain defaulting default msvcrt.dll) linking problems that I saw
when tried your patches.
On 15.11.2017 23:11, Martin Storsjö wrote:
> This
On 16.11.2017 20:46, Martin Storsjö wrote:
> So even with the latest MSVC, _snprintf retains the legacy behaviour,
> and the same goes for vsnprintf vs _vsnprintf.
That good to know. Thanks for testing.
Jacek
--
On 17.11.2017 10:12, Martin Storsjö wrote:
> + HANDLE ucrt;
> +
> + ucrt = GetModuleHandle("ucrtbase.dll");
> + real__tzset = (void __cdecl (*)(void)) GetProcAddress(ucrt, "_tzset");
Can we get away without GetModuleHandle()/GetProcAddress()? For example:
keep _tzset in .def file and call it
On 17.11.2017 18:45, Martin Storsjö wrote:
> On Fri, 17 Nov 2017, Jacek Caban wrote:
>
>> On 17.11.2017 10:12, Martin Storsjö wrote:
>>> + HANDLE ucrt;
>>> +
>>> + ucrt = GetModuleHandle("ucrtbase.dll");
>>> + real__tzset =
I would prefer to have the same names for headers and crt configure
arguments. Other than that, it looks good to me.
Thanks,
Jacek
On 11/10/17 10:07 PM, Kai Tietz via Mingw-w64-public wrote:
Patch looks ok to me. Jacek any comments?
Thanks
Kai
Am 10.11.2017 21:50 schrieb "Martin Storsjö"
On 14.11.2017 11:29, Shinchiro Shinchiro wrote:
> So how about the status of this issue?
It still needs debugging, AFAIK.
Cheers,
Jacek
--
Check out the vibrant tech community on one of the world's most
engaging tech
Hi Martin,
On 11/17/17 9:20 PM, Martin Storsjö wrote:
We can't redirect these to the function calls via defines, since "#define
timezone _timezone" will break any code that uses e.g. "struct timezone",
since "_timezone" is a define that expands into "(* __timezone())".
We can provide compat
Hi Shinchiro,
On 20.11.2017 10:46, Shinchiro Shinchiro wrote:
> Actually, I tried to create diff between wine's widl and mingw-w64's
> widl before but maybe it's difficult to cherry-pick changes that never
> got upstreamed
I can see that your patches remove changes from mingw-w64 branch. If
Hi Martin,
On 11/5/17 12:25 PM, Martin Storsjö wrote:
#ifdef __MINGW_INTRIN_INLINE
-#if !defined (__clang__)
+#if !defined (__clang__) || (__clang_major__ > 3 || __clang_major__ == 3 &&
__clang_minor__ >= 4)
Could __has_builtin be used instead?
Thanks,
Jacek
Hi Martin,
On 11/7/17 10:29 PM, Martin Storsjö wrote:
Normally calling code should use the right version of headers,
but the getopt implementation in libmingwex will have a hardcoded
reference to __imp___argv.
I think it would be better to add __p___argv (a trivial wrapper around
__argv) to
Hi Martin,
On 11/7/17 10:29 PM, Martin Storsjö wrote:
I've tested our support for ucrtbase.dll a bit further by trying building
a handful of libraries with it, and fixing the issues I run into. With
these patches, I'm able to build among others Qt (tested with 5.7.1).
That's good to hear.
On 07.11.2017 13:48, Martin Storsjö wrote:
> This simplifies the compat wrapper quite significantly, and the
> workarounds weren't sufficient in any case (the local __mb_cur_max
> value only was updated if the current module set the locale).
>
> Signed-off-by: Martin Storsjö
>
On 07.11.2017 13:48, Martin Storsjö wrote:
> This avoids having libmingwex refer to the dllimported data symbol
> from msvcrt.dll, easing linking to ucrtbase.dll which doesn't have
> it exported as a data symbol.
>
> ___mb_cur_max_func didn't exist in msvcrt.dll until Windows XP,
> while
Hello,
On 11/9/17 11:10 AM, Martin Storsjö wrote:
On Thu, 9 Nov 2017, Kai Tietz via Mingw-w64-public wrote:
Hmm, C99 support is pretty important for the gnu world. Nevertheless
patch ok. Please go ahead.
Yup, I know. However, right now if you build with msvcrt.dll and use
e.g. wprintf(),
On 11/9/17 1:08 PM, Martin Storsjö wrote:
Just one detail:
Note that supporting legacy behavior in this care is easy. One can
achieve that with -D__USE_MINGW_ANSI_STDIO=0.
Not quite - if you're in an environment that defaults to ucrtbase, you
need to set
sjö <mar...@martin.st>
> ---
> mingw-w64-crt/Makefile.am | 4
> 1 file changed, 4 deletions(-)
As soon as patch 1/2 is committed, this looks good.
Thanks,
Jacek
Signed-off-by: Jacek Caban <ja...@codeweavers.com>
--
-)
Looks good to me.
Thanks,
Jacek
Signed-off-by: Jacek Caban <ja...@codeweavers.com>
--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
___
Hi Martin,
On 09.11.2017 15:59, Martin Storsjö wrote:
> +AC_MSG_CHECKING([default __MSVCRT_VERSION__])
> +AC_ARG_WITH([default-msvcrt-version],
> + [AS_HELP_STRING([--with-default-msvcrt-version=VER],
> +[Default value of __MSVCRT_VERSION__ (default: 0x700)])],
> + [],
> +
TIME_INLINE __CRT_INLINE
> +#endif
As an unrelated side note, I'd be tempted to say that it would be a nice
thing to do for other versions as well.
Thanks,
Jacek
Signed-off-by: Jacek Caban <ja...@codeweavers.com>
Hi Martin,
On 09.11.2017 16:00, Martin Storsjö wrote:
> This makes sure that all code in libmingwex that refers to stderr
> etc will work correctly even if linked to ucrtbase.
>
> Signed-off-by: Martin Storsjö
> ---
> mingw-w64-crt/Makefile.am | 121
>
Hi Martin,
On 09.11.2017 20:33, Martin Storsjö wrote:
> if !W32API
> -lib32_DATA += lib32/libglut.a lib32/libmsvcp60.a lib32/libmsvcr80.a
> lib32/libmsvcr90.a lib32/libmsvcr90d.a lib32/libmsvcr100.a \
> - lib32/libmsvcr110.a lib32/libmsvcr120.a
> lib32/libmsvcr120d.a
| 8 +--
> mingw-w64-headers/crt/stdio.h | 9 ---
> mingw-w64-headers/crt/wchar.h | 9 ---
> 6 files changed, 135 insertions(+), 56 deletions(-)
> create mode 100644 mingw-w64-crt/stdio/acrt_iob_func.c
Looks good now.
Thanks,
On 08.11.2017 14:30, Martin Storsjö wrote:
> On Wed, 8 Nov 2017, Jacek Caban wrote:
>
>> Hi Martin,
>>
>>
>> On 11/7/17 10:29 PM, Martin Storsjö wrote:
>>> I've tested our support for ucrtbase.dll a bit further by trying
>>> building
>>> a
Hi Martin,
On 07.11.2017 22:29, Martin Storsjö wrote:
> Alternatively, we could just skip providing these when building with
> ucrtbase, but that would require updating any calling code.
That's a tempting solution. It seems to be deprecated for years now. Do
you know a real code that needs it?
changed, 2 insertions(+), 2 deletions(-)
Looks good to me.
Thanks,
Jacek
Signed-off-by: Jacek Caban <ja...@codeweavers.com>
--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slash
..@martin.st>
> ---
> mingw-w64-crt/lib-common/msvcrt.def.in | 2 ++
> 1 file changed, 2 insertions(+)
Looks good to me.
Thanks,
Jacek
Signed-off-by: Jacek Caban <ja...@codeweavers.com>
--
Check out
On 08.11.2017 23:19, Martin Storsjö wrote:
> + // We have a fallback definition of __p___argv for msvcrt versions that
> + // lack it, so always declare it as available, but without _CRTIMP.
> + char ***__cdecl __p___argv(void);
How about using __MINGW_IMP_SYMBOL in compat implementation so
t/lib32/crtdll.def | 1 +
> mingw-w64-crt/lib32/msvcr80.def.in | 1 +
> mingw-w64-crt/lib64/crtdll.def | 1 +
> mingw-w64-crt/lib64/msvcr80.def.in | 1 +
> 5 files changed, 4 insertions(+), 16 deletions(-)
Looks good to me.
Thanks,
Jacek
Signed-off
On 08.11.2017 23:19, Martin Storsjö wrote:
> - Fix getopt by using __p___argv there instead of __argv, as Jacek
> suggested. In order to do this, I ended up cleaning up a few other
> inconsistencies surrounding the build of msvcr* dlls/defs while
> I was touching that area, cleanly separated
On 08.11.2017 23:19, Martin Storsjö wrote:
> We provide a version of __p___argv in all msvcrt versions now. This
> allows linking getopt from libmingwex against ucrtbase.
>
> Signed-off-by: Martin Storsjö
> ---
> mingw-w64-crt/misc/getopt.c | 2 +-
> 1 file changed, 1
On 07.11.2017 15:42, Liu Hao wrote:
> On 2017/11/7 21:55, Jacek Caban wrote:
>> On 07.11.2017 14:42, Shinchiro Shinchiro wrote:
>>
>>> There's bug in some mingw-w64's idls which give segfault with
>>> widl which of course fixed in wine
>>
>> Yeah, imp
On 07.11.2017 17:03, Shinchiro Shinchiro wrote:
> Forgot to mention the compiler, widl actually is working fine. The
> culprit is in some idl files which might be outdated which produce
> segfault (in dxgi1_6.idl case)
That's interesting. Do you know which ones?
BTW, widl crash is a bug in widl
On 07.11.2017 14:42, Shinchiro Shinchiro wrote:
> At the moment, only directx idl is imported from wine. I want to suggest
> you to import other IDLs from wine/include too if it doesn't break
> anything.
That's not really true, there are some other IDLs. I occasionally import
some more. If you
Based on patch by Tom Ritter.
Tom explained it here:
https://sourceforge.net/p/mingw-w64/mailman/message/36238073/
I added WIDL_EXPLICIT_AGGREGATE_RETURNS preprocessor guards so that it's
not used then not needed.
Signed-off-by: Jacek Caban <ja...@codeweavers.com>
---
mingw-w64-headers/i
Based on patch by Tom Ritter.
Signed-off-by: Jacek Caban <ja...@codeweavers.com>
---
mingw-w64-headers/include/d2d1.h | 129
+++
1 file changed, 129 insertions(+)
diff --git a/mingw-w64-headers/include/d2d1.h b/mingw-w64-headers/include/d2d1.h
On 05/15/2018 05:25 PM, Liu Hao wrote:
> The trick should work, but it can still be improved a little by changing
>
> ```
> + D2D1_SIZE_F __ret;
> + return *GetSize(&__ret);
> ```
>
> to
>
> ```
> + D2D1_SIZE_F __ret;
> + GetSize(&__ret);
> + return __ret;
> ```
Based on patch by Tom Ritter.
Signed-off-by: Jacek Caban <ja...@codeweavers.com>
---
mingw-w64-headers/include/d2d1.h | 125
+++
1 file changed, 125 insertions(+)
diff --git a/mingw-w64-headers/include/d2d1.h b/mingw-w64-headers/include/d2d1.h
On 05/22/2018 01:02 PM, Liu Hao wrote:
> 在 2018/5/22 17:46, Jacek Caban 写道:
>> +#ifndef WIDL_EXPLICIT_AGGREGATE_RETURNS
>> STDMETHOD_(D2D1_SIZE_F, GetSize)(void) const PURE;
>> +#else
>> + virtual D2D1_SIZE_F* STDMETHODCALLTYPE GetSize(D2D1_SIZE_F
On 06/14/2018 12:09 PM, Liu Hao wrote:
> 在 2018/6/14 16:16, Martin Storsjo 写道:
>> This reverts commit cd954f93225128f352c80d680096f880749eccb7.
>>
>> That commit was nonsense; this function is supposed to clean up
>> a path name in-place in the source buffer, if possible. By using
>> a separate
Signed-off-by: Jacek Caban
---
mingw-w64-crt/Makefile.in | 2 +-
mingw-w64-crt/{libarm32 => lib-common}/wscapi.def | 0
mingw-w64-crt/lib64/Makefile.am | 1 +
3 files changed, 2 insertions(+), 1 deletion(-)
rename mingw-w64-crt/{libarm32 => lib-
On 06/13/2018 03:05 PM, Martin Storsjö wrote:
> On Wed, 13 Jun 2018, Jacek Caban wrote:
>
>> Signed-off-by: Jacek Caban
>> ---
>> mingw-w64-crt/Makefile.in | 2 +-
>> mingw-w64-crt/{libarm32 => lib-common}/wscapi.def | 0
&
Signed-off-by: Jacek Caban
---
mingw-w64-headers/include/dcomp.h | 46
+++
1 file changed, 23 insertions(+), 23 deletions(-)
diff --git a/mingw-w64-headers/include/dcomp.h b/mingw-w64-headers/include/dcomp.h
index 0b4c362d..160f3de7 100644
--- a/mingw-w64
On 06/13/2018 08:57 PM, Martin Storsjö wrote:
> On Wed, 13 Jun 2018, Jacek Caban wrote:
>
>> Signed-off-by: Jacek Caban
>> ---
>> mingw-w64-headers/crt/stdio.h | 9 -
>> 1 file changed, 8 insertions(+), 1 deletion(-)
>
> LGTM
>
> Signe
Signed-off-by: Jacek Caban
---
mingw-w64-headers/crt/stdio.h | 9 -
1 file changed, 8 insertions(+), 1 deletion(-)
diff --git a/mingw-w64-headers/crt/stdio.h b/mingw-w64-headers/crt/stdio.h
index 8a240e10..49c84e65 100644
--- a/mingw-w64-headers/crt/stdio.h
+++ b/mingw-w64-headers/crt
On 06/27/2018 08:33 AM, Martin Storsjö wrote:
> This adds def files for all the api-ms-win-crt-* dll files. I've tried to
> add all the manual modifications from ucrtbase.def.in into these files, so
> that linking to libucrt.a should work pretty much the same as linking to
> libucrtbase.a, but
On 06/26/2018 01:56 PM, Liu Hao wrote:
> 在 2018/6/26 19:33, Jacek Caban 写道:
>> I did tests with c89 and c99 before submitting the patch (C++ is
>> irrelevant, I don't change behaviour there). I'm also able to build
>> Firefox with this patch (which in the past revealed a few i
On 06/14/2018 01:25 PM, Liu Hao wrote:
> 在 2018年06月14日 18:49, Jacek Caban 写道:
>> On 06/14/2018 04:15 AM, Liu Hao wrote:
>>> 在 2018年06月14日 00:47, Jacek Caban 写道:
>>>> + __mingw_ovr
>>>> + int __cdecl _vscwprintf(const wchar_t * __
Hello,
On 27/06/2018 07:52, Martin Storsjö wrote:
I did a test build with this patch, and it failed at building
winpthreads, with the following error:
Yeah, I can reproduce problems. Sorry for not catching it earlier,
initial testing looked promising.
And just to make it clear - this is
in that case (also __ptr32, __unaligned and __w64, so
I changed them as well while I was at it). Without -fms-extensions,
__always_inline__ is expected and as far as I can tell (and my testing
conforms that), it matches intended behaviour without additional attributes.
Signed-off-by: Jacek Caban
---
mingw
On 06/26/2018 01:14 PM, Liu Hao wrote:
> 在 2018年06月26日 18:35, Jacek Caban 写道:
>> I still need to run some test, but I it seems ready for review.
>>
>> The important change is __forceinline. We add extern to GCC macro,
>> because that matches closer intende
On 06/26/2018 01:44 PM, Martin Storsjö wrote:
> On Tue, 26 Jun 2018, Jacek Caban wrote:
>
>> I still need to run some test, but I it seems ready for review.
>>
>> The important change is __forceinline. We add extern to GCC macro,
>> because that matches closer inte
On 26.10.2017 20:13, Tom Ritter wrote:
> We missed one, here is the additional patch.
Looks good, I pushed it to the repo.
Thanks,
Jacek
--
Check out the vibrant tech community on one of the world's most
engaging tech
Fixes compilation with clang.
Signed-off-by: Jacek Caban <ja...@codeweavers.com>
---
mingw-w64-headers/include/psdk_inc/intrin-impl.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/mingw-w64-headers/include/psdk_inc/intrin-impl.h b/mingw-w64-headers/include/psdk_inc/
Signed-off-by: Jacek Caban <ja...@codeweavers.com>
---
mingw-w64-headers/include/psdk_inc/intrin-impl.h | 83
1 file changed, 41 insertions(+), 42 deletions(-)
diff --git a/mingw-w64-headers/include/psdk_inc/intrin-impl.h b/mingw-w64-headers/include/psdk_inc/
On 01/12/2018 03:25 AM, Liu Hao wrote:
> On 2018/1/12 4:28, Jacek Caban wrote:
>> Fixes compilation with clang.
>>
>> Signed-off-by: Jacek Caban <ja...@codeweavers.com>
>> ---
>> mingw-w64-headers/include/psdk_inc/intrin-impl.h | 16
Hi,
On 1/20/18 8:54 PM, Mateusz wrote:
Hello,
The patch "intrin-impl.h: Use volatile argument for __buildbittesti-based
functions."
is not working well for me (I can't compile GCC with this patch).
The problem is with one extra 'volatile':
@@ -615,28 +614,28 @@ __buildstos(__stosq,
clang doesn't support it, so it emits tons of warnings.
Signed-off-by: Jacek Caban <ja...@codeweavers.com>
---
mingw-w64-headers/crt/stdio.h | 21 +
1 file changed, 13 insertions(+), 8 deletions(-)
diff --git a/mingw-w64-headers/crt/stdio.h b/mingw-w64-headers/crt/s
Hi Tom,
On 01/22/2018 07:15 PM, Tom Ritter wrote:
> mingw-w64-headers/include/error.h| 1 +
> mingw-w64-headers/include/winerror.h | 1 +
> mingw-w64-headers/include/winnt.h| 29 -
> mingw-w64-tools/widl/include/winnt.h | 1 +
The version in widl/
On 01/12/2018 09:14 AM, Martin Storsjö wrote:
> This patch doesn't make much sense for use in wine and probably
> can't be upstreamed as such.
Well, I think that such feature could be accepted to Wine, but current
code doesn't seem to be in a shape acceptable upstream.
> This also reapplies the
On 01/12/2018 09:14 AM, Martin Storsjö wrote:
> None of the imported files actually are part of the autoconf/automake
> source files, so no import currently would actually require
> rerunning this.
>
> If the import script is adjusted to touch Makefile.am or configure.ac,
> such a command probably
On 01/12/2018 03:36 AM, Liu Hao wrote:
> On 2018/1/12 4:31, Jacek Caban wrote:
>> Signed-off-by: Jacek Caban <ja...@codeweavers.com>
>> ---
>> mingw-w64-crt/Makefile.am | 1 +
>> mingw-w64-crt/misc/onexit_table.c | 74
>>
On 01/12/2018 09:14 AM, Martin Storsjö wrote:
> There is no such file present in upstream wine, nor in the current
> mingw-w64 tree either.
>
> Signed-off-by: Martin Storsjö
> ---
> mingw-w64-tools/widl/wine-import.sh | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>
On 01/12/2018 09:14 AM, Martin Storsjö wrote:
> These seem to have been removed by mistake as part of a change
> touching the line above, in 4581de347f.
>
> Signed-off-by: Martin Storsjö
> ---
> mingw-w64-tools/widl/Makefile.am | 2 +-
> 1 file changed, 1 insertion(+), 1
On 01/12/2018 09:14 AM, Martin Storsjö wrote:
> Some of the imported source files are autogenerated, and if wine
> was built outside of the source tree, the caller of wine-import.sh
> needs to provide paths to both trees.
>
> In such an import, the generated bison/flex files will have stray
>
301 - 400 of 810 matches
Mail list logo