Re: OleFont: Fix IDispatch Implementation (was Re: olefont: GetIDsOfNames)

2005-09-16 Thread Robert Shearman
Walt Ogburn wrote: I'm not sure how to proceed here. My small patch makes the GetIDsOfNames in olefont work, so that programs that want to use GetIDsOfNames to get a DISPID code and then use Invoke to set font size, bold, etc. will work. With your bigger patch, IFontDisp uses the GetIDsOfNames

Re: OleFont: Fix IDispatch Implementation (was Re: olefont: GetIDsOfNames)

2005-09-15 Thread Walt Ogburn
On Wed, 14 Sep 2005, Robert Shearman wrote: > > You're not being lazy enough. All of this stuff can be easily > implemented in one go using CreateStdDispatch, as long as you know how > to use it. Try the attached patch. > > Changelog: > Return the standard font IDispatch interface created using > C

Re: OleFont: Fix IDispatch Implementation (was Re: olefont: GetIDsOfNames)

2005-09-14 Thread Walt Ogburn
Wait a minute, hold that - after this patch, you can't use Invoke any more to get and set the members of IFontDisp! With the current code, it works. Alexandre, please don't commit that yet. Thanks, Walter On Wed, 14 Sep 2005, Robert Shearman wrote: > Walt Ogburn wrote: > > >Changelog: > >

Re: OleFont: Fix IDispatch Implementation (was Re: olefont: GetIDsOfNames)

2005-09-14 Thread Walt Ogburn
On Wed, 14 Sep 2005, Robert Shearman wrote: > > You're not being lazy enough. All of this stuff can be easily > implemented in one go using CreateStdDispatch, as long as you know how > to use it. Try the attached patch. > > Changelog: > Return the standard font IDispatch interface created using > C

OleFont: Fix IDispatch Implementation (was Re: olefont: GetIDsOfNames)

2005-09-14 Thread Robert Shearman
Walt Ogburn wrote: Changelog: Implement IFontDisp::GetIDsOfNames You're not being lazy enough. All of this stuff can be easily implemented in one go using CreateStdDispatch, as long as you know how to use it. Try the attached patch. Changelog: Return the standard font IDispatch

Re: olefont: GetIDsOfNames

2005-09-14 Thread Alexandre Julliard
Walt Ogburn <[EMAIL PROTECTED]> writes: >OLEFontImpl *this = impl_from_IDispatch(iface); > - FIXME("(%p,%s,%p,%d,%04x,%p), stub!\n", this, debugstr_guid(riid), > rgszNames, > - cNames, (int)lcid, rgDispId > - ); > - return E_NOTIMPL; > + TRACE("(%p,%s,%p,%d,%04x,%p)\n", this, debugstr