[cfe-commits] [patch] Add codegen support for __uuidof

Kim Gräsman kim.grasman at gmail.com
Mon Oct 1 00:12:29 PDT 2012


Hi Nico,

On Mon, Oct 1, 2012 at 8:56 AM, Nico Weber <thakis at chromium.org> wrote:
>
> The name of the symbol generated for __uuidof constants seems to be an
> implementation detail, so I just made up a mangling ("__uuid_"
> followed by the contents of the uuid, see GetAddrOfIIDDescriptor()).

It's been years since I did any COM, and I might be missing a nuance,
but should the method name really be GetAddrOfIIDDescriptor? I think
__declspec(uuid()) can apply to non-interface constructs as well,
typically coclasses. Singling it out as an IID might be too narrow,
even if IID and CLSID are type compatible in the C++ sense.

How about GetAddrOfUUIDDescriptor?

- Kim



More information about the cfe-commits mailing list