dmcompos: Partially implement Chord Map track GetParam/SetParam methods.

dlls/dmcompos/chordmaptrack.c

@@ -117,20 +117,38 @@ static HRESULT WINAPI chordmap_track_Play(IDirectMusicTrack8 *iface, void *pStat
 	return S_OK;
 }
 
-static HRESULT WINAPI chordmap_track_GetParam(IDirectMusicTrack8 *iface, REFGUID rguidType,
-        MUSIC_TIME mtTime, MUSIC_TIME *pmtNext, void *pParam)
+static HRESULT WINAPI chordmap_track_GetParam(IDirectMusicTrack8 *iface, REFGUID type,
+        MUSIC_TIME time, MUSIC_TIME *next, void *param)
 {
-        IDirectMusicChordMapTrack *This = impl_from_IDirectMusicTrack8(iface);
-	FIXME("(%p, %s, %d, %p, %p): stub\n", This, debugstr_dmguid(rguidType), mtTime, pmtNext, pParam);
-	return S_OK;
+    IDirectMusicChordMapTrack *This = impl_from_IDirectMusicTrack8(iface);
+
+    TRACE("(%p, %s, %d, %p, %p)\n", This, debugstr_dmguid(type), time, next, param);
+
+    if (!type)
+        return E_POINTER;
+    if (!IsEqualGUID(type, &GUID_IDirectMusicChordMap))
+        return DMUS_E_GET_UNSUPPORTED;
+
+    FIXME("GUID_IDirectMusicChordMap not handled yet\n");
+
+    return S_OK;
 }
 
-static HRESULT WINAPI chordmap_track_SetParam(IDirectMusicTrack8 *iface, REFGUID rguidType,
-        MUSIC_TIME mtTime, void *pParam)
+static HRESULT WINAPI chordmap_track_SetParam(IDirectMusicTrack8 *iface, REFGUID type,
+        MUSIC_TIME time, void *param)
 {
-        IDirectMusicChordMapTrack *This = impl_from_IDirectMusicTrack8(iface);
-	FIXME("(%p, %s, %d, %p): stub\n", This, debugstr_dmguid(rguidType), mtTime, pParam);
-	return S_OK;
+    IDirectMusicChordMapTrack *This = impl_from_IDirectMusicTrack8(iface);
+
+    TRACE("(%p, %s, %d, %p)\n", This, debugstr_dmguid(type), time, param);
+
+    if (!type)
+        return E_POINTER;
+    if (!IsEqualGUID(type, &GUID_IDirectMusicChordMap))
+        return DMUS_E_SET_UNSUPPORTED;
+
+    FIXME("GUID_IDirectMusicChordMap not handled yet\n");
+
+    return S_OK;
 }
 
 static HRESULT WINAPI chordmap_track_IsParamSupported(IDirectMusicTrack8 *iface, REFGUID rguidType)

dlls/dmcompos/tests/dmcompos.c

@@ -321,12 +321,10 @@ static void test_chordmaptrack(void)
     ok(hr == S_OK, "IDirectMusicTrack8_EndPlay failed: %08x\n", hr);
     hr = IDirectMusicTrack8_Play(dmt8, NULL, 0, 0, 0, 0, NULL, NULL, 0);
     ok(hr == S_OK, "IDirectMusicTrack8_Play failed: %08x\n", hr);
-    todo_wine {
     hr = IDirectMusicTrack8_GetParam(dmt8, NULL, 0, NULL, NULL);
     ok(hr == E_POINTER, "IDirectMusicTrack8_GetParam failed: %08x\n", hr);
     hr = IDirectMusicTrack8_SetParam(dmt8, NULL, 0, NULL);
     ok(hr == E_POINTER, "IDirectMusicTrack8_SetParam failed: %08x\n", hr);
-    }
 
     hr = IDirectMusicTrack8_IsParamSupported(dmt8, NULL);
     ok(hr == E_POINTER, "IDirectMusicTrack8_IsParamSupported failed: %08x\n", hr);
@@ -348,11 +346,11 @@ static void test_chordmaptrack(void)
                 "IsParamSupported(%s) failed: %08x, expected DMUS_E_TYPE_UNSUPPORTED\n",
                     unsupported[i].name, hr);
         hr = IDirectMusicTrack8_GetParam(dmt8, unsupported[i].type, 0, NULL, &chordmap);
-        todo_wine ok(hr == DMUS_E_GET_UNSUPPORTED,
+        ok(hr == DMUS_E_GET_UNSUPPORTED,
                 "GetParam(%s) failed: %08x, expected DMUS_E_GET_UNSUPPORTED\n",
                 unsupported[i].name, hr);
         hr = IDirectMusicTrack8_SetParam(dmt8, unsupported[i].type, 0, chordmap);
-        todo_wine ok(hr == DMUS_E_SET_UNSUPPORTED,
+        ok(hr == DMUS_E_SET_UNSUPPORTED,
                 "SetParam(%s) failed: %08x, expected DMUS_E_SET_UNSUPPORTED\n",
                 unsupported[i].name, hr);
     }