No, sorry, I don't have the setup to build mingw. Not likely that I will
have time to do it any time soon either.
I meant this as a bug report. I hope the problem has been investigated
sufficiently for mingw devs to act on it.
Vadim
On Mon, Aug 18, 2014 at 9:34 PM, dw wrote:
> Did you ever
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
Am 17.08.2014 um 16:10 schrieb Ruben De Smet:
> Hi mingw-w64 list and especially André Hentschel, (CC: a teammember of mine)
>
> I decided to mail this directly to mr Hentschel too, as I was told you were
> the one working on these kind of problems.
OK, I pushed it with "extern inline" declaration.
Thanks for reviews,
Jacek
On 08/19/14 15:38, Kai Tietz wrote:
> Hmm, this patch looks to me not 100% ok.
>
> Making functions static inline seems to be not the proper thing to do
> here AFAICS. This will lead to warning for cases those functions
Yes, wasn't at that file ... patch is ok.
Please apply.
Thanks,
Kai
2014-08-19 14:04 GMT+02:00 Jacek Caban :
> It came out that we need to not duplicate QueryInterface in child
> interface declarations, because they shadow its templated inline variant.
>
> ---
> mingw-w64-headers/include/dwrit
Thanks, please apply.
Kai
2014-08-19 14:31 GMT+02:00 Jacek Caban :
> ---
> mingw-w64-headers/include/tsattrs.h | 100
>
> 1 file changed, 100 insertions(+)
> create mode 100644 mingw-w64-headers/include/tsattrs.h
>
>
>
>
Patch is ok. Please apply.
Thanks,
Kai
2014-08-19 14:05 GMT+02:00 Jacek Caban :
> ---
> mingw-w64-headers/include/dwrite_1.h | 148
> +++
> 1 file changed, 148 insertions(+)
> create mode 100644 mingw-w64-headers/include/dwrite_1.h
>
>
>
> --
Thanks, please apply.
Kai
2014-08-19 14:04 GMT+02:00 Jacek Caban :
> ---
> mingw-w64-headers/include/wlanapi.h | 11 +++
> 1 file changed, 11 insertions(+)
>
>
>
> --
>
> _
Patch is ok.
Thanks,
Kai
2014-08-19 14:04 GMT+02:00 Jacek Caban :
> ---
> mingw-w64-headers/include/l2cmn.h | 20
> 1 file changed, 20 insertions(+)
> create mode 100644 mingw-w64-headers/include/l2cmn.h
>
>
>
> --
Hmm, this patch looks to me not 100% ok.
Making functions static inline seems to be not the proper thing to do
here AFAICS. This will lead to warning for cases those functions
aren't used. So I would think that attribute unused is missing here.
Additionally the 'inline' looks to me bogus. Eithe
Patch is ok.
Thanks,
Kai
2014-08-19 14:04 GMT+02:00 Jacek Caban :
> ---
> mingw-w64-headers/include/sddl.h | 8
> 1 file changed, 8 insertions(+)
>
>
>
> --
>
> ___
>
---
mingw-w64-headers/include/tsattrs.h | 100
1 file changed, 100 insertions(+)
create mode 100644 mingw-w64-headers/include/tsattrs.h
diff --git a/mingw-w64-headers/include/tsattrs.h b/mingw-w64-headers/include/tsattrs.h
new file mode 100644
index 000.
---
mingw-w64-headers/crt/sec_api/string_s.h | 3 +++
mingw-w64-headers/crt/sec_api/wchar_s.h | 3 +++
2 files changed, 6 insertions(+)
diff --git a/mingw-w64-headers/crt/sec_api/string_s.h b/mingw-w64-headers/crt/sec_api/string_s.h
index 34d1c99..4b10820 100644
--- a/mingw-w64-headers/crt/sec_
---
mingw-w64-headers/include/l2cmn.h | 20
1 file changed, 20 insertions(+)
create mode 100644 mingw-w64-headers/include/l2cmn.h
diff --git a/mingw-w64-headers/include/l2cmn.h b/mingw-w64-headers/include/l2cmn.h
new file mode 100644
index 000..8157df9
--- /dev/null
+++
It came out that we need to not duplicate QueryInterface in child
interface declarations, because they shadow its templated inline variant.
---
mingw-w64-headers/include/dwrite.h | 59
--
1 file changed, 57 insertions(+), 2 deletions(-)
diff --git a/mingw-w64
---
mingw-w64-headers/include/wlanapi.h | 11 +++
1 file changed, 11 insertions(+)
diff --git a/mingw-w64-headers/include/wlanapi.h b/mingw-w64-headers/include/wlanapi.h
index fa0e506..1f7dc49 100644
--- a/mingw-w64-headers/include/wlanapi.h
+++ b/mingw-w64-headers/include/wlanapi.h
@@ -
---
mingw-w64-headers/include/sddl.h | 8
1 file changed, 8 insertions(+)
diff --git a/mingw-w64-headers/include/sddl.h b/mingw-w64-headers/include/sddl.h
index 0bf19ba..c4ae5aa 100644
--- a/mingw-w64-headers/include/sddl.h
+++ b/mingw-w64-headers/include/sddl.h
@@ -23,6 +23,7 @@ extern
---
mingw-w64-headers/include/dwrite_1.h | 148
+++
1 file changed, 148 insertions(+)
create mode 100644 mingw-w64-headers/include/dwrite_1.h
diff --git a/mingw-w64-headers/include/dwrite_1.h b/mingw-w64-headers/include/dwrite_1.h
new file mode 100644
index 0
Patch is ok.
Thanks,
Kai
2014-08-18 23:35 GMT+02:00 André Hentschel :
> Please review, i'll commit it, just need an OK.
> (these are the rather easy ones, it mostly looks like the RFC i sent some
> days ago)
>
> --
>
> _
18 matches
Mail list logo