Types implementing interfaces are handled incorrectly

[DefaultRyan]

Take CFunctionDiscoveryNotificationWrapper. It's defined as a default implementation of IFunctionDiscoveryNotification like so:

class CFunctionDiscoveryNotificationWrapper : public IFunctionDiscoveryNotification
{
  // Implements all 3 of IFunctionDiscoveryNotification methods with a stub, returning S_OK
}

CFunctionDiscoveryNotificationWrapper is appearing in the metadata, but it has two big issues:

• It provides duplicate definitions of the 3 override methods. Specifically, IFunctionDiscoveryNotification has three methods in addition to IUnknown: OnUpdate, OnError, and OnEvent. CFunctionDiscoveryNotificationWrapper erroneously duplicates these three methods in its own type definition.
• It is defined as an interface in the metadata. Seems like this should probably be a class. (Or perhaps not in the metadata at all?)

[AArnott]

I thought the goal was to remove all code from the metadata dll.

[sotteson1]

Andrew's right, we don't want code in there. I'll exclude this one.