I think we can live with this. Please include your clarification as comment to the code.
-->Henry > -----Original Message----- > From: [EMAIL PROTECTED] > [mailto:[EMAIL PROTECTED] On Behalf Of Bo Tian > Sent: Thursday, June 22, 2006 2:50 PM Ping > To: 'Greg Wright' > Cc: [email protected] > Subject: RE: [Audio-dev] CR: CLSID_HXAUDIODEVICEHOOK change > for TLC ofRealPlayer > > Here is a little bit clarification: > > 1). Actually, CLSID_HXAUDIODEVICEHOOK is only used as class > identifier of the DMO object being searched for, and that > class implements actual interface of > IID_IHXAudioDeviceHookDMO, so basically the CLSID could be > different and the interface ID is never changed though. > > 2). Since the RealPlayer needs to be sandboxed from other > applications as well as Rhapsody, they cannot share the same > global CLSID. We need a solution that allows for any product > to be able to use a different CLSID for the global objects so > they can be sandboxed. > > We are looking for better solution to this problem. Please > let us know if anybody has some good idea. > > Thanks, > Bo > > -----Original Message----- > From: Greg Wright [mailto:[EMAIL PROTECTED] > Sent: Wednesday, June 21, 2006 3:42 PM > To: Bo Tian > Cc: [email protected] > Subject: Re: [Audio-dev] CR: CLSID_HXAUDIODEVICEHOOK change > for TLC of RealPlayer > > First, just remember: > > +#ifdef HELIX_FEATURE_REALPLAYER > +DEFINE_GUID(CLSID_HXAUDIODEVICEHOOK, 0xb84b8d32, 0xa391, > 0x4308, 0x93, > 0x71, 0x7d, 0x51, 0xc9, 0xc6, 0x5b, 0x3); > +#else > DEFINE_GUID(CLSID_HXAUDIODEVICEHOOK, 0X2CFA30DA, 0X118B, > 0X4CA3, 0XAA, 0XF3, 0XF4, 0X74, 0X16, 0X23, 0X2, 0XE5); > +#endif > + > > that !HELIX_FEATURE_REALPLAYER != Rhapsody. That is, there > can be lots of applications that build this code that are not > either one of those apps. > > I don't think any of them will cause problems like the one > your are trying to fix though. > > Second, is this the only way to fix the problem? I really > hate the need to have different GUIDs for the same interface. > Seems like we can get ourselves into trouble that way. > > --greg. > > > > > Bo Tian wrote: > > Modified by: [EMAIL PROTECTED] > > > > Reviewed by: [EMAIL PROTECTED] > > > > Date: 6/16/2006 > > > > Project: audio > > > > > > > > Synopsis: Currently, HXAudioDeviceHook.dll is shipped and > registered > > by Rhapsody; so if same class id (CLSID_HXAUDIODEVICEHOOK) > is used by > > RealPlayer, this DLL will be shared; thus we want a new > class ID for > > this DLL specific for RealPlayer. > > > > > > > > Overview: > > > > N/A > > > > > > > > Files Modified: > > > > Audio/device/platform/win/HXAudioDeviceHook/HXAudioDeviceHook.h > > > > > > > > Image Size and Heap Use impact (Client-Only): > > > > N/A > > > > > > > > Platforms and Profiles Affected: > > > > All > > > > > > > > Distribution Libraries Affected: > > > > N/A > > > > > > > > Distribution library impact and planned action: > > > > N/A > > > > > > > > Platforms and Profiles Build Verified: > > > > Windows > > > > helix-client-all-defines > > > > > > > > Platforms and Profiles Functionality verified: > > > > Windows > > > > helix-client-all-defines > > > > > > > > Branch: > > > > Thxclient_1_5_0_cayenne > > > > > > > > > > > ---------------------------------------------------------------------- > > -- > > > > _______________________________________________ > > Audio-dev mailing list > > [email protected] > > http://lists.helixcommunity.org/mailman/listinfo/audio-dev > > > _______________________________________________ > Audio-dev mailing list > [email protected] > http://lists.helixcommunity.org/mailman/listinfo/audio-dev _______________________________________________ Audio-dev mailing list [email protected] http://lists.helixcommunity.org/mailman/listinfo/audio-dev
