Implement OLE object notifications, making sure to cope with the case

of the advise holder not being created because no notifications are needed.

Implement OLE object notifications, making sure to cope with the case
fe401520 · Robert Shearman · Alexandre Julliard · 40ffc286 · fe401520
Commit fe401520 authored Nov 03, 2005 by Robert Shearman Committed by Alexandre Julliard Nov 03, 2005
Hide whitespace changes
Inline Side-by-side

Showing with 14 additions and 3 deletions

defaulthandler.c dlls/ole32/defaulthandler.c +14 -3

No files found.
--- a/dlls/ole32/defaulthandler.c
+++ b/dlls/ole32/defaulthandler.c
@@ -1339,13 +1339,23 @@ static void WINAPI DefaultHandler_IAdviseSink_OnRename(
    IAdviseSink *iface,
    IMoniker *pmk)
 {
-    FIXME(": stub\n");
+    DefaultHandler *This = impl_from_IAdviseSink(iface);
+
+    TRACE("(%p)\n", pmk);
+
+    if (This->oleAdviseHolder)
+        IOleAdviseHolder_SendOnRename(This->oleAdviseHolder, pmk);
 }

 static void WINAPI DefaultHandler_IAdviseSink_OnSave(
    IAdviseSink *iface)
 {
-    FIXME(": stub\n");
+    DefaultHandler *This = impl_from_IAdviseSink(iface);
+
+    TRACE("()\n");
+
+    if (This->oleAdviseHolder)
+        IOleAdviseHolder_SendOnSave(This->oleAdviseHolder);
 }

 static void WINAPI DefaultHandler_IAdviseSink_OnClose(
@@ -1355,7 +1365,8 @@ static void WINAPI DefaultHandler_IAdviseSink_OnClose(
    
    TRACE("()\n");

-    IOleAdviseHolder_SendOnClose(This->oleAdviseHolder);
+    if (This->oleAdviseHolder)
+        IOleAdviseHolder_SendOnClose(This->oleAdviseHolder);

    DefaultHandler_Stop(This);
 }