Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Enable classic COM on mingw-w64#1215

Merged
kennykerr merged 4 commits intomicrosoft:mastermicrosoft/cppwinrt:masterfrom
alvinhochun:alvin/mingw-comalvinhochun/cppwinrt:alvin/mingw-comCopy head branch name to clipboard
Oct 24, 2022
Merged

Enable classic COM on mingw-w64#1215
kennykerr merged 4 commits intomicrosoft:mastermicrosoft/cppwinrt:masterfrom
alvinhochun:alvin/mingw-comalvinhochun/cppwinrt:alvin/mingw-comCopy head branch name to clipboard

Conversation

@alvinhochun
Copy link
Contributor

Mingw-w64 does not use __declspec(uuid(x)) but has its own way of supporting __uuidof by defining UUIDs with a special macro __CRT_UUID_DECL, which does actually work here.

Mingw-w64 does not use `__declspec(uuid(x))` but has its own way of
supporting `__uuidof` by defining UUIDs with a special macro
`__CRT_UUID_DECL`, which does actually work here.
@kennykerr
Copy link
Collaborator

Here we are again with classic COM. 😅 @sylveon do you want to take a stab at reviewing this, given your recent overhaul of this code?

@sylveon
Copy link
Contributor

sylveon commented Oct 21, 2022

From a quick read, it appears that everything is fine here.

@kennykerr
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@kennykerr kennykerr merged commit d6ef811 into microsoft:master Oct 24, 2022
@alvinhochun alvinhochun deleted the alvin/mingw-com branch December 11, 2022 16:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.