Re: [1/3] include: Add COM interface definitions needed for PrintDlgEx implementation.

2013-04-09 Thread Dmitry Timoshkov
Vincent Povirk wrote: > Does PrintDlgEx really use an interface with the same iid but > different function signatures depending on whether the A or W version > is called? If so, I think it would be best to define both versions, so > that our tests and implementation can easily provide/use both.

Re: [1/3] include: Add COM interface definitions needed for PrintDlgEx implementation.

2013-04-09 Thread Vincent Povirk
>> And why bother checking UNICODE? > > Please clarify. +MIDL_INTERFACE("509aaeda-5639-11d1-b6a1-f8757bf9") +IPrintDialogServices : public IUnknown +{ +#ifdef UNICODE +virtual HRESULT STDMETHODCALLTYPE GetCurrentDevMode(LPDEVMODEW,UINT *) = 0; +virtual HRESULT STDMETHODCALLTYPE GetCurr

Re: [1/3] include: Add COM interface definitions needed for PrintDlgEx implementation.

2013-04-08 Thread Dmitry Timoshkov
Nikolay Sivov wrote: > Why not use DECLARE_INTERFACE_ here? Because it seemed simpler to me without it. > And why bother checking UNICODE? Please clarify. -- Dmitry.

Re: [1/3] include: Add COM interface definitions needed for PrintDlgEx implementation.

2013-04-08 Thread Nikolay Sivov
Why not use DECLARE_INTERFACE_ here? And why bother checking UNICODE? On Mon, Apr 8, 2013 at 12:10 PM, Dmitry Timoshkov wrote: > --- > dlls/uuid/uuid.c | 1 + > include/commdlg.h | 80 > +++ > 2 files changed, 81 insertions(+) > > diff --gi