[PATCH] dmime: IDirectMusicPerformance8 SetNotificationHandle to set a minimum timeout if not already set

Alistair Leslie-Hughes leslie_alistair at hotmail.com
Tue Oct 1 00:48:58 CDT 2019


- Reformat to use spaces
- Changed FIXME -> TRACE
- Remove stub from trace.
- Set the mimimum timeout if one hasn't been specified.

Signed-off-by: Alistair Leslie-Hughes <leslie_alistair at hotmail.com>
---
 dlls/dmime/performance.c | 14 +++++++++-----
 1 file changed, 9 insertions(+), 5 deletions(-)

diff --git a/dlls/dmime/performance.c b/dlls/dmime/performance.c
index b7b4d01017..cc7783d9ef 100644
--- a/dlls/dmime/performance.c
+++ b/dlls/dmime/performance.c
@@ -519,12 +519,16 @@ static HRESULT WINAPI IDirectMusicPerformance8Impl_SetGraph(IDirectMusicPerforma
 static HRESULT WINAPI IDirectMusicPerformance8Impl_SetNotificationHandle(IDirectMusicPerformance8 *iface,
         HANDLE hNotification, REFERENCE_TIME rtMinimum)
 {
-  IDirectMusicPerformance8Impl *This = impl_from_IDirectMusicPerformance8(iface);
+    IDirectMusicPerformance8Impl *This = impl_from_IDirectMusicPerformance8(iface);
 
-  FIXME("(%p, %p, 0x%s): stub\n", This, hNotification, wine_dbgstr_longlong(rtMinimum));
-  This->hNotification = hNotification;
-  if (rtMinimum) This->rtMinimum = rtMinimum;
-  return S_OK;
+    TRACE("(%p, %p, 0x%s)\n", This, hNotification, wine_dbgstr_longlong(rtMinimum));
+
+    This->hNotification = hNotification;
+    if (rtMinimum)
+        This->rtMinimum = rtMinimum;
+    else if(!This->rtMinimum)
+        This->rtMinimum = 20000000; /* 2 seconds */
+    return S_OK;
 }
 
 static HRESULT WINAPI IDirectMusicPerformance8Impl_GetNotificationPMsg(IDirectMusicPerformance8 *iface,
-- 
2.17.1




More information about the wine-devel mailing list