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 GetIDsOf

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

2005-09-15 Thread Walt Ogburn
ch interface created using > CreateStdDispatch on top of a typelib instead of manually implementing > the necessary methods. > Hi Robert, 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

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

2005-09-14 Thread Walt Ogburn
gt; > >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. > > Ch

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

2005-09-14 Thread Walt Ogburn
ch interface created using > CreateStdDispatch on top of a typelib instead of manually implementing > the necessary methods. > That doesn't look very lazy to me, but it simplifies olefont very nicely (300 fewer lines!). Please use this patch for GetIDsOfNames instead of mine. My patc

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

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

Re: GetIDsOfNames

2005-09-12 Thread Walt Ogburn
TypeInfo needs to get IFontDisp from stdole2.tlb instead of IDispatch from stdole32.tlb. That way, you can use GetTypeInfo and DispGetIDsOfNames together to implement GetIDsOfNames. I'll submit some tests and proposed patches soon. - Walter

Re: GetIDsOfNames

2005-09-12 Thread Robert Shearman
problem fixed). Next, there is the issue that some of them call GetIDsOfNames on ole font objects (countries of Europe, http://olivier.leflon.free.fr/jeux/europe.htm). OLEFontImpl_GetIDsOfNames is currently a stub. What's needed is basically just to convert the strings "Name", &quo

GetIDsOfNames

2005-09-11 Thread Walt Ogburn
, there is the issue that some of them call GetIDsOfNames on ole font objects (countries of Europe, http://olivier.leflon.free.fr/jeux/europe.htm). OLEFontImpl_GetIDsOfNames is currently a stub. What's needed is basically just to convert the strings "Name", "Size", "Bol