Yes. I don't think it's a problem that one GUID would be different, it
would be just a good example for future improvements, but your patch is
fine as well.
On 02/04/15 17:13, Martell Malone wrote:
Will I consider this reviewed?
Just need the go ahead
On Wed, Feb 4, 2015 at 3:23 PM, Martell
Will I consider this reviewed?
Just need the go ahead
On Wed, Feb 4, 2015 at 3:23 PM, Martell Malone martellmal...@gmail.com
wrote:
I know that it's not what we do in other cases, but something like that
would avoid hardcoding this uuid in two places:
DEFINE_GUID(KSCATEGORY_
ENCODER,
---
mingw-w64-headers/crt/memory.h | 1 +
mingw-w64-headers/crt/sec_api/memory_s.h | 26 --
mingw-w64-headers/crt/sec_api/string_s.h | 5 -
mingw-w64-headers/crt/string.h | 1 +
4 files changed, 2 insertions(+), 31 deletions(-)
delete mode
Please Review :)
0001-added-new-file-functiondiscoverykeys_devpkey.h-and-k.patch
Description: Binary data
--
Dive into the World of Parallel Programming. The Go Parallel Website,
sponsored by Intel and developed in
Hi Martell,
On 02/04/15 12:10, Martell Malone wrote:
+#warning COM interfaces layout in this header has not been verified.
+#warning COM interfaces with incorrect layout may not work at all.
+__MINGW_BROKEN_INTERFACE(INTERFACE)
The interface seems correct, why do you add this warning? Same
Hi Georg,
On 02/03/15 11:53, Georg Koppen wrote:
Hi,
I am putting a patch inline (not sure if attachments are allowed on this
mailing list) that does not make it possible for a local process anymore
to replace canaries which in turn might disable SSP. Comments and/or
review are much
Small header changes for various missing uuid's and 2 classes
Please Review
0001-Added-IEnumPIDMap-and-IMPEG2PIDMap-to-bdaiface.h.patch
Description: Binary data
0001-Add-some-missing-uuid-s.patch
Description: Binary data
0001-include-ksmedia-uuids-in-libuuid.patch
Description: Binary data
The interface seems correct, why do you add this warning? Same for
IMPEG2PIDMap
All the other interfaces in the file also have this warning so I didn't
want to remove it.
Attached updated patch :)
Those two should be in wmcodecdsp.idl AFAICS.
Yes as discussed on IRC there are a few uuid's
Hi Martell,
Those patches look good to me now. I just have one suggestion.
On 02/04/15 16:14, Martell Malone wrote:
DEFINE_GUID(KSCATEGORY_DATADECOMPRESSOR,0x2721ae20,0x7e70,0x11d0,0xa5,0xd6,0x28,0xdb,0x4,0xc1,0,0);
I know that it's not what we do in other cases, but something like that
would avoid hardcoding this uuid in two places:
DEFINE_GUID(KSCATEGORY_
ENCODER, STATIC_KSCATEGORY_ENCODER);
Yes I was going todo it that way but I saw that many of the others from
that header were defined already and
10 matches
Mail list logo