#1057 Incorrect declaration of DWebBrowserEvents2

closed
Danny Smith
2014-09-22
2007-07-11
Yuji Kuwabara
No

In w32api-3.9, "exdisp.h", I think those "STDDISP()" should not be placed in the interface declaration. Those are not member functions. Instead, single function IDispatch::Invoke() is used.
Please refer to Microsoft Platform SDK header file.
See also:
http://msdn2.microsoft.com/en-us/library/ms221479.aspx

---------------------------------------------
DECLARE_INTERFACE_(DWebBrowserEvents2,IDispatch)
{
:
:
STDDISP(StatusTextChange)(THIS_ BSTR) PURE;
STDDISP(ProgressChange)(THIS_ long,long) PURE;
STDDISP(CommandStateChange)(THIS_ long,VARIANT_BOOL) PURE;
STDDISP(DownloadBegin)(THIS) PURE;
STDDISP(DownloadComplete)(THIS) PURE;
STDDISP(TitleChange)(THIS_ BSTR) PURE;
STDDISP(PropertyChange)(THIS_ BSTR) PURE;
STDDISP(BeforeNavigate2)(THIS_ IDispatch*,VARIANT*,VARIANT*,VARIANT*,VARIANT*,VARIANT*,VARIANT_BOOL*) PURE;
STDDISP(NewWindow2)(THIS_ IDispatch**,VARIANT_BOOL*) PURE;
STDDISP(NavigateComplete)(THIS_ IDispatch*,VARIANT*) PURE;
STDDISP(DocumentComplete)(THIS_ IDispatch*,VARIANT*) PURE;
STDDISP(OnQuit)(THIS) PURE;
STDDISP(OnVisible)(THIS_ VARIANT_BOOL) PURE;
STDDISP(OnToolBar)(THIS_ VARIANT_BOOL) PURE;
STDDISP(OnMenuBar)(THIS_ VARIANT_BOOL) PURE;
STDDISP(OnStatusBar)(THIS_ VARIANT_BOOL) PURE;
STDDISP(OnFullScreen)(THIS_ VARIANT_BOOL) PURE;
STDDISP(OnTheaterMode)(THIS_ VARIANT_BOOL) PURE;
STDDISP(WindowSetResizable)(THIS_ VARIANT_BOOL) PURE;
STDDISP(WindowSetLeft)(THIS_ long) PURE;
STDDISP(WindowSetTop)(THIS_ long) PURE;
STDDISP(WindowSetWidth)(THIS_ long) PURE;
STDDISP(WindowSetHeight)(THIS_ long) PURE;
STDDISP(WindowClosing)(THIS_ VARIANT_BOOL,VARIANT_BOOL*) PURE;
STDDISP(ClientToHostWindow)(THIS_ long*,long*) PURE;
STDDISP(SetSecureLockIcon)(THIS_ long) PURE;
STDDISP(FileDownload)(THIS_ VARIANT_BOOL*) PURE;
};
---------------------------------------------

Discussion

  • Danny Smith
    Danny Smith
    2007-07-12

    • milestone: --> IINR_-_Include_In_Next_Release
    • assigned_to: nobody --> dannysmith
    • status: open --> closed
     
  • Danny Smith
    Danny Smith
    2007-07-12

    Logged In: YES
    user_id=11494
    Originator: NO

    The problem you report has caused modification in some fashion in the official CVS for the given package. The w32api and
    mingw-runtime official CVS reside in the winsup CVS directory tree for Cygwin. Those package CVS trees are periodically
    merged into the MinGW CVS tree. If you still find problems then please open a new report.